Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Simple fix here, turn two columns into one.
Slightly complicated because it looks like we've implemented these floats using unsemantic css classes.
Best do this right and rewrite the markup for semantic mark up + media queries.
Comment | File | Size | Author |
---|---|---|---|
#13 | admin-responsive.png | 81.51 KB | alanburke |
#12 | Responsive-admin-panels-1150134-12.patch | 2.12 KB | nod_ |
#9 | Responsive-admin-panels-1150134-9.patch | 2.4 KB | dcmouyard |
#7 | Responsive-admin-panels-1150134-7.patch | 1.62 KB | dcmouyard |
Comments
Comment #1
LewisNymanPart of #1137782: Remove unused CSS in Seven theme that uses desktop-sized layouts
Comment #2
JohnAlbinThis is for pages like the admin/configuration landing page. We should document all of those pages and make sure they are all built very similarly, preferably with the exact same classes.
Comment #3
jefflinwood CreditAttribution: jefflinwood commentedI'm happy to do this, but I don't want to step all over work that's already been done. Has anyone gone through and done an audit core's admin panels?
Comment #4
LewisNymanI've gone through the code. It's a right mess. The html output ends up being a "left" and a "right" div split in two.
We need to kill that and output a basic unordered list.
Comment #5
JohnAlbinOver in #1137782: Remove unused CSS in Seven theme that uses desktop-sized layouts, Jeff Linwood says:
I think that comment will help move this issue along.
Comment #6
dcmouyard CreditAttribution: dcmouyard commentedI'll be working on a patch for this.
Comment #7
dcmouyard CreditAttribution: dcmouyard commentedHere's a first stab at this issue. I prefer to use em values in media queries, so I set the breakpoint at 40em.
Comment #8
dcmouyard CreditAttribution: dcmouyard commentedMarking as needs review.
Comment #9
dcmouyard CreditAttribution: dcmouyard commentedForgot to edit the rtl stylesheet in the last patch...
Comment #10
LewisNymanThe patch works in the browser at a 640px breakpoint.
I'd be happy to approve this but I am unable to test this on mobile devices because they simulate a 960px viewport. This issue is now reliant on: #1468582: Add mobile friendly meta tags to the html.tpl.php
Comment #11
nod_works for me, but it looks like it needs a reroll.
Comment #12
nod_Here it is.
Comment #13
alanburke CreditAttribution: alanburke commentedThis is a good improvement.
Tested with Chrome .
Comment #14
yoroy CreditAttribution: yoroy commentedThis patch is a Good Thing.
Comment #15
catchMakes sense. Committed/pushed to 8.x.
Comment #16
JohnAlbinNice work!