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

FIXED: Login button can disappear under some circumstances #15730

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

akemidx
Copy link
Collaborator

@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
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
Owner

@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
Collaborator 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 snipe: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