Skip to content

Conversation

@Mauritz8
Copy link
Contributor

@Mauritz8 Mauritz8 commented May 6, 2024

When the user rating graph initially loads, it does not display the selected timespan. This pull request addresses this issue.
Before:
image
After:
image

initial.isSame(endDate.subtract(b.duration))
? `<button class = "btn-rack__btn active">${b.t}</button>`
: `<button class = "btn-rack__btn">${b.t}</button>`;

Copy link
Collaborator

Choose a reason for hiding this comment

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

please refactor to remove code duplication

Copy link
Collaborator

@ornicar ornicar left a comment

Choose a reason for hiding this comment

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

looking good

@Mauritz8 Mauritz8 requested a review from ornicar May 6, 2024 09:25
@ornicar ornicar merged commit 02a5615 into lichess-org:master May 6, 2024
@Mauritz8 Mauritz8 deleted the mauritz/show-selected-timespan branch July 4, 2024 16:07
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.

2 participants