-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add overlayRole property to customize dialog role #7582
Conversation
@@ -64,6 +64,12 @@ describe('vaadin-dialog', () => { | |||
expect(overlay.getAttribute('role')).to.be.eql('dialog'); | |||
}); | |||
|
|||
it('should change role attribute on the overlay based on overlayRole', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: Could it be a snapshot test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, thanks. Moved this test and removed two others for default role
and theme
as they are also covered by existing snapshot tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Extracted changes to unrelated tests to #7586 so that this PR doesn't touch unit tests.
Quality Gate passedIssues Measures |
This ticket/PR has been released with Vaadin 24.5.0.alpha6 and is also targeting the upcoming stable 24.5.0 version. |
Description
Part of #4977
Added
overlayRole
property similar to the one we now have invaadin-popover
to customize the dialog role.Type of change