Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the tab order in dialogs #21395

Merged
merged 3 commits into from
Oct 7, 2024
Merged

Fix the tab order in dialogs #21395

merged 3 commits into from
Oct 7, 2024

Conversation

thalieht
Copy link
Contributor

@thalieht thalieht commented Sep 26, 2024

  • Torrent options
  • Torrent creator
  • Preferences

In "torrent creator" the tabChangesFocus property is not set for the textEdit fields so tab is trapped in there AFAIK. Is there any reason not to set it?

The options dialog is a mess and i don't know how to fix it. e.g. in Behavior the second tabstop in the list of the .ui file is the language and yet it takes 15 tabs to get there.
Figured it out.

Closes #21387.

@thalieht thalieht added the GUI GUI-related issues/changes label Sep 26, 2024
@thalieht thalieht changed the title Fix the tab order in 2 dialogs Fix the tab order in 3 dialogs Sep 27, 2024
src/gui/optionsdialog.ui Outdated Show resolved Hide resolved
src/gui/optionsdialog.ui Outdated Show resolved Hide resolved
@Chocobo1 Chocobo1 mentioned this pull request Oct 5, 2024
@Chocobo1
Copy link
Member

Chocobo1 commented Oct 5, 2024

@thalieht
Please rebase it on master.

<item row="3" column="0">
<widget class="QCheckBox" name="checkExportDirFin">
<property name="text">
<string>Copy .torrent files for finished downloads to:</string>
</property>
</widget>
</item>
<item row="3" column="1">
<widget class="FileSystemPathLineEdit" name="textExportDirFin" native="true"/>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I moved this here manually because tabstop wasn't enough to fix the order.

@Chocobo1 Chocobo1 added this to the 5.1 milestone Oct 6, 2024
@Chocobo1 Chocobo1 requested a review from a team October 6, 2024 08:57
Copy link
Member

@glassez glassez left a comment

Choose a reason for hiding this comment

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

Untested. Formally approved so that you don't have to wait for my approval later.

@Chocobo1 Chocobo1 changed the title Fix the tab order in 3 dialogs Fix the tab order in dialogs Oct 7, 2024
@Chocobo1 Chocobo1 merged commit d73201c into qbittorrent:master Oct 7, 2024
14 checks passed
@Chocobo1
Copy link
Member

Chocobo1 commented Oct 7, 2024

@thalieht
Thank you!

@thalieht thalieht deleted the tabstops branch October 7, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI GUI-related issues/changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tab Navigation (keyboard) on the Torrent options window
4 participants