[Bug]: Icon line weight changes with icon size
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 22/100
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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
- Linguagem predominante
- TypeScript
- Estrelas
- 24.8k
- Forks
- 6.4k
- Merge médio
- 59min
- PRs com merge (30d)
- 418
Preparar o ambiente
Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.
- Sem Dockerfile nem arquivo Docker Compose
- Tem um modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de pingdotgg/t3code
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 66/100
Mantenedores costumam responder em até 1 dia
-
via-triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 Meio dia Facilidade para iniciantes 68/100
Mantenedores costumam responder em até 1 dia
-
Question card options show LaTeX as raw source, though the question text renders itTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 76/100
Mantenedores costumam responder em até 1 dia
-
[Bug]: Settings inheritance popover shows raw slugs for branch naming, default permissions and response streamingTalvez já em andamento @juliusmarminge assumiu há 1 dia. Abertavouch:denounced
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 84/100
Mantenedores costumam responder em até 1 dia
Todas as issues de pingdotgg/t3code
Issues semelhantes
-
skill fork: kickoff message "Run the X skill now" makes the child call the skill tool on itselfAbertaarea: skills bug priority: medium
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 68/100
griffinwork40/agent-afk#4874 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
paperclipai/paperclip#21503 ·
Mantenedores costumam responder em até 1 dia
-
[Bug] List: the reorder hint's Move up stays enabled on the first row, and Move down on the lastAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 83/100
-
Blocklist addition requestAbertablocklist addition
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
MetaMask/eth-phishing-detect#312343 ·
Mantenedores costumam responder em até 1 dia
-
packages:player
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia