[Bug]: Icon line weight changes with icon size
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 22/100
Piste de recherche
The problem is a global stroke-width rule for lucide icons under apps/web, plus per-icon overrides (1.75, 1.9, 2, 2.25) in the timeline and toast surfaces. Linked PR 36078 is already open, so review that work before starting a parallel change. Done means icons of 12, 14 and 16 px draw the same line weight in light and dark themes, which needs visual checking by someone other than the author.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Before submitting
- I searched existing issues and did not find a duplicate.
- I included enough detail to reproduce or investigate the problem.
Area
apps/web
Steps to reproduce
- Open the web or desktop app on main.
- Find icons of different sizes that sit together, like the sidebar header, a timeline row or a toast.
- Compare the line of a 12 px icon with a 16 px icon.
Expected behavior
Icons that sit together draw the same line weight, whatever their size.
Actual behavior
One global rule sets stroke-width: 1.75 on lucide's 24-unit grid, so the line scales with the icon:
| Icon size | Line drawn on main |
|---|---|
| 12 px | 0.88 px |
| 14 px | 1.02 px |
| 16 px | 1.17 px |
Some icons also carry one-off overrides (1.75, 1.9, 2 and 2.25). That makes timeline and toast icons heavier than their neighbors. Hand-built SVGs next to lucide icons use their own weights too.
Top is main, bottom is the fix I have ready:
I'll link the PR here. Three things to know:
- I tested and verified it by eye on most surfaces, and I made the design calls along the way (the weight, how large icons scale, which overrides go).
- One person's eyes aren't enough for a visual change this wide. Someone else needs to see it for themselves.
- It's one part of a larger set. Some places are unclear today because of color and icon choice. I'm leaving those out for scope, and because unvouched contributors can only have 3 open PRs.
Impact
Cosmetic issue
Version or commit
main @ 290182dc83
Environment
macOS 27.2. Chromium 156, Firefox 157 and WebKit 27.2.
Logs or stack traces
Screenshots, recordings, or supporting files
pair-toast-theme-light-2x.png
pair-app-light-1x.png
pair-app-light-2x.png
pair-app-dark-2x.png
Workaround
No response
- Langage dominant
- TypeScript
- Étoiles
- 24.8k
- Forks
- 6.4k
- Merge moyen
- 59 min
- PR mergées (30 j)
- 418
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de pingdotgg/t3code
-
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
Les mainteneurs répondent en général sous 1 jour
-
via-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: MCP App inline frames never go below 80px, so compact apps get blank space under themOuverte
Difficulté 2/5 Une demi-journée Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
Question card options show LaTeX as raw source, though the question text renders itPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Settings inheritance popover shows raw slugs for branch naming, default permissions and response streamingPeut-être pris @juliusmarminge l’a pris il y a 1 jour. Ouvertevouch:denounced
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de pingdotgg/t3code
Issues similaires
-
skill fork: kickoff message "Run the X skill now" makes the child call the skill tool on itselfOuvertearea: skills bug priority: medium
Difficulté 1/5 Moins d'une heure Accessibilité débutants 68/100
griffinwork40/agent-afk#4874 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
paperclipai/paperclip#21503 ·
Les mainteneurs répondent en général sous 1 jour
-
[Bug] List: the reorder hint's Move up stays enabled on the first row, and Move down on the lastOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
-
Blocklist addition requestOuverteblocklist addition
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
MetaMask/eth-phishing-detect#312343 ·
Les mainteneurs répondent en général sous 1 jour
-
packages:player
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour