Skip to content

feat(pivotGrid): Improve PivotGrid visually when there's no data or has empty area at the bottom.#15376

Merged
damyanpetev merged 18 commits intomasterfrom
skrastev/pivot-grid-empy-enh
Mar 11, 2025
Merged

feat(pivotGrid): Improve PivotGrid visually when there's no data or has empty area at the bottom.#15376
damyanpetev merged 18 commits intomasterfrom
skrastev/pivot-grid-empy-enh

Conversation

@skrustev
Copy link
Copy Markdown
Member

@skrustev skrustev commented Feb 18, 2025

Closes #15366

Old vs new pivot grid loading vertical layout:

Old vs new pivot grid with horizontal layout:

Additional information (check all that apply):

  • Bug fix
  • New functionality
  • Documentation
  • Demos
  • CI/CD

Checklist:

  • All relevant tags have been applied to this PR
  • This PR includes unit tests covering all the new code (test guidelines)
  • This PR includes API docs for newly added methods/properties (api docs guidelines)
  • This PR includes feature/README.MD updates for the feature docs
  • This PR includes general feature table updates in the root README.MD
  • This PR includes CHANGELOG.MD updates for newly added functionality
  • This PR contains breaking changes
  • This PR includes ng update migrations for the breaking changes (migrations guidelines)
  • This PR includes behavioral changes and the feature specification has been updated with them

@skrustev skrustev added 🛠️ status: in-development Issues and PRs with active development on them and removed ❌ status: awaiting-test PRs awaiting manual verification labels Feb 18, 2025
@skrustev skrustev added ❌ status: awaiting-test PRs awaiting manual verification and removed 🛠️ status: in-development Issues and PRs with active development on them labels Feb 18, 2025
@skrustev skrustev requested a review from simeonoff February 20, 2025 13:59
@skrustev
Copy link
Copy Markdown
Member Author

skrustev commented Feb 20, 2025

@simeonoff
There are a few changes in the PR that alter the default look when there's less rows than the view (mainly the area bellow the row dimension headers) and a few style changes. Could you could take a look if you are ok with it or if you think someone else should review it as well.

mddragnev
mddragnev previously approved these changes Feb 21, 2025
simeonoff
simeonoff previously approved these changes Feb 24, 2025
Comment on lines 3495 to 3514
.igx-grid__tbody-pivot-dimension {
> .igx-grid-thead__wrapper {
border-bottom: initial;
}

.igx-grid-th {
height: 100%;
}
}

%igx-grid__tbody-pivot-mrl-dimension {
> .igx-grid-thead__wrapper {
border-bottom: initial;
}

.igx-grid-th {
height: 100%;
border-bottom: none;
}
}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Not BEM compliant but will accept for now as the grid styles will be completely rewritten in the coming sprints.

@MarielaTihova MarielaTihova added 👀 status: in-review Issue is currently being reviewed and removed ❌ status: awaiting-test PRs awaiting manual verification labels Feb 26, 2025
@skrustev skrustev dismissed stale reviews from simeonoff and mddragnev via 55017cf February 27, 2025 08:37
@MarielaTihova MarielaTihova added ✅ status: verified Applies to PRs that have passed manual verification and removed 👀 status: in-review Issue is currently being reviewed labels Feb 27, 2025
@dkamburov dkamburov requested review from mddragnev and simeonoff March 7, 2025 08:37
@damyanpetev damyanpetev added the squash-merge Merge PR with "Squash and Merge" option label Mar 11, 2025
@damyanpetev damyanpetev merged commit d1fb678 into master Mar 11, 2025
5 checks passed
@damyanpetev damyanpetev deleted the skrastev/pivot-grid-empy-enh branch March 11, 2025 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

grid: pivot squash-merge Merge PR with "Squash and Merge" option version: 19.2.x ✅ status: verified Applies to PRs that have passed manual verification

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pivot grid loading template is not possitioned correct

6 participants