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

chore: update modal accessibility and translation #1465

Merged
merged 4 commits into from
Jun 19, 2024

Conversation

Chisomchima
Copy link
Member

Implements Jira Ticket


Description

Implements Jira Ticket by updating the Modal CloseButton component to update title and aria-label attributes, as well as the close onClick of the escape button


Known issues

  • None

@Chisomchima Chisomchima requested a review from a team as a code owner March 20, 2024 11:03
@dhis2-bot
Copy link
Contributor

dhis2-bot commented Mar 20, 2024

🚀 Deployed on https://pr-1465--dhis2-ui.netlify.app

@dhis2-bot dhis2-bot temporarily deployed to netlify March 20, 2024 11:07 Inactive
@Chisomchima Chisomchima requested a review from kabaros March 20, 2024 11:09
Copy link

cypress bot commented Mar 20, 2024

4 failed tests on run #3393 ↗︎

4 580 0 0 Flakiness 0

Details:

chore: update modal accessibility and translation
Project: ui Commit: 8f7b8b84c3
Status: Failed Duration: 06:47 💡
Started: May 24, 2024 12:19 PM Ended: May 24, 2024 12:26 PM
Failed  components/sharing-dialog/src/features/add-entity.feature • 4 failed tests • e2e

View Output

Test Artifacts
Allows users to add entities to the allowed list > User can give user view only access (example #1) Screenshots
Allows users to add entities to the allowed list > User can give user view and edit access (example #2) Screenshots
Allows users to add entities to the allowed list > User can give group view only access (example #3) Screenshots
Allows users to add entities to the allowed list > User can give group view and edit access (example #4) Screenshots

Review all test suite changes for PR #1465 ↗︎

@kabaros kabaros force-pushed the LIBS-567/modal-accessibility-translation branch 4 times, most recently from 5631917 to db9d5be Compare April 25, 2024 15:34
@dhis2-bot dhis2-bot temporarily deployed to netlify April 25, 2024 15:38 Inactive
Copy link
Collaborator

@kabaros kabaros left a comment

Choose a reason for hiding this comment

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

@Chisomchima tested and worked as expected. I updated the PR to delete some files that shouldn't have been there - when you merge it, then please squash merge it.

modal-accessibility

@Chisomchima Chisomchima force-pushed the LIBS-567/modal-accessibility-translation branch from db9d5be to 61b1231 Compare May 23, 2024 15:51
@dhis2-bot dhis2-bot temporarily deployed to netlify May 23, 2024 15:55 Inactive
chore: remove unnecessary files in PR

chore: update modal accessibility and translation
@Chisomchima Chisomchima force-pushed the LIBS-567/modal-accessibility-translation branch from 61b1231 to 8f7b8b8 Compare May 24, 2024 12:10
@dhis2-bot dhis2-bot temporarily deployed to netlify May 24, 2024 12:18 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 5, 2024 15:01 Inactive
@dhis2-bot dhis2-bot temporarily deployed to netlify June 19, 2024 05:54 Inactive
@Chisomchima Chisomchima requested a review from d-rita June 19, 2024 07:10
@Chisomchima Chisomchima merged commit 72f88eb into master Jun 19, 2024
16 checks passed
@Chisomchima Chisomchima deleted the LIBS-567/modal-accessibility-translation branch June 19, 2024 09:03
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 9.8.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

4 participants