Skip to content

fix: sanitize generated room item HTML id to remove unsupported characters (including dots) - MEED-10200 - Meeds-io/meeds#4028#489

Merged
hakermi merged 1 commit intofeature/maintenancefrom
TASKK-10200
Feb 17, 2026
Merged

fix: sanitize generated room item HTML id to remove unsupported characters (including dots) - MEED-10200 - Meeds-io/meeds#4028#489
hakermi merged 1 commit intofeature/maintenancefrom
TASKK-10200

Conversation

@hakermi
Copy link
Member

@hakermi hakermi commented Feb 17, 2026

Prior to this change, the generated HTML id could contain dots and other non-alphanumeric characters coming from spaceId or dmMemberId.

Although dots are technically valid in HTML ids, they cause issues when used in CSS selectors (requiring escaping) and can lead to unexpected behavior in querySelector usage.

This change restricts the id to [A-Za-z0-9_-] only and replaces any other character with an underscore to ensure safe and predictable DOM, CSS, and JavaScript usage.

…cters (including dots) - MEED-10200 - Meeds-io/meeds#4028

Prior to this change, the generated HTML id could contain dots and other
non-alphanumeric characters coming from spaceId or dmMemberId.

Although dots are technically valid in HTML ids, they cause issues when
used in CSS selectors (requiring escaping) and can lead to unexpected
behavior in querySelector usage.

This change restricts the id to [A-Za-z0-9_-] only and replaces any
other character with an underscore to ensure safe and predictable
DOM, CSS, and JavaScript usage.
@hakermi hakermi requested a review from ahamdi February 17, 2026 06:46
@hakermi hakermi enabled auto-merge (squash) February 17, 2026 06:46
@sonarqubecloud
Copy link

@hakermi hakermi merged commit c4a28f8 into feature/maintenance Feb 17, 2026
8 checks passed
@hakermi hakermi deleted the TASKK-10200 branch February 17, 2026 13:56
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