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

Icons for settings and logout #797

Merged
merged 5 commits into from
Jan 28, 2025
Merged

Icons for settings and logout #797

merged 5 commits into from
Jan 28, 2025

Conversation

aldrin312
Copy link
Collaborator

Issue: #792

Added react-icons for settings and logout in the header menu.

image

Copy link
Collaborator

@humphd humphd left a comment

Choose a reason for hiding this comment

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

Screenshot 2025-01-26 at 12 40 34 PM

Let's be consistent with icons we use in the UI--we already have a "gear" icon being used in the Options menu, let's reuse that.

Copy link

cloudflare-workers-and-pages bot commented Jan 27, 2025

Deploying chatcraft-org with  Cloudflare Pages  Cloudflare Pages

Latest commit: 19401ca
Status: ✅  Deploy successful!
Preview URL: https://e36b349c.console-overthinker-dev.pages.dev
Branch Preview URL: https://menuicons.console-overthinker-dev.pages.dev

View logs

@humphd
Copy link
Collaborator

humphd commented Jan 28, 2025

Merge branch 'main' into menuIcons please don't do this. If you need to update your branch, rebase it. We only want to merge once when we land changes.

This looks good otherwise.

@humphd humphd merged commit 2da7fc9 into main Jan 28, 2025
4 checks passed
mulla028 pushed a commit that referenced this pull request Jan 30, 2025
* added hover over effect on sidebar items

* added settings and logout icons

* match settings icon with options

---------

Co-authored-by: aldrin312 <[email protected]>
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.

2 participants