Skip to content

Conversation

@akemidx
Copy link
Member

@akemidx akemidx commented Oct 24, 2024

Description

The login button can disappear when the header color in Admin>Branding is set to white (#FFFFFF) since the button is set on a white background.
It appears on hover, and is still usable.
Some styling has been removed and the button will no longer take on the header color and thus will always be visible.

before
Screenshot 2024-10-24 at 5 20 43 PM
current button
Screenshot 2024-10-24 at 5 20 47 PM

Fixes # (26974)

Type of change

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

How Has This Been Tested?

visual test and test suite

Test Configuration:

  • PHP version: 8.1
  • OS version: Sonoma 14.6

Checklist:

@akemidx akemidx requested a review from snipe as a code owner October 24, 2024 21:33
@what-the-diff
Copy link

what-the-diff bot commented Oct 24, 2024

PR Summary

  • Elimination of Custom Styles for Button

    The unique styling previously set for the primary button, which determined its background and border color, has been discarded. This could potentially allow for a more consistent interface look, maintainable design, and increase overall user experience quality.

Copy link
Member

@snipe snipe left a comment

Choose a reason for hiding this comment

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

This doesn't account for the scenario where someone has a light header color, so you could end up with white text on a white header. We don't currently ask folks to set font colors, link colors, etc though, so I'm not really sure how to handle this properly (per our meeting a few weeks ago.)

@akemidx
Copy link
Member Author

akemidx commented Oct 28, 2024

This doesn't account for the scenario where someone has a light header color, so you could end up with white text on a white header. We don't currently ask folks to set font colors, link colors, etc though, so I'm not really sure how to handle this properly (per our meeting a few weeks ago.)

correct, this does NOT fix the header color or header washing out. This ONLY fixes the login button disappearing while a user has a header color chosen to be white.

The logic being that, if a user logs out while having that white header, at LEAST they will have a normal login, to get back in and change that header back. Our login window is static in its coloring, EXCEPT for that login button, so this makes it completely standardized.

@snipe snipe merged commit 02ba13b into grokability:develop Nov 13, 2024
9 checks passed
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