Skip to content

Conversation

@desig9stein
Copy link
Contributor

closes #4

@desig9stein desig9stein added 🪛 enhancement New feature or request ❌ status: awaiting-test PRs awaiting manual verification labels Nov 28, 2025
@coveralls
Copy link

coveralls commented Nov 28, 2025

Pull Request Test Coverage Report for Build 19890096273

Details

  • 16 of 16 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 96.579%

Totals Coverage Status
Change from base Build 19739224499: 0.1%
Covered Lines: 2630
Relevant Lines: 2712

💛 - Coveralls

@SisIvanova SisIvanova added 💥 status: in-test PRs currently being tested and removed ❌ status: awaiting-test PRs awaiting manual verification labels Dec 2, 2025
border-block-end: $row-border-width $row-border-style $row-border-color;
background: $row-odd-background;
color: $row-odd-text-color;
border-block-end: rem(1px) solid var-get($theme, 'row-border-color');

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use --row-border-style and --row-border-width as before, instead of hardcoded values.

Copy link
Contributor Author

@desig9stein desig9stein Dec 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The grid schema does not contain those props

border-bottom: 0;
}

:host(:nth-of-type(odd)) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should use the odd colors here.

@SisIvanova SisIvanova added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Dec 3, 2025
@damyanpetev damyanpetev changed the title [grid-lite] consuming the colors from the grid schemas refactor(themes): consume grid schemas from theming package Dec 3, 2025
@damyanpetev damyanpetev merged commit 47e296b into master Dec 3, 2025
5 checks passed
@damyanpetev damyanpetev deleted the mpopov/refactor-grid-styling branch December 3, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪛 enhancement New feature or request ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GridLite Custom Theme Sample refactoring Move to consuming the colors from the grid schemas in the theming package

7 participants