Skip to content

fix(web): improve totals strip semantics - #326

Open
drnecrotix wants to merge 1 commit into
midt-bg:mainfrom
drnecrotix:fix/totals-strip-semantics
Open

fix(web): improve totals strip semantics#326
drnecrotix wants to merge 1 commit into
midt-bg:mainfrom
drnecrotix:fix/totals-strip-semantics

Conversation

@drnecrotix

@drnecrotix drnecrotix commented Aug 21, 2026

Copy link
Copy Markdown

Какво и защо

Този PR подобрява семантиката на статистическия компонент TotalsStrip.

До момента компонентът използваше <dl>, но отделните стойности и техните описания бяха представени чрез обикновени <span> елементи. Така връзката между етикета и стойността не беше изразена чрез подходящ семантичен HTML.

Промяната използва <dt> за описанието и <dd> за стойността на всяка статистика.

След feedback от предишния review визуалното оформление остава изцяло в CSS - без inline стилове. Съществуващият визуален ред и дизайн се запазват.

Свързан issue

Няма свързан issue - промяната е малко и самостоятелно подобрение на семантичния HTML и достъпността.

Вид промяна

  • fix - поправка на бъг
  • feat - нова функционалност
  • docs - документация
  • refactor / perf / style - без промяна в поведението
  • test / ci / build / chore - поддръжка

Как е тествано

Проверено е, че:

  • всяка статистика използва семантична двойка <dt> / <dd>;
  • стойността остава визуално над описанието;
  • оформлението се управлява чрез съществуващите CSS класове;
  • margin: 0 премахва браузърния default margin на <dd>;
  • промяната не засяга данните или поведението на компонента.

Чеклист

  • Комитите следват conventional commits и нямат Co-Authored-By: trailer, който сочи към агент (Claude Code, Codex, Cursor, Copilot)
  • PR-ът е с един логически обхват и е от форк към midt-bg/sigma:main
  • pnpm typecheck минава
  • pnpm test (поне за засегнатите пакети) минава
  • pnpm lint е чисто
  • Няма комитнати тайни, .env* или .dev.vars
  • Документацията в docs/ е обновена, ако промяната го налага

Discord

dr.necrotix

@drnecrotix

Copy link
Copy Markdown
Author

Възстанових промяната върху отделен branch, за да не може бъдещ Sync fork да изтрие diff-а. Включени са и CSS корекциите от review-а на #315.

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