Skip to content
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

fix: Ensure the dialog name can be wrapped if needed #1074

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

susnux
Copy link
Contributor

@susnux susnux commented Oct 17, 2023

Tested the library with 320px width and noticed the heading did not wrap, so this needs to be fixed for accessibility.

@susnux susnux added bug Something isn't working 3. to review labels Oct 17, 2023
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

Merging #1074 (8b93e5e) into main (1e08381) will not change coverage.
Report is 4 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1074   +/-   ##
=======================================
  Coverage   31.15%   31.15%           
=======================================
  Files          15       15           
  Lines        1194     1194           
  Branches       60       60           
=======================================
  Hits          372      372           
  Misses        764      764           
  Partials       58       58           

@skjnldsv skjnldsv merged commit eae05f7 into main Oct 17, 2023
10 checks passed
@skjnldsv skjnldsv deleted the fix/a11y-on-small-screens branch October 17, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants