Just found this module and really like it as it solves a lot of date range display needs and the admin side makes things easy for editors.
One request, no matter how I set up custom formatters for the rendered date display, there is are repeat of the event time added to the end of the string inside parentheses: 1-1-20, 2:00pm-3:00pm (2:00-3:00). That is fine for ranges within the same day, but looks weird for ranges over multiple dates: 1-1-20, 2:00pm–1-2-20, 3:00pm (2:00-3:00).
Can an option be added so a site builder can choose whether or not the duplicated time in the parentheses is displayed? I found out how to remove it in Twig: {{ item.content|without('site_time') but it could be a nice UI addition for those not great at Twig.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 3164868-add-site-time-display-toggle.patch | 7.27 KB | robertshell22 |
Comments
Comment #2
mandclu commentedHmmm if it's always an exact duplicate, that could be a configuration issue. What widget are you using? If it's the timezone widget, what do you have it set to?
Comment #3
robertshell22 commentedHere's a patch that adds a toggle checkbox to turn the Site Time info on and off in the format settings.
Comment #5
mandclu commented@robertshell22 thanks very much for the patch. The "double negative" approach is simpler (in that you don't have to update existing formats) but seems like it would be more likely to confuse site builders. I decided to reverse it, so it would be more intuitive. In other words, in the merged version the box is checked by default, and this causes the site time to display. Uncheck it to hide the site time display.
I considered adding some conditional states so this field would only be shown if there was a timezone token in the time format string, but realistically that would probably require moving the field up in the form, and I wasn't sure it was worth reorganizing the form. Open to suggestions if you have ideas on how the form could be improved.