Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement time-aware slider ticks as found in eodash #186

Merged
merged 18 commits into from
Nov 15, 2023

Conversation

spectrachrome
Copy link
Member

@spectrachrome spectrachrome commented Aug 8, 2023

This pull request takes the slider ticks implementation from eodash and translates it into a Lit component so that it can also easily be used by other applications.

Screen.Recording.2023-11-15.at.11.06.21.mov

The logic generating the year marks has been fixed and is now working properly.

@spectrachrome spectrachrome self-assigned this Aug 8, 2023
@spectrachrome spectrachrome force-pushed the timecontrol/feature/sliderticks branch from 2cdfe75 to 318e99a Compare August 8, 2023 08:38
@spectrachrome spectrachrome changed the title Implement eodash slider ticks Implement slider ticks as found in eodash Aug 8, 2023
@spectrachrome spectrachrome changed the title Implement slider ticks as found in eodash Implement time-aware slider ticks as found in eodash Aug 8, 2023
@spectrachrome spectrachrome marked this pull request as ready for review August 8, 2023 09:21
@spectrachrome spectrachrome marked this pull request as draft August 10, 2023 08:09
@silvester-pari silvester-pari changed the base branch from timecontrol/main to main October 17, 2023 08:59
@spectrachrome spectrachrome marked this pull request as ready for review November 15, 2023 10:20
@spectrachrome spectrachrome requested review from silvester-pari and removed request for silvester-pari November 15, 2023 10:20
Copy link
Collaborator

@silvester-pari silvester-pari left a comment

Choose a reason for hiding this comment

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

Great addition, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants