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

Reorder accessibility page #129

Merged
merged 3 commits into from
Dec 9, 2024
Merged

Reorder accessibility page #129

merged 3 commits into from
Dec 9, 2024

Conversation

cjrace
Copy link
Contributor

@cjrace cjrace commented Dec 9, 2024

Overview of changes

Made some quick changes to update the order of the page, and tidy up some of the headings in the checking accessibility for spreadsheets section.

These have cleaned the page up / make it easier for me to navigate, though interested to know if the same goes for anyone else.

Why are these changes being made?

Was linking someone else to the page and found the main bits I wanted to reference were now split, so felt it would be cleaner if the spreadsheet accessibility section was one big section and at the bottom.

Detailed description of changes

Not needed

Issue ticket number/s and link

No related issue

Checklist before requesting a review

  • I have checked the contributing guidelines
  • I have checked for and linked any relevant issues that this may resolve
  • I have checked that these changes build locally
  • I understand that if merged into main, these changes will be publicly available

@cjrace cjrace marked this pull request as ready for review December 9, 2024 12:56
@cjrace cjrace requested review from rmbielby and jen-machin December 9, 2024 12:56
Copy link
Contributor

@jen-machin jen-machin left a comment

Choose a reason for hiding this comment

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

Sorry - left many comments that are nothing to do with reordering

learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Outdated Show resolved Hide resolved
learning-development/accessibility.qmd Outdated Show resolved Hide resolved
learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Show resolved Hide resolved
learning-development/accessibility.qmd Outdated Show resolved Hide resolved
@cjrace cjrace requested a review from jen-machin December 9, 2024 15:16
@cjrace cjrace force-pushed the shuffle-acessibility-page-order branch from 492a44b to 873a153 Compare December 9, 2024 15:17
@cjrace cjrace merged commit 80a5d9a into main Dec 9, 2024
1 check passed
@cjrace cjrace deleted the shuffle-acessibility-page-order branch December 9, 2024 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants