Skip to content

PR: Allow user to configure tab close button position - #26200

Open
athompson673 wants to merge 13 commits into
spyder-ide:masterfrom
athompson673:tab-close-position
Open

PR: Allow user to configure tab close button position#26200
athompson673 wants to merge 13 commits into
spyder-ide:masterfrom
athompson673:tab-close-position

Conversation

@athompson673

@athompson673 athompson673 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

  • Wrote at least one-line docstrings (for any new functions)
  • Added unit test(s) covering the changes (if testable)
  • Included a screenshot or animation (if affecting the UI, see Licecap)

Desire for user choice to position close buttons on QTabBar's has been expressed in both directions. More complicated tab size calculation mentioned in the linked issue was not attempted due to complexity (compare to tab re-sizing on firefox for example).

Issue(s) Resolved

Fixes #26148

Affirmation

By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: athompson673

@athompson673 athompson673 changed the title Tab close position PR: Allow user to configure tab close button position Jul 22, 2026
@athompson673
athompson673 marked this pull request as ready for review July 23, 2026 01:37
@athompson673
athompson673 marked this pull request as draft July 31, 2026 14:11
@athompson673

athompson673 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Anyone have thoughts on what is causing the "Linux tests with PyQt6 / Linux - Py3.13, pyqt6, pip, fast/slow" to fail with TypeError: invalid result from CloseButtonTabBarStyle.styleHint(), 'ButtonPosition' object cannot be interpreted as an integer over and over? It's fine on all other testing configurations, and forcing the enum to an int seems to break things as enum.value is not always available.

Edit: fixed

@athompson673
athompson673 marked this pull request as ready for review August 8, 2026 19:01
@athompson673

Copy link
Copy Markdown
Contributor Author

@ccordoba12 I think the failing test at: spyder/plugins/ipythonconsole/tests/test_ipythonconsole:test_kernel_restart_after_manual_restart_and_crash should not be related to this modification? It's a new error I didn't see in earlier CI runs.

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.

Customize location to close tabs on macOS

1 participant