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

chore: final touches #3841

Merged
merged 21 commits into from Sep 28, 2023
Merged

chore: final touches #3841

merged 21 commits into from Sep 28, 2023

Conversation

bigint
Copy link
Member

@bigint bigint commented Sep 28, 2023

What does this PR do?

πŸ€– Generated by Copilot at e0eaabb

This pull request updates the UI components and styles to use the new brand color palette with shades of pink. It adds a useBrandColor prop to the Slug component and uses it in the Mention component. It also overrides the buffer bar color of the video player to use white. The purpose of these changes is to improve the consistency, contrast, and accessibility of the UI and to align it with the new branding and design system.

Related issues

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (non-breaking small changes to existing functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Explanation of the changes

πŸ€– Generated by Copilot at e0eaabb

  • Add useBrandColor prop to Slug component to control text color (link, link)
  • Update Sidebar and TabButton components to use new brand color palette (link, link)
  • Override buffer bar color in video player with white (link)
  • Define new brand color object with 10 shades of pink in tailwind-preset.ts (link)

Emoji

🎨🎬🏷️

@vercel
Copy link

vercel bot commented Sep 28, 2023

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Updated (UTC)
prerender βœ… Ready (Inspect) Visit Preview Sep 28, 2023 1:49pm
web βœ… Ready (Inspect) Visit Preview Sep 28, 2023 1:49pm

@vercel vercel bot temporarily deployed to Preview – prerender September 28, 2023 06:34 Inactive
Copy link
Contributor

@harish-sethuraman harish-sethuraman left a comment

Choose a reason for hiding this comment

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

LFGTM with 69 file changes

@vercel vercel bot temporarily deployed to Preview – prerender September 28, 2023 13:46 Inactive
@bigint bigint merged commit 4c8fbd1 into main Sep 28, 2023
7 of 8 checks passed
@bigint bigint deleted the final-touches branch September 28, 2023 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants