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.
Meta Issue:#1870944: [Meta] Mobile friendly admin pages
Problem/Motivation
Search results both user and nodes are ordered lists. The numbers 1, 2 etc are partly visible in Narrow screens.
Admin URL examples:
- search/node/%
- search/user/%
Proposed resolution
To be determined.
Remaining tasks
To be determined.
Comment | File | Size | Author |
---|---|---|---|
#3 | search-result-1902856-3.patch | 428 bytes | echoz |
#3 | search-result-bartik.png | 29.03 KB | echoz |
#2 | Screen Shot 2013-02-04 at 4.10.39 PM.png | 32.7 KB | rcaracaus |
#2 | Screen Shot 2013-02-04 at 4.10.50 PM.png | 32.47 KB | rcaracaus |
#1 | fixed-padding-for-search-results-1902856-1.patch | 430 bytes | rcaracaus |
Comments
Comment #1
rcaracaus CreditAttribution: rcaracaus commentedI have tried my hand at submitting a fix for this. However there are still some design decisions that need to be made.
This fix reverts to the themes default padding for ols, adding back a padding of what looks like 15px to the left. The bad part is that the border is also pushed over, can't think of a way to fix that without adding extra markup and adding negative padding. Not sure if this is an issue, it beats not the numbers being cut off on mobile and the numbers being pulled between the columns even on normal displays.
Comment #2
rcaracaus CreditAttribution: rcaracaus commentedComment #3
echoz CreditAttribution: echoz commentedHow about list-style-position: inside; and I think it looks better wide too, so not wrapping with a media query. Note that the ol qualifier was needed on the selector :-( and @rcaracaus, you know there's no such css as negative padding, right?
Comment #4
rcaracaus CreditAttribution: rcaracaus commentedNice.. I didn't know about list-style-position: inside; That is very useful... also I meant to say negative margin.
Comment #5
rcaracaus CreditAttribution: rcaracaus commentedI tested this locally.. and looked at it in different displays.. looks good.
Comment #6
Shyamala CreditAttribution: Shyamala commentedadding tags
Comment #7
echoz CreditAttribution: echoz commented@rcaracaus, so is the patch in #3 rtbc for you? The file hasn't changed since so it should apply fine.
Comment #8
echoz CreditAttribution: echoz commentedtags removed by d.o unintentionally
Comment #9
rcaracaus CreditAttribution: rcaracaus commentedI have tested the patch and it looked OK to me. I am not sure what standards are for rtbc.
Comment #10
carwin CreditAttribution: carwin commentedPatch in #3 applies cleanly and
list-style-position: inside;
seems like a fine solution to the problem.RTBC. Huzzah!
Comment #11
webchickAwesome work on this!
Committed and pushed to 8.x. Thanks!