Skip to content

ENH: Add Axis Settings Modal#137

Merged
zdomke merged 7 commits into
ui_redesignfrom
zdomke/axis_settings
Apr 22, 2025
Merged

ENH: Add Axis Settings Modal#137
zdomke merged 7 commits into
ui_redesignfrom
zdomke/axis_settings

Conversation

@zdomke

@zdomke zdomke commented Apr 16, 2025

Copy link
Copy Markdown
Collaborator

Add a modal for controlling the settings of an axis.

The modal class, AxisSettingsModal, takes in 3 arguments:

  • parent: QWidget
    • This is the parent object of the modal. Should be whatever button opens the modal
  • plot: PyDMArchiverTimePlot
    • This is the main plot of Trace
    • This is only used for redrawing the plot when the orientation changes. This should probably be done in PyDM instead of here
  • axis: BasePlotAxisItem
    • The axis item that is controlled by the modal

The modal contains settings for:

  • Changing the axis' orientation (left/right side)
  • Enabling/disabling log mode for the axis
  • Enabling/disabling gridlines on the axis
    • The opacity relies on the opacity slider from the PlotSettingsModal. This is done to keep the gridline opacity consistent across the plot

@zdomke zdomke self-assigned this Apr 17, 2025
@zdomke zdomke added the enhancement New feature or request label Apr 17, 2025
@zdomke zdomke added this to the Trace 1.5 milestone Apr 17, 2025
@zdomke zdomke linked an issue Apr 17, 2025 that may be closed by this pull request
@zdomke zdomke requested a review from a team April 17, 2025 18:06
@zdomke zdomke marked this pull request as ready for review April 17, 2025 18:06

@ngAlexander9 ngAlexander9 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

okay!

@zdomke

zdomke commented Apr 21, 2025

Copy link
Copy Markdown
Collaborator Author

Just realized that I didn't include a way of changing the axis' name. This should be included in this settings modal, similar to the name in #138.

@zdomke zdomke force-pushed the zdomke/axis_settings branch from e4568fc to 56df06a Compare April 22, 2025 16:55
@zdomke zdomke requested a review from YektaY April 22, 2025 17:25
@zdomke

zdomke commented Apr 22, 2025

Copy link
Copy Markdown
Collaborator Author

Just realized that I didn't include a way of changing the axis' name. This should be included in this settings modal, similar to the name in #138.

Holding off on this change for now. I talked with @shilorigins and we might make the axis labels in #139 editable instead.

@YektaY YektaY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks good to me--you may want to add docstrings to some of the new methods. but that can also be done in a later pass

@zdomke zdomke merged commit f28dc2a into ui_redesign Apr 22, 2025
@shilorigins shilorigins deleted the zdomke/axis_settings branch April 22, 2025 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] - Redesign Trace

3 participants