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

Navigation in the apps charts carousel is not working correctly #12480

Open
IsaSih opened this issue Nov 1, 2024 · 3 comments
Open

Navigation in the apps charts carousel is not working correctly #12480

IsaSih opened this issue Nov 1, 2024 · 3 comments
Labels
area/charts kind/bug QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this status/backport-candidate
Milestone

Comments

@IsaSih
Copy link
Contributor

IsaSih commented Nov 1, 2024

Setup

  • Rancher version: v2.10-212d8b6e92992235d791d8f2aaea8436ab4f6b77-head
  • Browser type & version: Version 130.0.6723.70 (Official Build) (arm64)

Describe the bug
The charts displayed in the carousel animation are misaligned leading to the first one (Vault) not being clickable at all and many blank spaces are shown in the carousel navigation

To Reproduce
1- From the Rancher UI, select a cluster in the side menu bar
2- From the cluster menu options, go to Apps > Charts
3- Observe how the carousel navigation is misaligned
4- Use the arrows to navigate the carousel

Result
User is not able to select the first chart that is displayed in the carousel, whereas there are a few blank spaces when navigating the charts carousel in both ways

Expected Result
Carousel must display an extension at a time and an chart should be displayed and clickable for each frame of the carousel, with no blank spaces

Screenshots

Screen.Recording.2024-11-01.at.15.47.27.mov
@github-actions github-actions bot added the QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this label Nov 1, 2024
@richard-cox richard-cox added this to the v2.11.0 milestone Nov 2, 2024
@richard-cox
Copy link
Member

I'm guessing another item was added (3 entries --> 4). We had this before i think when the number changed.

Definitely need unit / e2e tests with varying numbers of entries

@IsaSih
Copy link
Contributor Author

IsaSih commented Nov 2, 2024

This is also occurring in v2.9-head (v2.9-1d1065cd5bf09c23834720420e1153712fa43439-head)

@nwmac nwmac changed the title Navigation in the extensions carousel is not working correctly Navigation in the apps charts carousel is not working correctly Jan 9, 2025
@IsaSih
Copy link
Contributor Author

IsaSih commented Feb 1, 2025

Same issue still identified in
Rancher | v2.11-a1bda936c809e0394520c9d40d2998529223401e-head
Dashboard | master 478e195
Rancher CLI | v2.11.0-rc.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/charts kind/bug QA/dev-automation Issues that engineers have written automation around so QA doesn't have look at this status/backport-candidate
Projects
None yet
Development

No branches or pull requests

3 participants