Search page results in narrow screens [#1902856]

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
#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
#1
	Search results page in narrow screens.PNG	17.49 KB	Shyamala

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

rcaracaus CreditAttribution: rcaracaus commented 4 February 2013 at 05:07

File	Size
fixed-padding-for-search-results-1902856-1.patch	430 bytes

I 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 commented 4 February 2013 at 05:10

File	Size
Screen Shot 2013-02-04 at 4.10.50 PM.png	32.47 KB
Screen Shot 2013-02-04 at 4.10.39 PM.png	32.7 KB

Comment #3

echoz CreditAttribution: echoz commented 4 February 2013 at 11:40

Status:

Active

» Needs review

File	Size
search-result-bartik.png	29.03 KB
search-result-1902856-3.patch	428 bytes

How 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?