Actions
Improvements #11001
closedFlexLayoutImpl: consider using gap for spacing instead of margins
Start date:
10/12/2022
Due date:
% Done:
100%
Estimated time:
Description
At the moment, when WBoxLayout is implemented using FlexLayoutImpl, any spacing is done using a margin.
This leads to an inconsistency between how flex layouts are rendered and how JavaScript layouts are rendered when the spacing is more than 2 times the contents margins.
I added an example to demonstrate what I'm talking about. It seems to me that gap (row-gap or column-gap) is really the appropriate property to set in order to make sure rendering is consistent.
I noticed this while working on issue #10967, and saw that the spacing variable was unused in the adjust function of FlexLayoutImpl.js.
Files
Updated by Matthias Van Ceulebroeck over 1 year ago
- Assignee set to Romain Mardulyn
- Target version changed from future to 4.11.2
Updated by Romain Mardulyn over 1 year ago
- Status changed from New to InProgress
Updated by Romain Mardulyn over 1 year ago
- Status changed from InProgress to Review
- Assignee deleted (
Romain Mardulyn)
Updated by Matthias Van Ceulebroeck over 1 year ago
- Assignee set to Matthias Van Ceulebroeck
Updated by Matthias Van Ceulebroeck over 1 year ago
- Status changed from Review to Implemented @Emweb
- Assignee changed from Matthias Van Ceulebroeck to Romain Mardulyn
- Target version changed from 4.11.2 to 4.11.1
- % Done changed from 0 to 100
Updated by Matthias Van Ceulebroeck over 1 year ago
- Status changed from Implemented @Emweb to Implemented @Test
Updated by Matthias Van Ceulebroeck about 1 year ago
- Status changed from Implemented @Test to Closed
Actions