Skip to content

Normalize desktop icon sizing#1088

Merged
klopez4212 merged 1 commit into
mainfrom
kennylopez-normalize-desktop-icon-sizing
Jun 17, 2026
Merged

Normalize desktop icon sizing#1088
klopez4212 merged 1 commit into
mainfrom
kennylopez-normalize-desktop-icon-sizing

Conversation

@klopez4212

Copy link
Copy Markdown
Contributor

Summary

  • normalize standard desktop icon glyphs to 16px across shared buttons and UI surfaces
  • update remaining channel member, huddle, settings, profile, message, and reminder icon callsites that were still 14px/18px/20px
  • leave non-standard avatars, status dots, skeletons, media illustrations, and decorative state artwork unchanged

Validation

  • git diff --check
  • cd desktop && ./node_modules/.bin/tsc --noEmit
  • cd desktop && ../node_modules/.bin/biome check <changed files>
  • cd desktop && node scripts/check-px-text.mjs
  • cd desktop && node scripts/check-file-sizes.mjs
  • targeted scan for lingering 18px / size-4.5 icon sizing
  • cd desktop && ./node_modules/.bin/vite build

Screenshot

Posted separately as a PR comment.

@klopez4212

klopez4212 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Refreshed channel-page screenshot for #1088 (Normalize desktop icon sizing).

The previous image was captured from stale desktop/dist output that still contained [&_svg]:size-4.5. After a clean rebuild, the channel header Users, Headphones, and settings icons measure 16px in Chromium.

Code checks/builds run:

  • git diff --check
  • cd desktop && ./node_modules/.bin/tsc --noEmit
  • cd desktop && ../node_modules/.bin/biome check <changed files>
  • cd desktop && node scripts/check-px-text.mjs
  • cd desktop && node scripts/check-file-sizes.mjs
  • targeted scan for lingering 18px / size-4.5 icon sizing
  • cd desktop && ./node_modules/.bin/vite build
  • clean rebuild after removing stale desktop/dist

Channel page icon sizing

icon-sizing-channel-page-verified

klopez4212 added a commit that referenced this pull request Jun 17, 2026
@klopez4212 klopez4212 requested a review from a team June 17, 2026 16:42
@klopez4212 klopez4212 marked this pull request as ready for review June 17, 2026 16:42
klopez4212 added a commit that referenced this pull request Jun 17, 2026
@klopez4212 klopez4212 merged commit 1a37ee3 into main Jun 17, 2026
23 checks passed
@klopez4212 klopez4212 deleted the kennylopez-normalize-desktop-icon-sizing branch June 17, 2026 17:08
klopez4212 added a commit that referenced this pull request Jun 17, 2026
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