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

all: smoother navigation icons (fixes #7980) #7982

Merged
merged 7 commits into from
Jan 10, 2025

Conversation

jessewashburn
Copy link
Member

image
Richard mentioned some confusion with the icons in the top right of the nav-bar, and I agree. I think this new icon is much clearer than the simple text bubble.

fixes #7980

@jessewashburn jessewashburn linked an issue Dec 20, 2024 that may be closed by this pull request
Copy link
Member

@Gavinp14 Gavinp14 left a comment

Choose a reason for hiding this comment

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

I agree, this looks a lot better than the previous icon.

@Mutugiii
Copy link
Member

Mutugiii commented Dec 20, 2024

Makes sense, potentially you can look into some of the other icons around the app particularly during mobile screens and tackle them in this PR as well

image

The ones I find a bit unclear are the courses, resources and maybe dashboard... In case of no suitable alternatives we can use the existing ones

@jessewashburn
Copy link
Member Author

jessewashburn commented Dec 23, 2024

Here are some proposed alternatives to the current icons. I chose solid white icons, as they're easier to see. I understand all of these may not get approved, so please let me know which ones you do and do not like. Note that the home icon is solid white as well. It's just selected here.

image

Copy link
Member

@Gavinp14 Gavinp14 left a comment

Choose a reason for hiding this comment

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

The dashboard icons are a lot crisper here. I am not sure how many options are out there for the ai chat icon but were you able to find anything with like an ai logo to let the user know that it is an ai message rather than talking to another person?

@jessewashburn jessewashburn changed the title navbar - clearer chat icon (fixes #7980) navbar - clearer icons (fixes #7980) Dec 23, 2024
@jessewashburn
Copy link
Member Author

The dashboard icons are a lot crisper here. I am not sure how many options are out there for the ai chat icon but were you able to find anything with like an ai logo to let the user know that it is an ai message rather than talking to another person?

I considered this, but the options for more specific AI chat icons would either have the word AI, which doesn't work for Spanish since they call it IA, or some kind of robot image, which I don't think suits our style.
chatbot(1)
chatbot

@jessewashburn
Copy link
Member Author

I opted to put all the icon options in this doc. If anyone wants to suggest new ones, feel free to add to it. https://docs.google.com/document/d/1tmiofPLZW1uve1n1z_5sOzE2EucfsghaeYUDEAcVWtI/edit?usp=sharing

@Mutugiii
Copy link
Member

Mutugiii commented Jan 2, 2025

I think the suggested ones are an overall improvement on the current ones and we should implement them, for the resources I prefer option 1

@jessewashburn
Copy link
Member Author

I think the suggested ones are an overall improvement on the current ones and we should implement them, for the resources I prefer option 1

Great, then this PR should be good to go.

@Mutugiii
Copy link
Member

For reference:

image

@dogi dogi changed the title navbar - clearer icons (fixes #7980) all: smoother navigation icons (fixes #7980) Jan 10, 2025
@dogi dogi merged commit 19a772a into master Jan 10, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nav-bar - more distinctive chat icon
4 participants