-
Notifications
You must be signed in to change notification settings - Fork 23
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
Accessibility meta issue #477
Comments
I have a question regarding the custom shortcodes, specifically the
|
@smeragoel Sorry about that. I removed the yamlToTable documentation (#570) because we are switching from yaml to toml for shortcodes (#484). There is a PR in progress, but I haven't finished it (#572). I updated the link above to point to the PR and removed the deadlink. I am planning to use the PST styling in the new PR, but haven't yet made much progress other than adding to class tags used by PST. I will try to get the tomlToTable PR ready to merge. Thanks for looking into this! |
Thanks for the update, @jarrodmillman! I'm happy to take a look at the table design once it's ready, so feel free to ping me. I came across the details component at https://theme.scientific-python.org/shortcodes/#details and noticed its functionality seems quite similar to the toggle buttons found in the PyData Sphinx Theme (https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/web-components.html#toggle-buttons), which also incorporate accessibility features. Was there a specific requirement or purpose that led to the creation of this new component, especially considering its simpler design? Also, this may not be relevant to this ticket, but I saw that there is a dropdown component (https://theme.scientific-python.org/shortcodes/#dropdown), but it lacks hover interaction and a chevron indicator for expandability - features that are present in the PyData Sphinx Theme's dropdown: https://pydata-sphinx-theme.readthedocs.io/en/stable/user_guide/web-components.html#dropdowns. Is this by design, or are there plans to enhance this component for consistency and usability? |
@jarrodmillman bumping this up! |
Adding this #618 as this seems to be an issue with theming (or lack of theming for dark mode) cc @smeragoel |
CAUTION: This is a rough placeholder for upcoming accessibility improvements and needs to be fleshed out in more detail.
We leverage (and plan to leverage more of) the accessibility work being done for the PST (pydata sphinx theme). For example, see pydata/pydata-sphinx-theme#1564
@trallard Here is a partial list of SPHT (scientific python hugo theme) specific work:
<button>
and not<a>
tagsThe text was updated successfully, but these errors were encountered: