Fix #33553: Allow Style System page to resize vertically

[officialasishkumar]

Resolves: #33553

The System page in Format > Style used a plain page widget, so its full size hint became the dialog minimum height. This wraps the page contents in a frameless scroll area, matching other tall Style pages, so the dialog can resize vertically while all controls stay reachable.

Validation:

• xmllint --noout src/notationscene/widgets/editstyle.ui

• git diff --check

• I signed the CLA

• The title of the PR describes the problem it addresses

• Each commit's message describes its purpose and effects, and references the issue it resolves

• If changes are extensive, there is a sequence of easily reviewable commits

• The code in the PR follows the coding rules

• There are no unnecessary changes

• The code compiles and runs on my machine, preferably after each commit individually

• I created a unit test or vtest to verify the changes I made (if applicable)

[claude]

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a8617e0f-a917-4e42-b01d-0b2ab8ed803b

📥 Commits

Reviewing files that changed from the base of the PR and between 81d6e24 and 49acc44.

📒 Files selected for processing (1)

• src/notationscene/widgets/editstyle.ui

---

📝 Walkthrough

Walkthrough

This pull request adds a scrollable container to the PageSystem page of the EditStyle UI dialog. A new QScrollArea widget named scrollArea_system is introduced with an internal layout structure (verticalLayout_system) to host system-related controls. The widget hierarchy is properly closed with updated XML tags, and the scroll area is registered in the dialog's keyboard tab stop order to ensure proper focus navigation.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and accurately describes the main change: fixing the vertical resizability of the Style System page dialog by referencing issue #33553.
Description check	✅ Passed	The description includes issue reference, clear motivation, and completion of key checklist items. Missing CLA signature and compilation verification are non-critical.
Linked Issues check	✅ Passed	The changes directly address issue #33553 by wrapping the System page contents in a frameless scroll area to allow vertical resizing of the dialog.
Out of Scope Changes check	✅ Passed	All changes are contained to the editstyle.ui file and directly relate to the objective of adding scroll area support for vertical resizing.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cbjeukendrup]

Is this AI-generated? For editing .ui files, it's better to use Qt's official "Designer" application.