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

Uhf x button color bug #833

Merged
merged 2 commits into from
Nov 22, 2023
Merged

Uhf x button color bug #833

merged 2 commits into from
Nov 22, 2023

Conversation

Arkkimaagi
Copy link
Contributor

@Arkkimaagi Arkkimaagi commented Nov 22, 2023

UHF-X

  • Dark themed banners have black buttons when they should be white

What was done

  • Figured out what was going on.
    • Due to CkEditor5 bugging and creating empty elements, at the last moment before code going live, we "fixed" ckeditor buttons by enforcing new button styles only to apply on non-links and links that have to have href (the cke5 bug does not have hrefs on links)
    • This change was forgotten from similar selector that's used elsewhere using a color helper scss file
    • Due to selector specificity changing to a higher level where it was not supposed to change, colors did not work on buttons as they were planned.
    • Similar rules were added to color helpers and it fixed the problem.

How to install

  • Make sure your instance is up and running on latest dev branch.
    • git pull origin dev
    • make fresh
  • Update the HDBT theme
    • composer require drupal/hdbt:dev-UHF-X_button_color_bug
  • Run make drush-cr

How to test

  • Check that buttons work on all theme backgrounds (especially dark banner background colors should have white buttons)
  • Check that code follows our standards

Designers review

  • This PR does not need designers review

Copy link
Contributor

@khalima khalima left a comment

Choose a reason for hiding this comment

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

This fix works. Thanks for this, awesome job to find it so fast! 💪 🚀

image

Copy link
Contributor

@teroelonen teroelonen left a comment

Choose a reason for hiding this comment

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

This works correctly for me too 🦖 Great work!

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.

3 participants