-
Notifications
You must be signed in to change notification settings - Fork 187
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
[AdminBundle] Update _form-group.scss #2199
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rmibelgium, your PR needs some changes
- It seems that our checklist is missing or incomplete
- your PR title should look like
[SomeBundle] Fixed some code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rmibelgium, your PR needs some changes
- Your answer if this PR is BC breaking seems to be incorrect.
- It seems that you should have submitted to the latest minor branch.
- This PR seems to need a milestone of a patch release.
@@ -16,6 +16,10 @@ | |||
max-width: none; | |||
} | |||
|
|||
.form-group--no-max-width textarea { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you group both selectors instead of duplicating the style declaration.
In our cms, rawHtml type didn't respect the form-group--no-max-width because the textarea under has class form-control which overwrite with max-width: 40rem;
* 5.1: update changelog updated changelog [AdminBundle] fix error state for select2 & wysiwyg fields (#2352) [AdminBundle] Revert PR #2049 changes (#2283) [ArticleBundle] Fixed missing sprintf with classname in deprecation message (#2334) [GeneratorBundle] Fix incorrect plural of certain table names (#2277) [Admin][Cache][Config][Generator][Pagepart] fix some routes that didn't use the configurable admin prefix (#2266) [AllBundles] Remove php 7.3 check as php-cs-fixer supports it now [AdminBundle] Update _form-group.scss (#2199)
In our cms, rawHtml type didn't respect the form-group--no-max-width because the textarea under has class form-control which overwrite with max-width: 40rem;