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

feat: Allows the XYZ plot to be sorted alphabetically #16448

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

zero41120
Copy link
Contributor

@zero41120 zero41120 commented Sep 1, 2024

Description

The XYZ plot currently lists options in an order that seems random, likely reflecting their UI placement. This can make locating specific options challenging, particularly as extensions are adding items to the list.

In this PR:

  • added a simple option to sort XYZ Plot options alphabetically
  • created a copy of axis_options and applied sorting during the XYZ Plot UI rendering
  • kept "Nothing" as the first option

Screenshots/videos:

Before After
Options are organized by functionality, with related items like Sampler and Hires-Sampler placed together. Options are sorted alphabetically.
Before Screenshot After Screenshot

Settings option:
Screenshot 2024-08-31 at 16 51 56

Checklist:

  • I have read contributing wiki page
  • I have performed a self-review of my own code
  • My code follows the style guidelines
  • My code passes tests
    • Unable to verify tests fully. My MacBook Pro M3pro on master without any changes does not pass all tests.
    • Changes in this PR are UI only. No changes to Python or other server-side code. It is unlikely that these changes will affect existing functionality.
    • Github test passes

@zero41120 zero41120 changed the title Feat sort xyz plot feat: Allows the XYZ plot to be sorted alphabetically Sep 1, 2024
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.

1 participant