scientific-python / scientific-python-hugo-theme

Hugo theme based on the design for numpy.org
https://theme.scientific-python.org/
Other
17 stars 21 forks source link

Accessibility meta issue #477

Open jarrodmillman opened 4 months ago

jarrodmillman commented 4 months ago

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 https://github.com/pydata/pydata-sphinx-theme/pull/1564

@trallard Here is a partial list of SPHT (scientific python hugo theme) specific work:

smeragoel commented 3 months ago

I have a question regarding the custom shortcodes, specifically the yamltotables component.

  1. I don't see the actual component in the link provided (same for youtube)
  2. I'll get more context when I see the component, but there's an existing PST tables component and we are doing active accessibility work on it, so we can leverage this component if we aren't already doing so 😸
jarrodmillman commented 3 months ago

@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!

smeragoel commented 3 months ago

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?

smeragoel commented 1 month ago

@jarrodmillman bumping this up!

trallard commented 3 weeks ago

Adding this https://github.com/scientific-python/scientific-python-hugo-theme/issues/618 as this seems to be an issue with theming (or lack of theming for dark mode) cc @smeragoel