Skip to content

Fix dark mode design compatibility #401

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

Merged
merged 1 commit into from
Oct 11, 2021
Merged

Fix dark mode design compatibility #401

merged 1 commit into from
Oct 11, 2021

Conversation

ostrolucky
Copy link
Collaborator

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Related tickets -
Documentation
License MIT

What's in this PR?

Before:

image image
After:
image image

Checklist

  • Updated CHANGELOG.md to describe BC breaks / deprecations | new feature | bugfix
  • Documentation pull request created (if not simply a bugfix)

To Do

  • Not compatible with Symfony < 4.2. Such Symfony versions are not maintained though. Worth adding conflict rule?

Copy link
Collaborator

@dbu dbu left a comment

Choose a reason for hiding this comment

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

very nice, thanks!

i would prefer to add the old values as fallback over dropping Symfony 3.4 support for just this reason. if there is a more compelling reason to drop support, i am fine with it, but having the fallback values seems easy enough to me to not need to drop it.

Copy link
Collaborator

@dbu dbu left a comment

Choose a reason for hiding this comment

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

great, thanks!

@dbu dbu merged commit a7f34dd into master Oct 11, 2021
@ostrolucky ostrolucky deleted the fix-darkmode branch October 16, 2021 18:24
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