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: Change back and next button labels when at start / end (fixes #310) #311

Merged
merged 12 commits into from
Aug 19, 2024

Conversation

swashbuck
Copy link
Contributor

@swashbuck swashbuck commented Aug 1, 2024

Fix #310

Fix

  • Changes Back button's aria-label to "Back" (_globals._accessibility._ariaLabels.previous) when on the first slide
  • Changes Next button's aria-label to "Next" (_globals._accessibility._ariaLabels.next) when on the last slide
  • Various code optimizations (e.g. renaming variables)

Notes

These label values will need to be updated manually in existing courses.

Copy link
Contributor

Choose a reason for hiding this comment

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

👀

properties.schema Outdated Show resolved Hide resolved
js/NarrativeView.js Outdated Show resolved Hide resolved
Copy link
Contributor

@joe-allen-89 joe-allen-89 left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

Choose a reason for hiding this comment

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

👀

Copy link
Contributor

@kirsty-hames kirsty-hames left a comment

Choose a reason for hiding this comment

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

Working as expected thanks 👍

@swashbuck swashbuck merged commit ad1b145 into master Aug 19, 2024
@swashbuck swashbuck deleted the issue/310 branch August 19, 2024 15:39
Copy link

🎉 This PR is included in version 7.10.1 🎉

The release is available on GitHub release

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.

Disabled previous/next button aria labels
5 participants