fix(sidebar): improve accessibility of sidebar bottom menu #925
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
(redacted) |
FarrelF
added a commit
to FarrelF/stack-modified
that referenced
this pull request
Feb 23, 2024
|
Note that #950 is targeting the same section of code, so once either one is merged, the other one needs to be rebased. |
ottok
pushed a commit
to ottok/hugo-theme-stack
that referenced
this pull request
Feb 24, 2024
…ements Cherry-picked from CaiJimmy#925 with: git fetch https://github.com/TheJiahao/hugo-theme-stack.git 4bb4325 git cherry-pick FETCH_HEAD
CaiJimmy
changed the title
|
@TheJiahao Could you confirm this change works fine in mobile size? I tried to cherry-pick this change to v4 branch, it seems that the language switcher disappeared. |
NapalmCodes
pushed a commit
to NapalmCodes/hugo-theme-stack
that referenced
this pull request
Mar 30, 2024
* fix(sidebar): hide translations selector if only one translation available (CaiJimmy#950) hide translations menu if only one translation available * release: 3.22.0 * fix(sidebar): translation selector - avoid cast per PR CaiJimmy#950 (CaiJimmy#961) avoid cast per PR CaiJimmy#950 * feat(comments): add support for Beaudar（表达）comment system (CaiJimmy#931) https://beaudar.lipk.org/ * fix(comment): moved waline `placeholder` parameter to `locale.placeholder` (CaiJimmy#914) fix waline parameter * fix(sidebar): improve accessibility of sidebar bottom menu (CaiJimmy#925) * fix(menu): <ol> containing other than <li>, <script> or <template> elements * fix(menu): add title to language select menu * release: 3.23.0 * fix(i18n): add missing `zh-cn` translations (CaiJimmy#964) * feat: update Hugo version and minimum version requirement to 0.123.0 (CaiJimmy#970) * fix: `.Site.LastChange` deprecation (CaiJimmy#968) Fix the deprecation warning emitted from Hugo, and fix the typo in ` exampleSite/content/page/about/index.md` * feat: bump KaTeX to latest released version `0.16.9` (CaiJimmy#969) Bump KaTeX to latest released version 0.16.9 * fix(sidebar): menu-bottom-section not showing in mobile (CaiJimmy#966) * fix(sidebar): Bottom section not showing in mobile * fix(sidebar): Align bottom section to bottom * fix(sidebar): Gap missing in bottom section * refactor(sidebar): Simplify styles * refactor(sidebar): Remove useless padding-left * refactor(sidebar): Remove useless margin-top * refactor(sidebar): Combine duplicate flex-direction * refactor(sidebar): Remove redundant width * release: 3.24.0 * fix(head): prevent Go template from escaping HTML entities in meta description (CaiJimmy#973) * fix: favicon does not load if baseurl includes a subfolder (CaiJimmy#972) Use relURL https://gohugo.io/functions/urls/relurl/ * release: 3.24.1 * fix: remove `\n` character from meta description (CaiJimmy#980) * fix: deprecation warning for `.Site.Author` (CaiJimmy#979) Fix deprecation warning for .Site.Author, deprecated in Hugo 0.124.0 * release 3.24.2 * fix: use page resource permalink in link image (CaiJimmy#990) closes CaiJimmy#982 * docs: update some invalid documentation links (CaiJimmy#983) Update some documentation links (some were dead links) * feat(i18n): update translations for `ja` (CaiJimmy#984) * feat(i18n): update translations for `ja` * Apply review suggestions * fix(i18n): wrap `footer.builtWith` with double quotes in Japanese translation (CaiJimmy#991) * feat: add anchor link to markdown heading (CaiJimmy#992) closes CaiJimmy#935 * fix: rename `.social-menu` to `.menu-social` to avoid being blocked by ad blockers (CaiJimmy#993) closes CaiJimmy#924 * fix: make KaTeX render all math inside `document.body` (CaiJimmy#994) Since it only accepts one element, I cannot pass `.article-content` and `#TableOfContent` to it. The official documentation uses `document.body` directly, so I guess that's fine. closes CaiJimmy#882 * release: 3.25.0 * Fix to dark mode toggle * Style fixes * Style fixes * Style fix * Tweak * Fix * Remove hash sign * Make sub heading text the anchor --------- Co-authored-by: James McMurry <jmcmurry@semanticnoise.com> Co-authored-by: Jimmy Cai <hi@jimmycai.com> Co-authored-by: Jiarong Hong <64763790+hongjr03@users.noreply.github.com> Co-authored-by: L-Super <40905056+L-Super@users.noreply.github.com> Co-authored-by: Jiahao Li <42709836+TheJiahao@users.noreply.github.com> Co-authored-by: Zeyu Huang <fhfuih@outlook.com> Co-authored-by: Andreas Deininger <andreas@deininger.net> Co-authored-by: Andrew Doering <delize@users.noreply.github.com> Co-authored-by: andrewmoise <andrew.moise@gmail.com> Co-authored-by: Shun Sakai <sorairolake@protonmail.ch>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move
<div class="menu-bottom-section">outside<ol class="menu" id="main-menu">to fix warning<li> elements must be contained in a <ul> or <ol>Add
title="language"to language select menu to fix errorSelect element must have an accessible name: Element has no title attributeThe appearance is identical after change.
Before:
After:
