Skip to content

Feature contrast slider - #118

Merged
cssma merged 4 commits into
mainfrom
feature-contrast-slider
May 10, 2026
Merged

Feature contrast slider#118
cssma merged 4 commits into
mainfrom
feature-contrast-slider

Conversation

@cssma

@cssma cssma commented Apr 19, 2026

Copy link
Copy Markdown
Collaborator

📄 Pull Request Description


🧩 What was changed?

Added a contrast slider accessible from the top bar. Clicking the contrast icon opens a popover with a slider.


💡 Why was it changed?

Users can adjust the contrast level to suit their visual needs


⚙️ How was it implemented?

The slider (range 1.0–1.2) applies filter: contrast() on the element. The selected value is persisted in localStorage. In light mode, borders are slightly darkened via a CSS class high-contrast to compensate for the filter washing them out. Used two new shadcn components: Slider (radix-ui) and Popover (radix-ui)


⚠️ Side Effects or Risks

none


✅ Checklist

  • All 4 sections above are clearly filled out
  • Tests and documentation updated

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cssma
cssma requested review from kingazm and kon-mtal and removed request for kingazm April 19, 2026 14:03

@kon-mtal kon-mtal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, nit: perhaps we should consider adding an outline to the Sign up button as it is hard to discern from the background with maximum contrast:

Image thanks I will change it

@kingazm

kingazm commented May 3, 2026

Copy link
Copy Markdown
Collaborator

@cssma please resolve conflicts and merge:)

@kingazm

kingazm commented May 9, 2026

Copy link
Copy Markdown
Collaborator

@cssma please resolve conflicts and merge

@cssma
cssma merged commit cd2711c into main May 10, 2026
1 check passed
@cssma
cssma deleted the feature-contrast-slider branch May 10, 2026 10:50
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.

3 participants