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

change: [M3-9433] - Update styles to match CDS for Entity Header & Action Panel #11857

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

cpathipa
Copy link
Contributor

@cpathipa cpathipa commented Mar 14, 2025

Description 📝

Update styles to match CDS for Entity Header & Action Panel.

Changes 🔄

List any change(s) relevant to the reviewer.

  • Removed minHeight 48 in Breadcrumb.styles.tsx
  • Removed unnecessary div wrapper in LinodesLanding.tsx
  • Update main content and LandingHeader spacing

Target release date 🗓️

Preview 📷

image

How to test 🧪

Verification steps

(How to verify changes)

  • Checkout the branch and verify figma changes for Entity Header and Action Panel
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 unit tests are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@cpathipa cpathipa requested a review from a team as a code owner March 14, 2025 20:02
@cpathipa cpathipa requested review from hkhalil-akamai and cliu-akamai and removed request for a team March 14, 2025 20:03
@cpathipa cpathipa marked this pull request as draft March 14, 2025 20:03
@cpathipa cpathipa self-assigned this Mar 14, 2025
@cpathipa cpathipa added the Design Tokens Laying the groundwork for Design Tokens label Mar 14, 2025
@cpathipa cpathipa changed the title change: [M3-9433] - Update styles to match CDS for Entity Header & Action Panel. change: [M3-9433] - Update styles to match CDS for Entity Header & Action Panel Mar 14, 2025
Copy link

github-actions bot commented Mar 14, 2025

Coverage Report:
Base Coverage: 79.96%
Current Coverage: 79.98%

@cpathipa cpathipa marked this pull request as ready for review March 17, 2025 19:48
Copy link
Contributor

@hkhalil-akamai hkhalil-akamai left a comment

Choose a reason for hiding this comment

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

✅ Entity header height is 34px
✅ Gap between action buttons is 24px

Thanks @cpathipa! 🚀

@cpathipa cpathipa added the Add'tl Approval Needed Waiting on another approval! label Mar 19, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 1 failing test on test run #7 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing534 Passing7 Skipped140m 54s

Details

Failing Tests
SpecTest
linode-config.spec.tsEnd-to-End » Clones a config

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/linode-config.spec.ts"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Add'tl Approval Needed Waiting on another approval! Design Tokens Laying the groundwork for Design Tokens
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants