Skip to content

Conversation

@vyakart
Copy link

@vyakart vyakart commented Dec 15, 2025

Summary

  • Fixes Fix calendar title #16 by adding the standard NAME property to iCalendar files
  • Ensures calendar titles are properly displayed in iOS Calendar and Thunderbird
  • Maintains backward compatibility with X-WR-CALNAME for older clients

Problem

iOS Calendar and Thunderbird were ignoring calendar titles because the layout only used the non-standard X-WR-CALNAME property.

Solution

Added the RFC 7986 standard NAME property alongside X-WR-CALNAME:

  • NAME: RFC 7986 official standard - recognized by iOS and modern clients
  • X-WR-CALNAME: Legacy property - maintained for backward compatibility

Testing

  • Verified iCalendar layout syntax
  • Confirmed dual-property approach follows RFC 7986 specification
  • Maintains compatibility with existing calendar clients

References

The mobile navigation was using :hover CSS pseudo-class which does not
work well on touch devices. On mobile, :hover is triggered on tap but
immediately deactivated, causing the menu to collapse instantly.

This fix adds JavaScript-based toggle functionality that:
- Keeps the menu open after tapping the first menu item
- Closes the menu when clicking outside the navigation
- Closes the menu when clicking any navigation link
- Only activates on mobile screens (max-width: 40rem)

The solution adds a mobile-menu-open class that works with the existing
CSS mobile navigation pattern from classless.css.

Fixes blr-today#15
Fixes blr-today#16 by adding the standard NAME property alongside the legacy X-WR-CALNAME property. This ensures calendar titles are properly displayed in iOS Calendar and Thunderbird, which don't recognize the non-standard X-WR-CALNAME property.

The NAME property is part of RFC 7986 and is the official standard for specifying calendar names. Keeping X-WR-CALNAME maintains backward compatibility with older calendar clients.
@netlify
Copy link

netlify bot commented Dec 15, 2025

Deploy Preview for blr-today ready!

Name Link
🔨 Latest commit 969148f
🔍 Latest deploy log https://app.netlify.com/projects/blr-today/deploys/693fa67ec8f4310008ee6ec3
😎 Deploy Preview https://deploy-preview-34--blr-today.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

Fix calendar title

1 participant