Skip to content

fix: [UIE-9657] - Alignment with Linode row backup cell icon - #13098

Merged
abailly-akamai merged 2 commits into
linode:developfrom
abailly-akamai:UIE-9657
Nov 20, 2025
Merged

fix: [UIE-9657] - Alignment with Linode row backup cell icon#13098
abailly-akamai merged 2 commits into
linode:developfrom
abailly-akamai:UIE-9657

Conversation

@abailly-akamai

Copy link
Copy Markdown
Contributor

Description 📝

Tiny PR to fix the styling of the Linode row backup cell

Changes 🔄

  • fix icon alignment
  • remove hover underline

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

Preview 📷

Before After
Screenshot 2025-11-17 at 10 00 57 Screenshot 2025-11-17 at 10 13 15
Screenshot 2025-11-17 at 10 13 03 Screenshot 2025-11-17 at 10 12 50

How to test 🧪

Verification steps

  • Navigate to /linodes (must have some linodes with backup enabled)
  • Confirm styling fix
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 Nov 17, 2025
@abailly-akamai abailly-akamai changed the title fix: [UIE-9657] - small alignment issue with Linode row backup cell fix: [UIE-9657] - Alignment with Linode row backup cell icon Nov 17, 2025
@abailly-akamai
abailly-akamai marked this pull request as ready for review November 17, 2025 09:28
@abailly-akamai
abailly-akamai requested a review from a team as a code owner November 17, 2025 09:28
@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 862 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing862 Passing11 Skipped46m 24s

@bnussman-akamai bnussman-akamai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks much nicer

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Nov 19, 2025
@abailly-akamai
abailly-akamai merged commit ad6aaa2 into linode:develop Nov 20, 2025
59 of 60 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Nov 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants