Skip to content

Conversation

@hasyed-akamai
Copy link
Contributor

@hasyed-akamai hasyed-akamai commented Jul 25, 2025

📝 Description

Fixes hover styling for disabled zebra-striped table rows.

🔄 Changes

  • Updated hover style for disabled rows.
  • Ensured consistency across Light and Dark themes.

📷 Preview

Theme Before After
Dark Before - Dark After - Dark
Light Before - Light After - Light

How to test

  1. Go to /linodes/create.
  2. Select a region and choose a GPU plan.
  3. Verify hover styling on disabled zebra-striped rows.
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

@hasyed-akamai hasyed-akamai self-assigned this Jul 25, 2025
@hasyed-akamai hasyed-akamai added the Bug Fixes for regressions or bugs label Jul 25, 2025
@hasyed-akamai hasyed-akamai marked this pull request as ready for review July 25, 2025 11:48
@hasyed-akamai hasyed-akamai requested a review from a team as a code owner July 25, 2025 11:48
@hasyed-akamai hasyed-akamai requested review from mjac0bs and pmakode-akamai and removed request for a team July 25, 2025 11:48
Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this! 🚢

Screen.Recording.2025-07-25.at.8.11.36.AM.mov

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jul 25, 2025
@mjac0bs mjac0bs added Approved Multiple approvals and ready to merge! and removed Ready for Review labels Jul 25, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 690 passing tests on test run #4 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing690 Passing4 Skipped124m 54s

@hasyed-akamai hasyed-akamai merged commit 2818947 into linode:develop Jul 28, 2025
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! Bug Fixes for regressions or bugs

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants