Skip to content

change: [M3-9654] - Akamai Design System: Notice component - #12004

Merged
hana-akamai merged 10 commits into
linode:developfrom
hana-akamai:M3-9654-update-notice-to-design-system
Apr 15, 2025
Merged

change: [M3-9654] - Akamai Design System: Notice component#12004
hana-akamai merged 10 commits into
linode:developfrom
hana-akamai:M3-9654-update-notice-to-design-system

Conversation

@hana-akamai

@hana-akamai hana-akamai commented Apr 9, 2025

Copy link
Copy Markdown
Contributor

Description 📝

Update Notice and Banner components to conform to the new Akamai Design System specifications

Changes 🔄

  • Updated notice/banner colors and icons
  • Updated Storybook Stories

Preview 📷

Before After
before.mov
after.mov

How to test 🧪

Verification steps

(How to verify changes)

  • Check notices and banners across the app
  • Check storybook stories locally
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

@hana-akamai hana-akamai added the Design Tokens Laying the groundwork for Design Tokens label Apr 9, 2025
@hana-akamai hana-akamai self-assigned this Apr 9, 2025
@hana-akamai
hana-akamai marked this pull request as ready for review April 10, 2025 16:30
@hana-akamai
hana-akamai requested a review from a team as a code owner April 10, 2025 16:30
@hana-akamai
hana-akamai requested review from bill-akamai, cpathipa and pmakode-akamai and removed request for a team April 10, 2025 16:30
@cpathipa cpathipa added the Add'tl Approval Needed Waiting on another approval! label Apr 10, 2025
Comment thread packages/ui/src/components/Notice/Notice.styles.ts

@bill-akamai bill-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!
[x] Check notices and banners across the app
[x] Check storybook stories locally

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Apr 14, 2025
@harsh-akamai harsh-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Apr 15, 2025
@davyd-akamai

Copy link
Copy Markdown

Looks good, thank you! I have just one comment for a multi-line content inside a notice. Paddings on top/bottom should be unchanged depending on the number of text lines and always be the same.
image

@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🎉 540 passing tests on test run #8 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing540 Passing4 Skipped106m 42s

@hana-akamai
hana-akamai merged commit c9363ed into linode:develop Apr 15, 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! Design Tokens Laying the groundwork for Design Tokens

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants