Include button margins in width calculations #26781
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
#25999 added a width selector for the button block, allowing a Button to be set to 25%, 50%, 75%, or 100% width. Margins were not included in the width calculation in the initial iteration, meaning that for example two 50% buttons will not fit on the same line.
This PR includes margins in the width calculation.
How has this been tested?
Manually with the Buttons block, testing with left/center/right alignments. Tested using Seedlet and Twenty Twenty.
Testing Instructions
Add a Buttons block with several buttons and change their width using the width selector in the Inspector Controls.
Screenshots
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist:
Fixes #26702