Skip to content

Přidání CSS třídy sloupce na obsahové buňky tree-view#1291

Open
radimvaculik wants to merge 1 commit into
masterfrom
fix-treeview-column-classes
Open

Přidání CSS třídy sloupce na obsahové buňky tree-view#1291
radimvaculik wants to merge 1 commit into
masterfrom
fix-treeview-column-classes

Conversation

@radimvaculik

Copy link
Copy Markdown
Member

Popis

V tree-view (datagrid_tree.latte) se na obsahové buňky pravého sloupcového bloku nepřidávala třída col-<sloupec>, kterou má jejich hlavička. Kvůli tomu šlo cílit CSS na konkrétní sloupec jen přes :first-child, :last-child, :nth-child apod.

Příčina

Nekonzistence mezi hlavičkou a obsahem v šabloně:

  • Hlavička (řádek 33): datagrid-tree-item-right-columns-column col-{\$column->getColumnName()} text-... → např. col-status
  • Obsah (řádek 99): datagrid-tree-item-right-columns-column text-... → třída col-status chyběla

Oprava

Doplněno col-{\$column->getColumnName()} na obsahovou buňku, takže používá stejný výraz třídy jako hlavička.

Closes #1290

V tree-view se na obsahové buňky nepřidávala třída col-<sloupec>, kterou má hlavička. Doplněno col-{$column->getColumnName()}, aby hlavička i obsah byly konzistentní.
@codecov

codecov Bot commented Jun 29, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.77%. Comparing base (d9144ab) to head (5f38c2a).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1291   +/-   ##
=======================================
  Coverage   48.77%   48.77%           
=======================================
  Files          61       61           
  Lines        2901     2901           
=======================================
  Hits         1415     1415           
  Misses       1486     1486           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Bug? Datagrid TreeView not adding classes on content rows

1 participant