Skip to content

fix: [UIE-10482] Parent / Child - SwitchAccountDrawer search focus - #13490

Merged
abailly-akamai merged 2 commits into
linode:developfrom
abailly-akamai:UIE-10482
Mar 13, 2026
Merged

fix: [UIE-10482] Parent / Child - SwitchAccountDrawer search focus#13490
abailly-akamai merged 2 commits into
linode:developfrom
abailly-akamai:UIE-10482

Conversation

@abailly-akamai

Copy link
Copy Markdown
Contributor

Description 📝

This PR fixes two small bugs with the SwitchAccountDrawer (IAM Parent/Child only, legacy flow was not impacted)

Changes 🔄

  • fix issue when user loses input focus when searching through user list
  • fix issue where paginating through users briefly shows the NoResultState

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

How to test 🧪

Prerequisites

  • Have an IAM Parent Child account with many child accounts

Reproduction steps

  • Search users in Switch Account Drawer
    • 🚫 input focus is lost after typing
  • Paginate through users (ex: page 2)
    • 🚫 NoResultState briefly shows before showing page 2

Verification steps

  • Search users in Switch Account Drawer
    • ✅ input focus remains after typing
  • Paginate through users (ex: page 2)
    • ✅ NoResultState does not show during pagination navigation
Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support

  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@abailly-akamai abailly-akamai self-assigned this Mar 13, 2026
@abailly-akamai abailly-akamai added IAM (Identity & Access Management) IAM Delegation Implementing the new Parent/Child delegation UI labels Mar 13, 2026
@abailly-akamai
abailly-akamai marked this pull request as ready for review March 13, 2026 09:10
@abailly-akamai
abailly-akamai requested a review from a team as a code owner March 13, 2026 09:10
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🔺 2 failing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
2 Failing878 Passing11 Skipped40m 47s

Details

Failing Tests
SpecTest
lke-create.spec.tsCloud Manager Cypress Tests→LKE Cluster Creation with LKE-E→shows the LKE-E flow with the feature flag on » creates an LKE-E cluster with the account capability
object-storage.e2e.spec.tsCloud Manager Cypress Tests→object storage end-to-end tests » can create and delete object storage buckets

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/kubernetes/lke-create.spec.ts,cypress/e2e/core/objectStorage/object-storage.e2e.spec.ts"

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Mar 13, 2026
@abailly-akamai
abailly-akamai merged commit 2965cf8 into linode:develop Mar 13, 2026
34 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Mar 13, 2026
grevanak-akamai pushed a commit that referenced this pull request Apr 23, 2026
…13490)

* fix switch account drawer loading state

* Added changeset: Parent / Child - SwitchAccountDrawer search focus
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

IAM Delegation Implementing the new Parent/Child delegation UI IAM (Identity & Access Management)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants