Skip to content

Support percentage accordion padding - #2360

Open
TaxolYang wants to merge 2 commits into
nikitabobko:mainfrom
TaxolYang:fix/accordion-padding-percent
Open

TaxolYang wants to merge 2 commits into
nikitabobko:mainfrom
TaxolYang:fix/accordion-padding-percent

Conversation

@TaxolYang

Copy link
Copy Markdown

Allow accordion-padding to be specified as a percentage of the monitor’s visible width or height, so the same configuration scales across displays. Integer point values keep their existing behavior. Percentage padding is capped at half the container dimension in nested layouts.

Addresses #599.

Validation:

  • Added config parsing and padding resolution tests, including invalid values and nested containers.
  • Checked Swift syntax and ran the parsing/resolution code in an isolated Swift harness.
  • Full swift test could not compile on this machine: the Command Line Tools installation lacks the SwiftUIMacros plugin. CI and full application verification are still needed.

@github-actions github-actions Bot added the not-actionable By default, all PRs have this label. If the PR makes sense, the label will be removed. label Oct 28, 2026
@TaxolYang
TaxolYang marked this pull request as ready for review October 28, 2026 08:56

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-actionable By default, all PRs have this label. If the PR makes sense, the label will be removed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant