Skip to content

7933 minor ticklabel format - #7935

Open
my-tien wants to merge 10 commits into
plotly:mainfrom
my-tien:7933-minor-ticklabel-format
Open

7933 minor ticklabel format#7935
my-tien wants to merge 10 commits into
plotly:mainfrom
my-tien:7933-minor-ticklabel-format

Conversation

@my-tien

@my-tien my-tien commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Fix #7933 by determining the tick format using the rounded ticks of the minor axis instead of the major axis when ticklabelindex is set.
my-tien added 8 commits August 4, 2026 16:01
@my-tien
Add flag useTicklabelIndex
5047961 
Previously, the ticklabelindex property itself was used in boolean tests which can be confusing.
@my-tien
Make sure period labels are not positioned below a different period.
c59fa9f
@my-tien
Mock ticklabelindex-2 to test correct ticklabel placement
9fbd6c4
@my-tien
Add draftlog for PR 7934
b12f71a
@my-tien
Add missing replacements of ticklabelindex with ax._useTicklabelIndex
c1ecb06
@my-tien
Baseline images for ticklabelindex-2 and date_axes_period2_ticklabeli…
65dbb62 
…ndex
@my-tien
When using ticklabelindex, make sure tick labels use formatting compa…
b1703cb 
…tible with minor ticks.

E.g. if major ticks are daily and minor ticks are hourly, tick labels on minor ticks should include hourly information.
@my-tien
Adjust date_axes_period2_ticklabelindex baseline image after fixing p…
77d7d26 
…eriod labels

The longer period labels need more space.
@my-tien
my-tien force-pushed the 7933-minor-ticklabel-format branch from b701674 to 77d7d26 Compare August 5, 2026 14:50
@robertclaus
robertclaus requested a review from camdecoster August 6, 2026 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@my-tien @camdecoster