Skip to content

revert calendar#1010

Merged
yuhattor merged 1 commit intomasterfrom
calendar-revert
Apr 11, 2025
Merged

revert calendar#1010
yuhattor merged 1 commit intomasterfrom
calendar-revert

Conversation

@yuhattor
Copy link
Copy Markdown
Member

@yuhattor yuhattor commented Apr 11, 2025

JavaScript functionality updates:

  • assets/js/script.js: Added a preloader that fades out when the window loads, dynamically embedded a Google Calendar iframe based on the user's timezone, and implemented accordion behavior for collapsible elements. Additionally, a slider was added for better content presentation, and a citation link toggle was implemented.

Content updates:

  • content/en/calendar/_index.md: Added a new section with a welcome message and a container for the calendar to enhance the calendar page's content and structure.

@yuhattor yuhattor requested a review from a team as a code owner April 11, 2025 16:48
@yuhattor yuhattor merged commit 8a62006 into master Apr 11, 2025
1 check failed
@yuhattor yuhattor deleted the calendar-revert branch April 11, 2025 16:49
@dellagustin-sap
Copy link
Copy Markdown
Contributor

Hi @yuhattor , do we need the timezone detection?
On my simple tests, when I access https://calendar.google.com/calendar/embed?src=c_62694f414055ac569e5cb12dafbb0890ca22f3640b177a4b10b53171fbc9bdd4%40group.calendar.google.com, without the ctz parameter, google calendar detects my timezone correctly.

dellagustin-sap pushed a commit to dellagustin-sap/innersourcecommons.org that referenced this pull request Jul 9, 2025
Removed the hardcoded timezone on google calendar (`ctz` url
parameter).

As per my tests, without the `ctz` paramenter, the calendar
is shown at my own timezone, so I assume there is some
automatic detection feature in it.

Was added with a15e770.

Related comment:
- InnerSourceCommons#1010 (comment)
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