-
-
Notifications
You must be signed in to change notification settings - Fork 31.3k
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 #34383 -- Fixed layout of admin fieldsets with multiple fields on the same line. #16678
Conversation
We should probably also add the |
I think we should remove diff --git a/django/contrib/admin/static/admin/css/responsive.css b/django/contrib/admin/static/admin/css/responsive.css
index a75d1c5d66..058e196968 100644
--- a/django/contrib/admin/static/admin/css/responsive.css
+++ b/django/contrib/admin/static/admin/css/responsive.css
@@ -564,8 +564,6 @@ input[type="submit"], button {
.aligned .form-row,
.aligned .form-row > div {
- display: flex;
- flex-wrap: wrap;
max-width: 100vw;
}
instead of: |
@knyghty Thanks 👍 Great team work 🤝 |
@felixxm Fixed those, seems to work as I'd expect now... from what I've tested. |
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.
@knyghty Thanks 👍
I added one more minor fix, and pushed changes to tests.
@camarone Does it work for you? |
…on the same line. Thanks Antonio Candido Nazareth junior for the report. Regression in 96a5983. Co-authored-by: Mariusz Felisiak <felisiak.mariusz@gmail.com>
buildbot, test on selenium. |
No description provided.