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.

Sign up for GitHub

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

Jump to bottom

Schedule - Current time line #81

Open
driezie opened this issue Nov 19, 2024 · 2 comments
Open

Schedule - Current time line #81

driezie opened this issue Nov 19, 2024 · 2 comments
Assignees
@driezie
Labels
Feature to make a feature

Comments

@driezie
Copy link
Collaborator

driezie commented Nov 19, 2024

Ik wil een live marker hebben waar je duidelijk kan zien wat de tijd van vandaag is.

Image
@driezie driezie moved this to In Progress in Mediahuis Nov 19, 2024
@driezie driezie self-assigned this Nov 20, 2024
@driezie driezie added the Feature to make a feature label Nov 20, 2024
@driezie
Copy link
Collaborator Author

driezie commented Nov 20, 2024

Counter bug

Ik ontdekte dat de gids soms onmogelijke tijdwaarden genereerde, zoals 14:88, terwijl minuten maximaal tot 60 kunnen gaan. Om dit op te lossen, heb ik een berekening aangepast: de waarde wordt nu gedeeld door 100 en vervolgens vermenigvuldigd met 60. Hierdoor worden de tijden correct weergegeven.

Dankzij deze aanpassing blijft de lijn nu op de juiste positie gebaseerd op de tijd. Als ik dit niet goed had gecontroleerd, was deze fout mogelijk op de gids gebleven

Reduced motion

Met reduced motion heb ik de tijdlijn verwijderd, omdat deze constant beweegt. Dit kan mogelijk irritatie of motion sickness veroorzaken.

Image

@driezie
Copy link
Collaborator Author

driezie commented Nov 22, 2024

Dit probleem heb ik gefixed in de volgende commit

driezie added a commit that referenced this issue Nov 29, 2024
@driezie
Added push
aefa02b 
Schedule - Timeline - Dikgedrukte tijden #35

Schedule - Horizontal Sticky scrolling #80

Schedule - Current time line #81

Schedule - Clock animatie op basis van begin tijd #94
@driezie driezie moved this from In Progress to Testing in Mediahuis Nov 29, 2024
@driezie driezie moved this from Testing to Done in Mediahuis Nov 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@driezie driezie

Labels
Feature to make a feature
Projects
Mediahuis
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@driezie