Sitemap Editor: Extend chart period config for ISO8601

[mherwege]

Adjust sitemap configuration in main UI for recent changes in sitemap chart period configuration, see openhab/openhab-core#3863.

[relativeci]

Job #1277: Bundle Size — 15.81MiB (+0.3%).

3056db5(current) vs b7270d2 main#1158(baseline)

Important

Bundle introduced 1 and removed 1 duplicate package – View changed duplicate packages

Warning

Bundle introduced 13 new packages: @jsep-plugin/regex, @jsep-plugin/arrow, @jsep-plugin/object and 10 more – View changed packages

Bundle metrics   10 changes 5 regressions 1 improvement

	Current Job #1277	Baseline Job #1158
Initial JS	1.89MiB(+13.12%)	1.67MiB
Initial CSS	609.63KiB(+0.12%)	608.89KiB
Cache Invalidation	93.81%	93.95%
Chunks	217(-0.91%)	219
Assets	683(-0.87%)	689
Modules	3027(+78.06%)	1700
Duplicate Modules	173(+92.22%)	90
Duplicate Code	1.61%(-17.44%)	1.95%
Packages	152(+10.14%)	138
Duplicate Packages	16(+6.67%)	15

Bundle size by type   3 changes 3 regressions

	Current Job #1277	Baseline Job #1158
JS	9.29MiB (+0.33%)	9.25MiB
Other	4.74MiB (+0.33%)	4.73MiB
CSS	860.87KiB (+0.16%)	859.49KiB
Fonts	526.1KiB	526.1KiB
Media	295.6KiB	295.6KiB
IMG	140.74KiB	140.74KiB
HTML	1.23KiB	1.23KiB

---

View job #1277 report View mherwege:sitemap_period_config branch activity

[mherwege]

@lolodomo FYI

[florian-h05]

Reading openhab/openhab-docs#2158, it seems that 2h, 5D etc. should still be supported, but the sitemap editor warns when trying to use them. Can you please fix that?

Also please have a look at my comment below, and consider to use the validate property of the input (see https://framework7.io/vue/inputs#input-properties) to let the user know his input is invalid while typing.

[mherwege]

Reading openhab/openhab-docs#2158, it seems that 2h, 5D etc. should still be supported, but the sitemap editor warns when trying to use them. Can you please fix that?

That should still work, and it did in my testing. There is even a testcase covering it (for 4h). It may have been the second (missing) commit.
While it will work, I opted for a placeholder for the general format.

[mherwege]

Also please have a look at my comment below, and consider to use the validate property of the input (see https://framework7.io/vue/inputs#input-properties) to let the user know his input is invalid while typing.

The validate property is now used. I still keep the separate validation as well as the a wrong value could be provided through the editor.

[mherwege]

@florian-h05 Thank you for your feedback. I believe I responded to your remarks.

[florian-h05]

LGTM, thanks!

FYI I have removed validate-on-blur.

[mherwege]

FYI I have removed validate-on-blur.

Fine with me, but I explicitely disabled it because it just gives an error already on typing the first character, even when it is part of a perfectly valid period definition. I didn't like that, especially because there are many valid combinations and checking the regex only makes sense when the input is complete. With validate-on-blur it only checks at the end.