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-8801] - Remove Individual Product Entity Icons #11537

Conversation

hasyed-akamai
Copy link
Contributor

@hasyed-akamai hasyed-akamai commented Jan 21, 2025

Description 📝

We’ve removed unused individual product icons and renamed the remaining SVG files to align with product families. Additionally, the EntityIcon story has been updated to reflect the changes and only use the relevant product family icons.

Changes 🔄

List any change relevant to the reviewer:

  • Removed unused individual product icons from the codebase.
  • Renamed the remaining individual product SVG files to reflect their corresponding product families.
  • Updated the EntityIcon story to use product family icons.

Preview 📷

Before After
Before After

How to test 🧪

Verification steps

  1. Verify that the individual product icons that are no longer needed have been removed.
  2. Check that the renamed SVG files correctly represent product families.
  3. Inspect the EntityIcon story:
    • Ensure it now uses only product family icons.
  4. Run the application and confirm that the product icons display correctly across the UI.

As an Author I have considered 🤔

Check all that apply

  • 👀 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

@hasyed-akamai hasyed-akamai marked this pull request as ready for review January 22, 2025 09:57
@hasyed-akamai hasyed-akamai requested a review from a team as a code owner January 22, 2025 09:57
@hasyed-akamai hasyed-akamai requested review from bnussman-akamai and carrillo-erik and removed request for a team January 22, 2025 09:57
@hasyed-akamai hasyed-akamai added Ready for Review Design Tokens Laying the groundwork for Design Tokens and removed Work in Progress Do Not Merge labels Jan 22, 2025
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 12 failing tests on test run #6 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
12 Failing462 Passing2 Skipped155m 37s

Details

Failing Tests
SpecTest
betas.spec.tsBetas landing page » can navigate to Betas landing page
search-images.spec.tsSearch Images » creates two images and make sure they show up in the table and are searchable
smoke-linode-landing-table.spec.tslinode landing checks » checks the landing page side menu items
smoke-linode-landing-table.spec.tslinode landing checks » checks the landing top menu items
lke-update.spec.tswith LKE ACL account capability » can enable ACL on an LKE cluster with ACL pre-installed and edit IPs
lke-update.spec.tswith LKE ACL account capability » can disable ACL and edit IPs
placement-groups-navigation.spec.tsPlacement Groups navigation » can navigate to Placement Groups landing page
search-linodes.spec.tsSearch Linodes » create a linode and make sure it shows up in the table and is searchable in main search tool
cloudpulse-navigation.spec.tsCloudPulse navigation » can navigate to Cloudpulse landing page
search-volumes.spec.tsSearch Volumes » creates two volumes and make sure they show up in the table and are searchable
vpc-navigation.spec.tsVPC navigation » can navigate to VPC landing page
clone-linode.spec.tsclone linode » shows DC-specific pricing information during clone flow

Troubleshooting

Use this command to re-run the failing tests:

yarn cy:run -s "cypress/e2e/core/account/betas.spec.ts,cypress/e2e/core/images/search-images.spec.ts,cypress/e2e/core/linodes/smoke-linode-landing-table.spec.ts,cypress/e2e/core/kubernetes/lke-update.spec.ts,cypress/e2e/core/placementGroups/placement-groups-navigation.spec.ts,cypress/e2e/core/linodes/search-linodes.spec.ts,cypress/e2e/core/cloudpulse/cloudpulse-navigation.spec.ts,cypress/e2e/core/volumes/search-volumes.spec.ts,cypress/e2e/core/vpc/vpc-navigation.spec.ts,cypress/e2e/core/linodes/clone-linode.spec.ts"

Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

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

thank you!

✅ confirmed icons on storybook
✅ confirmed icons throughout CM

@coliu-akamai coliu-akamai added Approved Multiple approvals and ready to merge! and removed Ready for Review labels Jan 23, 2025
@hasyed-akamai hasyed-akamai merged commit 66eb997 into linode:cds-redesign-2025 Jan 24, 2025
22 of 23 checks passed
abailly-akamai added a commit that referenced this pull request Feb 6, 2025
* upcoming: [M3-9001] - Footer Update (#11446)

* upcoming: [M3-9002] - Footer Update

* global tokens formatting

* fix

* left align for desktop - center for mobile

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>

* Adjust dev tools icon to the right

* UX Request: Reduce spacing between footer items

* upcoming: [M3-8982] - Update Left Navigation  (#11463)

* upcoming: [M3-9002] - Left Navigation Update

* removed akamai color logo

* updated pin icon tooltip text

* upcoming: [M3-8982] - Update Left Navigation

* updated theme file's format

* updated standby values

* corrected badge width and fixed colors for product family

* Revert "corrected badge width and fixed colors for product family"

This reverts commit 6fcb80b.

* Fix font styles for side nav, adjust spacing, revert active states

* Update product family icons

* integrated footer updates and updated icon's svg colors to be dynamic

* Fix borders for mobile

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>

* upcoming: [M3-9006] - Update Table Component with Design Tokens (#11461)

* upcoming: [9006] - Update Table Component with Design Tokens

* Change unit tests for `CollapsibleRow` component

* Complete table overhaul

* Second pass through tables

* Update linode groups

* Cleanup

* Add hover background transitions to tables - `Part1`

* Add zebra striping and selected states

* Add selected state for plan table

* Fix dark mode styles

* Add hover background transitions to tables - `Part2`

* Add hover background transitions to tables - `Part3`

* Revert "Add hover background transitions to tables - `Part3`"

This reverts commit bffeef5.

* Revert "Add hover background transitions to tables - `Part2`"

This reverts commit 0ecfc55.

* Revert "Add hover background transitions to tables - `Part1`"

This reverts commit 8f87cb6.

* Make hover the default for TableRow

* Add table striping as a setting preference

* Add border to table attachment headers

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Co-authored-by: Jaalah Ramos <125309814+jaalah-akamai@users.noreply.github.com>

* upcoming: [M3-9002] - Replace hamburger icon in left navigation (#11462)

* upcoming: [M3-9002] - Left Navigation Update

* removed akamai color logo

* updated pin icon tooltip text

* Fix table focus state and global header background

* Update logo color

* Revert logo change

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>

* [CDS Updates] - Handle cds sticky sidebar (#11479)

* Handle dynamic sidebar

* SideMenu temp fonts

* Toggler

* Toggler

* better handling

* better util

* cleanup

* cleanup

* icon fix

* remove chips and improve preference managing (#11492)

* Simplify footer/toggle

* change: [M3-9003] – CDS: Restructure Top Menu Design with Akamai Logo and CDS Compass Updates (#11452)

* Initial changes: top menu spans full width of app, logo moved to top menu, rearrangement of top menu elements, reordering of components in MainConent.tsx & Root.tsx, addition of new icons for top menu

* Update icon svg's; match mockups more closely and fix colors; implement hamburger icon in tablet/mobile view when side menu disappears; implement company name truncation

* Adjust import path

* Update MuiButton padding, remove minWidth on buttons; use GlobalHeader tokens; fix icon and text colors for Search bar; use chevron icons for KeyboardArrowUp & KeyboardArrowDown (color still needs adjusting)

* Remove sx prop from CreateMenu button

* Fix KeyboardArrowUp and KeyboardArrowDown color in UserMenu.tsx

* MaxWidth initial

* Remove search bar border on hover

* Stop container holding x button in search bar from spilling outside of the bar

* Reduce zIndex of <Box /> wrapping <SideMenu /> to prevent the side menu from covering up the search dropdown

* Mock preference in PrimaryNav.test.tsx so items are expanded by default to prevent failures

* Saving...

* Saving...

* Saving...

* Final cleanup of top menu

* Cleanup UserMenu

* Remove commented code

* Fix TS errors

---------

Co-authored-by: Dajahi Wiley <dwiley@linode.com>
Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Co-authored-by: Jaalah Ramos <125309814+jaalah-akamai@users.noreply.github.com>

* new sidebar styles

* responsiveness adjustments

* Remove parentColumn - No longer need

* chore: [M3-9054] - CDS: Update all the fonts (#11517)

* Initial commit: all the font things

* smooth it all

* side nav and some captions

* small changes

* more instances

* more instances and linting rules + defensive props

* ubuntu no longer

* Update the footer and add hover to product families

* Update fonts from fontsource; Hide logo for mobile < 430

* Add yarn lock

* Fix the bolding of the active link

* Use theme for tokens

* cleanup and fixes

* moar cleanup and fixes

* fix test & linting errors

* update snapshots

* fix the e2e

* small style adjustments

* Revert "fix the e2e"

This reverts commit 35ba8aa.

* fix: [M3-9124] - Link styles in KubernetesClusterRow and clean up one-off styles  (#11523)

Cleans up one-off styles in KubernetesClusterRow
Uses Link from src/components/Link rather than react-router-dom directly so that colors are correct
In dark mode, the raw react-router-dom link is too dark


Co-authored-by: Banks Nussman <banks@nussman.us>

* CDS Feedback & Review Fixes

* Footer & Notification Updates

* Updates to Search, Search Menu, Accordion Icons

* Algolia Search

* Fix pin border issue

* Badge Padding, Pin Toggle Adjustments

* Small styling fixes

* fix beta chip getting cut off

* top nav responsiveness adjustments and fixes

* dark mode table label link and padding fixes

* refactor: [M3-8801] - Remove Individual Product Entity Icons (#11537)

* refactor: [M3-8801] - Remove unused product icons and rename remaining SVG files to reflect product families

* Replace EntityIcons with Product Family Icon and Update EntityIcon Story Accordingly

* Added changeset: Remove Individual Product Entity Icons and Update Story Accordingly

* Fix failed unit test cases

* fix in-progress chip border radius

* more dark mode table fixes

* side nav refinements

* update warning icon and notice component accordingly

* default akamai accounts to blue

* Update table header color

* Finishing Touches

* Fix issue with mobile visibility for search bar

* Increase product icons in nav and search menu, adjust top menu spacing, search placeholder italicized

* Padding around search

* Update our primary nav headings

* Update search final

* Fix issue with product names disappearing

* Prevent icon from shrinking when names are two lines, add padding

* Convert product icon to span

* fix lke-update.spec.ts

* fix unit fail

* fix remainning e2e tests- thx @jdamore-linode

---------

Co-authored-by: Harsh Shankar Rao <hrao@akamai.com>
Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Co-authored-by: hasyed-akamai <hasyed@akamai.com>
Co-authored-by: Alban Bailly <130582365+abailly-akamai@users.noreply.github.com>
Co-authored-by: Alban Bailly <abailly@akamai.com>
Co-authored-by: Dajahi Wiley <114682940+dwiley-akamai@users.noreply.github.com>
Co-authored-by: Dajahi Wiley <dwiley@linode.com>
Co-authored-by: Banks Nussman <115251059+bnussman-akamai@users.noreply.github.com>
Co-authored-by: Banks Nussman <banks@nussman.us>
Co-authored-by: Hana Xu <hxu@akamai.com>
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