Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(igxHierarchicalGrid): Add api to navigate to a specific child grid. #15201

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

MayaKirova
Copy link
Contributor

Closes #14912

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

@MayaKirova MayaKirova requested a review from dkamburov January 6, 2025 13:02
@mddragnev mddragnev self-assigned this Jan 6, 2025
@mddragnev mddragnev added 💥 status: in-test PRs currently being tested and removed ❌ status: awaiting-test PRs awaiting manual verification labels Jan 6, 2025
@MayaKirova MayaKirova requested a review from mddragnev January 7, 2025 09:11
@mddragnev mddragnev added ✅ status: verified Applies to PRs that have passed manual verification and removed 💥 status: in-test PRs currently being tested labels Jan 9, 2025
@MayaKirova MayaKirova requested a review from mddragnev January 13, 2025 13:00
mddragnev
mddragnev previously approved these changes Jan 13, 2025
Copy link
Contributor

@viktorkombov viktorkombov left a comment

Choose a reason for hiding this comment

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

In AppBuilder, the implementation works correctly in the design surface, but only when the zoom level is set to 100%. If the zoom level is changed (e.g., below 100%), the scrolling behavior breaks, and navigation to the child grids no longer works as expected.

You can reproduce this issue in the dev demos by adding a transform: scale(0.5, 0.5) style to the .content class in the _app-layout.scss file. This might indicate that the scrolling logic does not properly handle transformations or scaled layouts.

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
grid: hierarchical-grid version: 19.1.x ✅ status: verified Applies to PRs that have passed manual verification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add functionality to navigate to a specific child grid within the hierarchical grid
5 participants