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

REMOVED: Dark Theme Button Text Coloring #14616

Merged
merged 2 commits into from
Apr 23, 2024
Merged

Conversation

akemidx
Copy link
Collaborator

@akemidx akemidx commented Apr 18, 2024

Description

Dark theme buttons when hovered became unreadable.
This removes that hover effect so the text remains readable.
The 'click' effect remains so a user will know that they clicked the button.

Additionally, a white border was added to the dark black theme buttons, and a black border was added to the dark yellow theme buttons

This affects the header search bar magnifying glass, the Back, and the Save buttons.

Screenshot 2024-04-18 at 7 43 46 PM Screenshot 2024-04-18 at 7 44 03 PM Screenshot 2024-04-18 at 7 43 19 PM Screenshot 2024-04-18 at 7 44 12 PM Screenshot 2024-04-18 at 7 43 08 PM Screenshot 2024-04-18 at 7 44 25 PM Screenshot 2024-04-18 at 7 42 55 PM Screenshot 2024-04-18 at 7 44 36 PM Screenshot 2024-04-18 at 7 42 38 PM Screenshot 2024-04-18 at 7 44 49 PM Screenshot 2024-04-18 at 7 42 18 PM Screenshot 2024-04-18 at 7 45 00 PM Screenshot 2024-04-18 at 7 41 43 PM Screenshot 2024-04-18 at 7 45 13 PM

Fixes # SC-23235

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Visually tested in local instance

Checklist:

Copy link

what-the-diff bot commented Apr 18, 2024

PR Summary

This Pull Request involves changes in visual elements for an improved and consistent user experience across different themes:

  • Change in Black Theme
    The main button (.btn-primary) in the black theme is now outlined with white color (#FFF).

  • Removal of Hovering Effect in Multiple Themes
    The color changes on hovering over the main button on the blue, green, orange, purple, and red themes have been removed. This simplifies user interactions and ensures consistency across themes.

  • Change in Yellow Theme
    The main button (.btn-primary) in the yellow theme is updated to have a black color (#000000) outline, enhancing its visibility and contrast against the yellow background.

@snipe snipe requested a review from Godmartinz April 22, 2024 13:41
@snipe snipe merged commit c50d5a6 into snipe:develop Apr 23, 2024
8 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.

2 participants