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

(WIP) Release/accessibility phase1 #434

Draft
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

st3phhays
Copy link
Member

Description Of Changes

TBD.

Motivation and Context

TBD.

Testing

TBD.

Operating Systems Testing

TBD.

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v3 compatibility checked?

Related Issue

TBD.

@st3phhays st3phhays marked this pull request as draft January 7, 2025 13:50
This includes some general accessibility
enhancements like increased spacing, link styles,
and other minor changes to improve the overall
accessibility of choco-theme.
This updates a lot of the colors in choco-theme to
ensure that they are accessible. This includes
changes to the structure of the scss as well, to
separate it out into folders for better organization.
This includes some general accessibility
enhancements like increased spacing, link styles,
and other minor changes to improve the overall
accessibility of choco-theme.
This updates a lot of the colors in choco-theme to
ensure that they are accessible. This includes
changes to the structure of the scss as well, to
separate it out into folders for better organization.
This allows multiple Toast notifications to stack
on top of each other. They are only removed from
the dom after the `hide` callback is triggered.
This gives the user the ability to see all of the
notifications that have been triggered.
This removes the jQuery Duallistbox library in
favor of the Select extension from datatables.js.
This updates a lot of the colors in choco-theme to
ensure that they are accessible. This also enhances some components to
take advantage of color, or add indicators other than color.
This updates the .stylelintignore file and addresses any stylelint
errors. This should not change the outcome of CSS, but enforces code
style.
This adds packages for Playwright that will enable accessibility tests
using Axe to be ran. These tests will automatically check for WCAG
several WCAG violations.
This ensures the .btn-sm class is not effected with a minimum height
like normal buttons.
This adjust the selectors used for the left side navigation to remove
the main element before. This will make it compatible with the new HTML
structure.
@st3phhays st3phhays force-pushed the release/accessibility-phase1 branch from bc8089c to 6ff76ec Compare January 15, 2025 11:19
While this doesn't remove the jQuery Duallistbox library yet, this is
the start of being able to do so. The changes here in combination with
others in the repository allow for similar functionality, yet in an
accessible way. This introduces the fuse.js library to help with search.
@st3phhays st3phhays force-pushed the release/accessibility-phase1 branch from 672e4d0 to 9da4468 Compare January 16, 2025 18:31
alexaveldanez and others added 6 commits January 24, 2025 14:40
Increase checkbox size to 25px by 25px.
…s-and-radios

(#436) Increase size of checkboxes to 25px by 25px
This removes the jQuery Duallistbox library and all the things related
to it. This library is no longer being used and needs removed.
With the increased size of checkboxes, they were somewhat being smooshed
together if they were stacked. Increasing the margin bottom on them
helps separate them some and look better in general.
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