Skip to content

Update style.css - #35

Open
SDK5209 wants to merge 1 commit into
tkmfoss:mainfrom
SDK5209:main
Open

SDK5209 wants to merge 1 commit into
tkmfoss:mainfrom
SDK5209:main

Conversation

@SDK5209

@SDK5209 SDK5209 commented Oct 24, 2026

Copy link
Copy Markdown

Summary

Increases the touch target sizes for the theme toggle, category filter buttons, and primary navigation links on mobile devices to meet accessibility guidelines without regressing desktop layout.

Related issue

Closes #21

Changes made

  • Updated .theme-toggle dimensions to 44px x 44px in css/style.css to hit the recommended touch target minimum.
  • Set min-height: 44px on .filter-button and .site-nav__link to ensure comfortable tap areas on mobile.
  • Added display: inline-flex and align-items: center to .site-nav__link to ensure text remains centered vertically with the new height.

Testing performed

  • I ran the site locally and tested the change.
  • I checked the browser console for errors.
  • I tested on desktop and mobile widths.
  • Other (please describe):

Screenshots

Summary

Increases the touch target sizes for the theme toggle, category filter buttons, and primary navigation links on mobile devices to meet accessibility guidelines without regressing desktop layout.

Related issue

Closes #21

Changes made

  • Updated .theme-toggle dimensions to 44px x 44px in css/style.css to hit the recommended touch target minimum.
  • Set min-height: 44px on .filter-button and .site-nav__link to ensure comfortable tap areas on mobile.
  • Added display: inline-flex and align-items: center to .site-nav__link to ensure text remains centered vertically with the new height.

Testing performed

  • I ran the site locally and tested the change.
  • I checked the browser console for errors.
  • I tested on desktop and mobile widths.
  • Other (please describe):

Screenshots

Summary

Increases the touch target sizes for the theme toggle, category filter buttons, and primary navigation links on mobile devices to meet accessibility guidelines without regressing desktop layout.

Related issue

Closes #21

Changes made

  • Updated .theme-toggle dimensions to 44px x 44px in css/style.css to hit the recommended touch target minimum.
  • Set min-height: 44px on .filter-button and .site-nav__link to ensure comfortable tap areas on mobile.
  • Added display: inline-flex and align-items: center to .site-nav__link to ensure text remains centered vertically with the new height.

Testing performed

  • I ran the site locally and tested the change.
  • I checked the browser console for errors.
  • I tested on desktop and mobile widths.
  • Other (please describe):

Screenshots

Screenshot_20261024_175455

Contributor checklist

  • My change follows the existing code style.
  • I have tested my change locally.
  • I have not committed unrelated files.
  • I have commented my code where non-obvious logic lives.

Contributor checklist

  • My change follows the existing code style.
  • I have tested my change locally.
  • I have not committed unrelated files.
  • I have commented my code where non-obvious logic lives.

Contributor checklist

  • My change follows the existing code style.
  • I have tested my change locally.
  • I have not committed unrelated files.
  • I have commented my code where non-obvious logic lives.

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.

ensure-touch-friendly-target-sizes-on-mobile

1 participant