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.

Sign up for GitHub

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

Jump to bottom

fix(NcDialog): Ensure the dialog is correctly labelled by its name #5771

Merged
merged 2 commits into from
Jul 4, 2024
Merged

fix(NcDialog): Ensure the dialog is correctly labelled by its name #5771

merged 2 commits into from
Jul 4, 2024

Commits on Jul 3, 2024

  1. fix(NcDialog): Ensure the dialog is correctly labelled by its name 

    * Added `labelId` prop to NcModal to allow label modal without a name
* Pass the navigation id (the ID of the headline element of NcDialog) to `labelId`
* Added cypress tests

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
    @susnux
    susnux committed Jul 3, 2024
    Configuration menu
    Copy the full SHA
    7245785 View commit details
    Browse the repository at this point in the history

  2. fix(NcAppSettingsDialog): Make sure sections are correctly labelled 

    Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
    @susnux
    susnux committed Jul 3, 2024
    Configuration menu
    Copy the full SHA
    4ae8678 View commit details
    Browse the repository at this point in the history