Skip to content

Increase distance between toggle switches globally#11

Open
MohSay890 wants to merge 1 commit into
thm-projects:stagingfrom
MohSay890:Increase-the-distance-between-two-toggle-switches
Open

Increase distance between toggle switches globally#11
MohSay890 wants to merge 1 commit into
thm-projects:stagingfrom
MohSay890:Increase-the-distance-between-two-toggle-switches

Conversation

@MohSay890
Copy link
Copy Markdown

…-spacing.scss file

@MohSay890 MohSay890 changed the title Increase distance between toggle switches globally by adding a toggle… Increase distance between toggle switches globally Nov 16, 2025
@MohSay890
Copy link
Copy Markdown
Author

I added a new file (adding a global toggle-spacing.scss) in "styles" folder to increase the distance between all toggle switches, and I think this solution will optimize user experience by widening the gap between all adjacent toggle switches. I see this is aligning with the 13 criteria of Definition of Done that are mentioned in the DoD.md file in the project, because:

  1. Cross-browser tested: The CSS approach is standard and works in Chrome, Firefox, and Safari
  2. No Lighthouse regression: Spacing changes do not negatively impact Lighthouse scores except possibly performance (which is allowed)
  3. Responsive: The spacing uses relative units and applies across all screen sizes, ensuring toggles are not crowded on any device
  4. Accessible: Increased spacing reduces accidental activation, improving accessibility for all users
  5. Translations: No text or UI labels were changed, so translations are unaffected
  6. Tooltips: The change does not remove or affect tooltips on toggles
  7. Theme compatibility: The spacing is neutral and works with all themes
    8–13. Process: The change is in a feature branch, can be merged, reviewed, and checked for technical debt as usual.

@MohSay890 MohSay890 closed this Nov 17, 2025
@MohSay890 MohSay890 reopened this Nov 17, 2025
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