Skip to content

Remove misleading "JavaScript is required" notice from sort dropdown#1400

Merged
stefan-burke merged 1 commit intomainfrom
claude/add-js-requirement-notice-faLfG
May 5, 2026
Merged

Remove misleading "JavaScript is required" notice from sort dropdown#1400
stefan-burke merged 1 commit intomainfrom
claude/add-js-requirement-notice-faLfG

Conversation

@stefan-burke
Copy link
Copy Markdown
Member

Summary

  • The <noscript> fallback for the sort dropdown already renders a fully working list of sort links, so claiming "JavaScript is required for the sort dropdown" is inaccurate — users without JS can still sort by clicking the links.
  • Removed the misleading <p> notice from src/_includes/filter-sort-dropdown.html; the link-list fallback remains.

Test plan

  • Disable JavaScript in the browser, visit a category page (e.g. /categories/doodahs), and confirm the sort link list renders without the misleading paragraph.
  • With JavaScript enabled, confirm the <select> dropdown still works and the noscript content stays hidden.

https://claude.ai/code/session_01L87rRST6ndvyUzJpMr8fLe


Generated by Claude Code

The noscript fallback already renders a working list of sort links, so
the claim that JavaScript is required is incorrect — sorting works fine
without JS via the link list.

https://claude.ai/code/session_01L87rRST6ndvyUzJpMr8fLe
@stefan-burke stefan-burke added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit b7924c9 May 5, 2026
3 checks passed
@stefan-burke stefan-burke deleted the claude/add-js-requirement-notice-faLfG branch May 5, 2026 02:17
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