Skip to content

Fix screen view issue with scroll#2

Open
m-danaee wants to merge 2 commits into
macan-dev:mainfrom
m-danaee:main
Open

Fix screen view issue with scroll#2
m-danaee wants to merge 2 commits into
macan-dev:mainfrom
m-danaee:main

Conversation

@m-danaee

@m-danaee m-danaee commented May 4, 2026

Copy link
Copy Markdown

No description provided.

Copilot AI and others added 2 commits May 4, 2026 08:18
fix: make top config form scrollable on small/fullscreen displays
Copilot AI review requested due to automatic review settings May 4, 2026 09:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the PySide main window so the top input form can scroll instead of forcing the whole window content off-screen. It fits the codebase by improving usability of the single-window scanner UI in main.py.

Changes:

  • Wrap the top form layout in a QScrollArea.
  • Disable the form area's horizontal scrollbar and remove its frame.
  • Add stylesheet rules so the new scroll area matches the existing dark theme.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread main.py
Comment on lines +747 to +752
form_scroll = QScrollArea()
form_scroll.setWidgetResizable(True)
form_scroll.setFrameShape(QFrame.NoFrame)
form_scroll.setHorizontalScrollBarPolicy(Qt.ScrollBarAlwaysOff)
form_scroll.setWidget(form_widget)
outer.addWidget(form_scroll)

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot apply changes based on this feedback

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants