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(a11y): migrate NcModal to NcDialog p2 #12997

Merged
merged 10 commits into from
Aug 16, 2024
Merged

Conversation

Antreesy
Copy link
Contributor

@Antreesy Antreesy commented Aug 15, 2024

☑️ Resolves

🖌️ UI Checklist

🖼️ Screenshots / Screencasts

🏚️ Before 🏡 After
image image
image image
image image
image image
image image
image image

🏁 Checklist

  • 🌏 Tested with different browsers / clients:
    • Chromium (Chrome / Edge / Opera / Brave)
    • Firefox
    • Safari
    • Talk Desktop
    • Not risky to browser differences / client
  • 🖌️ Design was reviewed, approved or inspired by the design team
  • ⛑️ Tests are included or not possible
  • 📗 User documentation in https://github.com/nextcloud/documentation/tree/master/user_manual/talk has been updated or is not required

@Antreesy
Copy link
Contributor Author

/backport to stable30

@Antreesy Antreesy marked this pull request as draft August 15, 2024 13:09
@Antreesy Antreesy changed the title fix(a11y): provide accessible heading to BreakoutRoomsActions fix(a11y): migrate NcModal to NcDialog p2 Aug 15, 2024
@ShGKme
Copy link
Contributor

ShGKme commented Aug 15, 2024

Ready to review or in progress?

@Antreesy
Copy link
Contributor Author

Ready with a fix up, but since it's not a 'drastic' visually, can wait until all modals are done

@Antreesy Antreesy force-pushed the fix/12970/migrate-ncdialog-2 branch 2 times, most recently from c9bb812 to 96efc86 Compare August 16, 2024 07:54
@Antreesy Antreesy marked this pull request as ready for review August 16, 2024 08:59
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
- set preview height in grid to 100% to prevent increasing from metadata height

Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy force-pushed the fix/12970/migrate-ncdialog-2 branch from 50ad996 to 7d34d8e Compare August 16, 2024 16:41
@Antreesy Antreesy merged commit 014f334 into main Aug 16, 2024
46 checks passed
@Antreesy Antreesy deleted the fix/12970/migrate-ncdialog-2 branch August 16, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants