Skip to content

Conversation

@karwosts
Copy link
Member

@karwosts karwosts commented Nov 18, 2025

Proposed change

Expose location for calendar events. Minor CSS tweaks.

Some before and after comparisons below

Before (no description no location):

State info alignment feels off here, the gap between the state-info ("Calendar 2" and "3 days ago") seems too big, it's supposed to be lined up with the lower calendar icon but it's spilling into the area next to the upper icon?

image

After:

image

Before (location, no description):

image

After (the entity name and location here just happened to be the same):

image

Before (location & description):

image

After:

image

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

const data: CalendarEventMutableParams = {
summary: this._summary,
description: this._description,
location: this._location || (this._hasLocation ? "" : undefined),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was trying to avoid here populating location key with an empty string where it wasn't previously present (I think every calendar integration supports location, but I'm not 100% sure).

But once location has a value, neither undefined or null will clear it, so we have to use empty string in that case.

@MindFreeze MindFreeze merged commit ba9c7f3 into home-assistant:dev Nov 19, 2025
15 checks passed
@karwosts karwosts deleted the calendar-location branch November 19, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Calendar event dialog does not show or allow editing the location (the backend supports it)

2 participants