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

💄 Replace the search button with a magnifying glass #483

Conversation

loikx
Copy link
Contributor

@loikx loikx commented Jan 3, 2024

What does this PR do?

This pr improve design of search bar with replacing search button with magnification glass.

Why is this change important?

Improve UI, make it look more modern, improve the user experience

Related issues

Closes #450

…image of any size to button, change bar.rs to set image
Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Thanks ❤️ for working on this PR. We really appreciate it 👍 everything looks good but I think the magnifying glass color might not work on dark colorschemes like for example I think it might become invisible on colorscheme like dark-chocolate. So I would suggest using a magnifying glass image of different color or maybe you can also change the image dynamically by taking the svg code and putting it in the HTML file directly and then changing the fillproperty of svg image to change image colors dynamically. 🙂

@loikx
Copy link
Contributor Author

loikx commented Jan 5, 2024

Thaks for such positive feedback, but I'm not clear about color change. I've checked all themes with black color. So if I need to change image color, what color I need to pick?

Dark Chocolate

Снимок экрана 2024-01-05 в 20 56 29

Solarized DarK

Снимок экрана 2024-01-05 в 20 59 01

Tomorrow Night

Снимок экрана 2024-01-05 в 21 00 05

Tokyo Night

Снимок экрана 2024-01-05 в 21 00 54

@neon-mmd
Copy link
Owner

neon-mmd commented Jan 6, 2024

Thaks for such positive feedback, but I'm not clear about color change. I've checked all themes with black color. So if I need to change image color, what color I need to pick?

Dark Chocolate

Снимок экрана 2024-01-05 в 20 56 29 ### Solarized DarK Снимок экрана 2024-01-05 в 20 59 01 ### Tomorrow Night Снимок экрана 2024-01-05 в 21 00 05 ### Tokyo Night Снимок экрана 2024-01-05 в 21 00 54

After through testing, all the color schemes on our side. We found that the icon you have chosen works on all the color schemes, and as such we would suggest leaving the icon as it is. 🙂

neon-mmd

This comment was marked as outdated.

Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Thanks ❤️ again for working on this PR. Since everything looks good. We will merge the PR right away 🚀 🙂

@neon-mmd neon-mmd merged commit 33846ce into neon-mmd:rolling Jan 6, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

💄 Replace the search button with a magnifying glass
2 participants