Include :datetime, :date, and :time with style options only#1077
Include :datetime, :date, and :time with style options only#1077
Conversation
catamorphism
left a comment
There was a problem hiding this comment.
LGTM with Addison's suggested changes
Co-authored-by: Addison Phillips <[email protected]>
sffc
left a comment
There was a problem hiding this comment.
We haven't agreed to land :date and :time.
|
I've been clear about my concern with |
|
I tend to agree with Shane's statement there. I see no value in the 3 way split; just more cumbersome for callers. Here's what he said:
|
|
An additional ergonomic issue with |
|
@sffc noted:
That temptation is not always wrong. UX designers sometimes want to enrich the text by separating the fields. @macchiati noted:
The functions As noted in the intro, this is NOT the final solution to date/time formatting. It's a stopgap while we decide on additional functions. I'm not opposed to reducing it to just |
|
To follow up on today's call: I'm not fully convinced with pulling out I would not be happy with an answer of "it was in MF1" because I have demonstrated how MF1 can be written on top of MF2 Semantic Skeletons. It needs to be independently motivated. If we move forward with
In summary, 4 of the formats, perhaps the most common ones, are fairly easy to add in this framework without being very opinionated or making any decisions that are hard to roll back. |
|
closed without merging. superseded by 1083 |
As discussed on the 2025-06-02 call, this PR removes the "Draft" status from the
:datetime,:date, and:timefunctions, while leaving their field options as "Draft". The intent here is to allow at least some datetime formatting to be made available while discussions are #1067 are ongoing.