Skip to content

ENH: Tick Font Size Sets Y-Axes Tick Font As Well#167

Merged
zdomke merged 2 commits into
ui_redesignfrom
zdomke/axis-tick-font-size
Jul 3, 2025
Merged

ENH: Tick Font Size Sets Y-Axes Tick Font As Well#167
zdomke merged 2 commits into
ui_redesignfrom
zdomke/axis-tick-font-size

Conversation

@zdomke

@zdomke zdomke commented Jul 2, 2025

Copy link
Copy Markdown
Collaborator

Description

Previously, the x_axis_font_size_spinbox only affected the font size for ticks on the X-Axis. This PR will set the font size for ticks of all axes. This also sets the font size of new axes to match the font size of existing axes.

Motivation

This was requested in a meeting in October. Part of the reason behind the request was because Trace looked odd when the font was larger for only one axis.

Closes #74
Closes SWAPPS-51

Screenshots

Before: Only the font size for the X-Axis changed
Screenshot 2025-07-02 at 14 01 50

After: Set the font size for all axes (X & Y)
Screenshot 2025-07-02 at 14 01 08

After: Change the setting name in PlotSettingsModal changed from "X Axis Font Size" to "Axis Tick Font Size"
Screenshot 2025-07-02 at 14 01 01

@zdomke zdomke requested a review from a team July 2, 2025 21:09
@zdomke zdomke self-assigned this Jul 2, 2025
@zdomke zdomke added the enhancement New feature or request label Jul 2, 2025

@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.

This works for me. Would there be a reason to have individual size control for each axis?

@zdomke

zdomke commented Jul 3, 2025

Copy link
Copy Markdown
Collaborator Author

This works for me. Would there be a reason to have individual size control for each axis?

I can't think of a reason. We can put that feature in if someone requests it. It shouldn't be too hard ¯\_(ツ)_/¯

@zdomke zdomke merged commit df49f54 into ui_redesign Jul 3, 2025
1 check passed
@zdomke zdomke deleted the zdomke/axis-tick-font-size branch July 3, 2025 16:40
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.

2 participants