Skip to content

fix: Ensure error container is always in DOM (autocomplete)#136

Merged
DafyddLlyr merged 3 commits intomainfrom
dp/always-render-error-container
Mar 29, 2022
Merged

fix: Ensure error container is always in DOM (autocomplete)#136
DafyddLlyr merged 3 commits intomainfrom
dp/always-render-error-container

Conversation

@DafyddLlyr
Copy link
Copy Markdown
Contributor

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 29, 2022

Deploy Preview for oslmap ready!

Name Link
🔨 Latest commit 54cf1a5
🔍 Latest deploy log https://app.netlify.com/sites/oslmap/deploys/62430af520cd9a0008fa8e90
😎 Deploy Preview https://deploy-preview-136--oslmap.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 site settings.

Copy link
Copy Markdown
Member

@jessicamcinchak jessicamcinchak left a comment

Choose a reason for hiding this comment

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

nice solution, each render role="status" as expected for me!

Comment thread src/components/address-autocomplete/index.ts Outdated
* Render an errorMessage container
* Must always be visible to ensure that role="status" works for screenreaders
* @param errorMessage
* @returns TemplateResult
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@DafyddLlyr DafyddLlyr merged commit e175014 into main Mar 29, 2022
@DafyddLlyr DafyddLlyr deleted the dp/always-render-error-container branch March 29, 2022 13:36
@DafyddLlyr DafyddLlyr mentioned this pull request Mar 29, 2022
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