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

Order event display based on length #102

Closed
bruceharrison1984 opened this issue Jul 27, 2023 · 0 comments · Fixed by #104
Closed

Order event display based on length #102

bruceharrison1984 opened this issue Jul 27, 2023 · 0 comments · Fixed by #104
Labels
enhancement New feature or request

Comments

@bruceharrison1984
Copy link
Owner

bruceharrison1984 commented Jul 27, 2023

Is your feature request related to a problem? Please describe.
Currently, events are displayed in the date order. However, CSS grid doesn't specifically choose precedence based on this in condensed mode. This means short events could cause longer events to be hidden one week, and in the following week the long event could be shown. Adding an order CSS prop could aid in consistently making sure events are ordered based on length and not arbitrarily.

It would also be nice to make the user configurable, and allow shorter or longer events to take precedence.

Describe the solution you'd like
A CSS order prop, based on the total length of the event. Lower values will take hither precedence in the CSS grid.

Describe alternatives you've considered
Currently events are ordered in the DOM based on length, but it doesn't seem to affect the CSS grid order as expected.

@bruceharrison1984 bruceharrison1984 added the enhancement New feature or request label Jul 27, 2023
@bruceharrison1984 bruceharrison1984 changed the title Displays events based on length Order event display based on length Jul 28, 2023
@bruceharrison1984 bruceharrison1984 linked a pull request Jul 28, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant