ioBroker.admin icon indicating copy to clipboard operation
ioBroker.admin copied to clipboard

[bug] jsonConfig - typ: devider not respecting xs, sm, md, lg and xl - always 100%

Open SKB-CGN opened this issue 9 months ago • 11 comments

No existing issues.

  • [x] There is no existing issue for my request.

Problem location

"_divider0": {
         "newLine": true,
         "type": "divider",
         "color": "primary",
         "style": {
            "width": "stretch"
         },
         "xs": 12,
         "sm": 12,
         "md": 6,
         "lg": 6,
         "xl": 6
      },

Warning message

No warning!

Source link

No link

Additional information

Admin: 7.6.3

The devider always adds:

margin-bottom: 0px;
text-align: left;
width: 100%;

I think, width should be removed, then xs, sm, md, lg and xl are working fine!

SKB-CGN avatar May 28 '25 11:05 SKB-CGN

Dont think this is a schena issue.

Please add admin version where this problem has been detected.

In addition Whats the usecase where a divider should not use the full screen width?

mcm1957 avatar May 28 '25 14:05 mcm1957

I have updated the post!

SKB-CGN avatar May 28 '25 15:05 SKB-CGN

I really did not think that some one wants divider in the 70% of the screen. But you can trick with width or style: { width: '80%'}

Image

GermanBluefox avatar Jun 07 '25 13:06 GermanBluefox

Sure, that's possible - but I thought, if using the styles for xs and so on, it should respect those values as well. Only my thoughts 😉

SKB-CGN avatar Jun 07 '25 14:06 SKB-CGN

Well in my oppinion those element (xs..xl) should either work as expected OR tehy should by removed from documenation / schema and hence be deprecated.

'Valid' elements not having the specified effect are bougous in my oppinion.

mcm1957 avatar Jun 08 '25 20:06 mcm1957

Yes, correct. That was meant by me as well.

SKB-CGN avatar Jun 08 '25 20:06 SKB-CGN

Sorry no time for things that already works. I will be happy if I will see PR for that

GermanBluefox avatar Jun 16 '25 08:06 GermanBluefox

I think, a PR is not really necessary for this, if you remove the width as then it will respect all the xs ... xl.

SKB-CGN avatar Jun 18 '25 12:06 SKB-CGN

I do not see a use case for xl=6

GermanBluefox avatar Jun 20 '25 19:06 GermanBluefox

That's the case, if you have textboxes configured with xl=6 and want the divider only underneath them and not through the whole page!

SKB-CGN avatar Jun 20 '25 19:06 SKB-CGN

We could decide to not support size attributes. I too think that a seperator shoukd be ful witsth.

But as stated above either size attributes should work OR size attributes should be removed as valid elements from docu and schema.

At least a remark at docu should be added.

mcm1957 avatar Jun 20 '25 19:06 mcm1957