Skip to content

x-axis labels include years and are at the starts of months#79

Merged
NickGerleman merged 1 commit intorn-versions:mainfrom
sophiebits:sophiebits--axis
May 21, 2025
Merged

x-axis labels include years and are at the starts of months#79
NickGerleman merged 1 commit intorn-versions:mainfrom
sophiebits:sophiebits--axis

Conversation

@sophiebits
Copy link
Copy Markdown
Contributor

screenshot of updated charts

@sophiebits sophiebits requested a review from a team as a code owner May 15, 2025 21:30
@sophiebits
Copy link
Copy Markdown
Contributor Author

@rickhanlonii

@NickGerleman
Copy link
Copy Markdown
Collaborator

I do not remember this code at all, so I'm just going to trust 😀.

Is there any overlap when the card is narrow, like on mobile?

Copy link
Copy Markdown
Contributor

@wojtekmaj wojtekmaj left a comment

Choose a reason for hiding this comment

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

Verified and tested locally, looks good to me 👍

@wojtekmaj
Copy link
Copy Markdown
Contributor

wojtekmaj commented May 20, 2025

@NickGerleman Nope, no overlap - just because the ticks are there doesn't mean the chart library must render them. But you can make it so that they touch :D

image

@wojtekmaj
Copy link
Copy Markdown
Contributor

wojtekmaj commented May 20, 2025

Interesting consequence of this change is that React-DOM which just recently started being tracked now shows a single point as opposed to showing daily ticks. This may or may not be a problem.

Nah it's just because the data is not up to date on this branch.

@NickGerleman NickGerleman merged commit 09fc137 into rn-versions:main May 21, 2025
3 checks passed
@sophiebits
Copy link
Copy Markdown
Contributor Author

Thanks both!

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.

3 participants