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

feature: add calendar grid layout #50

Conversation

jdwillemse
Copy link
Contributor

In order to condense the events for easier consumption a grid view was added for larger views. To do this the template was updated.

  • added some wrappers
  • added classes based on the day so the layout can be consistent no matter which day the month starts on
  • on smaller screens the empty calendar days are hidden in order to reduce boilerplate

This PR affects #45

@xhain-ci
Copy link

xhain-ci commented Jan 11, 2024

Deployment of preview was successful: https://xHain-hackspace-xhain-website-pr-50.surge.sh

@pantierra
Copy link
Collaborator

Oh, wow, very nice! The calendar has been rolled back, but we are planning to bring it back. So this work won't be lost, but on-hold for a bit.

@pantierra pantierra changed the base branch from main to feature/calendar-revamp-again January 16, 2024 22:19
@pantierra pantierra merged commit 0b5e0ae into xHain-hackspace:feature/calendar-revamp-again Jan 16, 2024
1 check passed
pantierra pushed a commit that referenced this pull request Jan 16, 2024
pantierra pushed a commit that referenced this pull request Jan 16, 2024
pantierra pushed a commit that referenced this pull request Jan 16, 2024
pantierra pushed a commit that referenced this pull request Jan 17, 2024
@pantierra
Copy link
Collaborator

pantierra commented Jan 17, 2024

Merged into #52 Many thanks!

pantierra pushed a commit that referenced this pull request Jan 17, 2024
pantierra pushed a commit that referenced this pull request Jan 19, 2024
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