[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