[wp-trac] [WordPress Trac] #60682: Use configured date and time format in daylight saving time notice (was: Enhanced Dynamic Date and Time Formatting for DST Transition Notifications)
WordPress Trac
noreply at wordpress.org
Mon Mar 4 09:45:15 UTC 2024
#60682: Use configured date and time format in daylight saving time notice
----------------------------+----------------------
Reporter: reklwera | Owner: (none)
Type: enhancement | Status: closed
Priority: normal | Milestone:
Component: Administration | Version:
Severity: trivial | Resolution: wontfix
Keywords: | Focuses:
----------------------------+----------------------
Changes (by swissspidy):
* status: new => closed
* focuses: ui, accessibility, administration =>
* component: Options, Meta APIs => Administration
* version: trunk =>
* milestone: Awaiting Review =>
* keywords: has-patch =>
* resolution: => wontfix
Comment:
Hi there and thanks for opening this ticket.
Unfortunately the GPT-written description is not really helpful and does
not actually explain the suggested enhancement. Also, I recommend checking
out
https://make.wordpress.org/core/handbook/tutorials/trac/submitting-a-patch/
to see how to create patch files for suggested changes. That makes it
easier to see the difference and apply the change locally.
From what I understand when looking at the code sample, you suggest using
the configured `date_format` and `time_format` values for the "Daylight
saving time begins on: %s" / "Standard time begins on: %s." sentences.
I don't think that makes sense. `date_format` could be changed by the user
to just show the year (`Y`), which would then make this sentence pretty
useless because it's lacking the day & month.
`date_format` and `time_format` in WordPress are not meant to be applied
''everywhere'' there is a date. They're mostly intended for the front end,
and not the admin.
As such, I am suggesting to close this as a ''wontfix''.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60682#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list