Skip to content

chore(transaction): убрать мёртвое правило :empty в строках списка операций - #200

Open
GlebYavorski wants to merge 1 commit into
ardov:masterfrom
GlebYavorski:chore/remove-dead-empty-margin-rule
Open

chore(transaction): убрать мёртвое правило :empty в строках списка операций#200
GlebYavorski wants to merge 1 commit into
ardov:masterfrom
GlebYavorski:chore/remove-dead-empty-margin-rule

Conversation

@GlebYavorski

Copy link
Copy Markdown

Небольшая уборка стилей в строке операции. Ref #142.

В Row осталось правило-хвост:

> *:empty + * {
  margin-left: 0;
}

Оно было парой к > *:first-child:empty { flex-grow: 0 } — тому самому правилу, из-за которого название счёта «съезжало» влево у операций без плательщика и комментария (#142). Второе правило удалено в 335a6b7, а первое осталось и теперь ни на что не влияет: пустой первый блок снова растягивается через flex-grow: 1, поэтому margin-left соседа не виден в любом случае.

Проверка

Собрал изолированный репро с тремя вариантами CSS и замерил правую границу блока со счётом (ширина ряда 448px):

right edges — old: withInfo=448 empty=100 | master: withInfo=448 empty=448 | PR: withInfo=448 empty=448

tsc --noEmit проходит.

Сам баг из #142 уже исправлен в master, так что ишью можно закрывать независимо от этого PR — здесь только чистка мёртвого CSS.

🤖 Generated with Claude Code

The `> *:empty + * { margin-left: 0 }` rule was the leftover pair of
`> *:first-child:empty { flex-grow: 0 }`, removed in 335a6b7 (the actual
fix for ardov#142). Now that the empty first child grows again, the sibling's
margin-left never affects layout, so the override does nothing.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.

1 participant