Completed
Last Updated: 21 May 2024 15:34 by Fabien
Fabien
Created on: 01 Mar 2024 10:31
Type: Bug Report
0
Ignored border-spacing 2-value syntax

Steps to reproduce the bug

In Themebuilder:

  1. enable Advanced edit and edit the Grid component.
  2. Once there, in Grid Construction Elements, go to the GRID DEFAULT part.
  3. In the top-right element tree area, go in <div> Grid ---> <table> Table.
  4. Once there, set the Table > Border Collapse property to Separate
  5. Finally try applying two different values for vertical and horizontal spacing in the Table > Border Spacing property. If necessary, apply a visible border to cells to better see the issue.

Expected behavior : The two different values should be persisted and applied.

Actual behavior : The behavior is strange and editing one value seem to overwrite the other.

 

5 comments
Fabien
Posted on: 21 May 2024 15:34

Hello Nick,

Great, thanks for the fix :)

 

Best regards

ADMIN
Nick Iliev
Posted on: 17 May 2024 06:21

Hello Fabien,


We have just released ThemeBuilder Q2 2024, which now includes a fix for the previously discussed issue related to the border-spacing properties not being properly applied (https://github.com/telerik/themebuilder/issues/86).

 

Regards,
Nick Iliev
Progress Telerik

A brand new ThemeBuilder course was just added to the Virtual Classroom. The training course was designed to help you get started with ThemeBuilder for styling Telerik and Kendo UI components for your applications. You can check it out at https://learn.telerik.com
ADMIN
Nick Iliev
Posted on: 01 Mar 2024 13:42

Hello Fabien,

 

Thank you for sharing your observations! We can confirm that we are also observing the same side effect. I will inform you as soon as the issue is resolved.

In the meantime, please follow the instructions that you shared to bypass the problem (setting both initials to zero and then setting a different value).

 

Regards,
Nick Iliev
Progress Telerik

Love the Telerik and Kendo UI products and believe more people should try them? Invite a fellow developer to become a Progress customer and each of you can get a $50 Amazon gift voucher.

Fabien
Posted on: 01 Mar 2024 13:34

Hello Nick,

I investigated the issue more thoroughly and you are correct. More precisely, there are four cases after editing:

  • (>0 ; >0) => seems to work
  • (>0 ; 0) => both set to zero if the second value was last edited and changed to zero (no matter the old value). But it is possible to bypass the bug by setting both value to zero and then setting the first value.
  • (0 ; >0) => both set to zero if the first value was last edited and changed to zero (no matter the old value). But it is possible to bypass the bug by setting both value to zero and then setting the second value.
  • (0 ; 0) => seems to work

Thanks

ADMIN
Nick Iliev
Posted on: 01 Mar 2024 11:31

Hello Fabien,

 

Thank you for bringing this to our attention.

I can confirm that we have observed unexpected behavior when trying to set a value of 0px for a border spacing properties that already has a pre-set value. This results in both properties being set to 0px. The team will investigate and fix the issue in a future release of the application.

However, we have not been able to reproduce the issue with values that differ from 0px. Could you please let us know if the issue is reproducible with values other than 0px? If it is, we would appreciate it if you could share your project with themebuilder@progress.com by using the Share option in the top-right corner of the ThemeBuilder application. Thank you in advance!

Regards,
Nick Iliev
Progress Telerik

Love the Telerik and Kendo UI products and believe more people should try them? Invite a fellow developer to become a Progress customer and each of you can get a $50 Amazon gift voucher.