Conversation
Greptile SummaryThis PR improves accessibility in the template details sidebar by adding a visually-hidden
Confidence Score: 5/5Safe to merge — small, isolated accessibility improvement with no functional risk. The only finding is a P2 style suggestion to use the pre-formatted integration name in the sr-only span for better screen reader pronunciation. This does not affect functionality, correctness, or visual output. The change is minimal and well-scoped. No files require special attention.
|
| Filename | Overview |
|---|---|
| pcweb/pages/templates/template_details/views/sidebar.py | Adds a sr-only span with the raw integration_name (may contain underscores) for screen reader accessibility in integrations_circle; consider using formatted_integration_name for a more natural screen-reader announcement. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A["integrations_circle(integration_name)"] --> B["format name\n(lowercase, replace _ with space)"]
B --> C["ui.tooltip\n(trigger: icon div, content: integration_name)"]
A --> D["rx.el.span sr-only\n(integration_name) ← NEW"]
C --> E["Visual users see icon + hover tooltip"]
D --> F["Screen readers announce integration name"]
Reviews (1): Last reviewed commit: "add integration name" | Re-trigger Greptile
No description provided.