ui fixes

[surendran82]

Description

UI fixes for logo and side nav

Quick Links 🚀

🤖 DX: Assertion Tests

[robertjdominguez]

Hey, @surendran82 👋

Thanks for this PR! Since you changed docs files, our automated Action assigned Rob as your reviewer 🎉

Before Rob looks over the changes, we ask you to take care of a couple of items:

• Make sure you've checked over our PR guide on the wiki. It ensures you've done your due diligence on the basics: spelling, heading casings, etc.
• Rob will need a review from someone on your team to ensure your team-specific content is accurate. Once that's done, ping Rob on Slack, and he'll give a docs review 🔥
• Additionally, please check this box to confirm your feature works as expected when using the documentation you've written. This is critical to ensuring our users have the information they need.

The docs team aims to get all PRs reviewed within 48 hours of your team doing a review in the form of a content pass. Let Rob know the level of urgency on Slack 👍

[robertjdominguez]

DX: Assertion Testing

✅ Diff

The CSS class modifications in the PR help improve the visual consistency of UI elements such as border colors, padding adjustments, and box-shadow consistency. The introduction of a new color variable (--ifm-color-info-dark) ensures theme cohesiveness in the dark mode. Replacing the 'OverviewIconCard' component with a 'kitchen-image-container' div streamlines the HTML structure and may improve the page load performance, although it's unclear why the change was made without additional context regarding functionality or design consistency.

❌ Integrated

The removal of 'OverviewIconCard' and its replacement with a generic div erodes the semantic value and potential interactivity of the card component. Without seeing the broader context of changes, this alteration seems to diminish the overall usability and descriptiveness of the component system. Also, it is noted that certain classes like 'boderRadius0' have typographical errors which can cause confusion and prevent the classes from being used properly. Lastly, the usage of '!important' should be minimized as it could lead to maintenance problems and specificity wars within the stylesheet's integrated form.

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	91c44f3
Status:	✅  Deploy successful!
Preview URL:	https://d3307b0b.v3-docs-eny.pages.dev
Branch Preview URL:	https://surendran-v3-docs-04-03-2024.v3-docs-eny.pages.dev

View logs