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

refactor: [M3-9435] [Akamai Design System] Breadcrumb Component #11841

Conversation

bill-akamai
Copy link
Contributor

@bill-akamai bill-akamai commented Mar 13, 2025

Description 📝

This PR updates the Breadcrumb component to conform to the Akamai Design System specifications.

Changes 🔄

  • Implemented Akamai Design System breadcrumb theme tokens
  • Updated typography, spacing, etc to match design specs
  • Used cx utility for cleaner class names in EditableText.tsx

Preview 📷

Before After

Verification steps

  • Open Cloud Manager, navigate to different views
  • Confirm breadcrumb design matches ADS specs
  • Also check if Dark Mode theme looks correct
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

@bill-akamai bill-akamai added the Design Tokens Laying the groundwork for Design Tokens label Mar 13, 2025
@bill-akamai bill-akamai self-assigned this Mar 13, 2025
@bill-akamai bill-akamai marked this pull request as ready for review March 13, 2025 16:45
@bill-akamai bill-akamai requested a review from a team as a code owner March 13, 2025 16:45
@bill-akamai bill-akamai requested review from hkhalil-akamai and cliu-akamai and removed request for a team March 13, 2025 16:45
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.

Apologies for the delay in reviewing this. Breadcrumb appears as expected according to CDS guidelines.

Test failures appear unrelated.

@bill-akamai bill-akamai changed the title refactor: [M3-9431] [Akamai Design System] Breadcrumb Component refactor: [M3-9435] [Akamai Design System] Breadcrumb Component Mar 18, 2025
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

It looks like we lost some spacing between entity headers and the content

Production This PR
Screenshot 2025-03-18 at 2 42 25 PM Screenshot 2025-03-18 at 2 42 00 PM

@bnussman-akamai
Copy link
Member

Looks good but I'm still seeing the spacing issue mention in my comment

@bill-akamai
Copy link
Contributor Author

Looks good but I'm still seeing the spacing issue mention in my comment
Sorry @bnussman-akamai, I overlooked that. Addressed.

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 539 passing tests on test run #10 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing539 Passing3 Skipped103m 12s

Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks for addressing that feedback 🙏

@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Mar 21, 2025
@bill-akamai bill-akamai merged commit 52af395 into linode:develop Mar 21, 2025
25 checks passed
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! Design Tokens Laying the groundwork for Design Tokens
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants