- Issue created by @thefancywizard
- Status changed to Needs review
11 months ago 3:45am 2 August 2023 - 🇮🇳India yash_khandelwal Mumbai
#2 patch is working fine for me, @thefancywizard, please review this patch and apply.
- 🇨🇦Canada mandclu
To clarify, for the sake of recurring dates, one is actually the default value. That's the reason it isn't currently stored.
I understand the perspective that for an editor it may seem strange to see a value explicitly set not displayed the next time the content is edited. From a data management standpoint it also seems strange to me to to have some content that doesn't store a value (because they use the default) and some that store a value (matching the default) that behave the same.
If the UX of the current widget is confusing to some, then I'm open to discussion about changing it, but I'm not sure the change in the current patch is the best way to address it. If anything, I'd be more inclined to set 1 as the default value in the form element, but still not store it.
- 🇺🇸United States sassafrass
Thanks for the patch. I found it very confusing to not have the value 1 save. It wasn't intuitive to me that it was the default; I thought it was broken.