Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
Bartik theme
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
28 May 2010 at 13:16 UTC
Updated:
29 Nov 2010 at 09:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
bleen commentedBlaaaarg ... wrong screenshot.

Comment #2
aspilicious commentedComment #3
Jeff Burnz commentedDo we really want all that padding for search results?
Comment #4
aspilicious commentedNo we don't want that ;) (my opinion)
Comment #5
Jeff Burnz commentedJust updating bleens patch, removes padding from the OL.
Comment #6
jensimmons commentedHere's what the search result page looks like now, with no patch applied:
Comment #7
jensimmons commentedComment #8
Jeff Burnz commented#5: bartik-search-result-v1.patch queued for re-testing.
Comment #9
Jeff Burnz commentedClearly the patch will fail re testing so I wont bother, however this really needs work, search results could look better.
Comment #10
mason@thecodingdesigner.com commentedI styled the search results a bit. not the left padding has been removed, and I've added a 1px border between listings.
Comment #12
mason@thecodingdesigner.com commentedcleaned up the code, rolling a new patch
Comment #13
tim.plunkettThis is exactly what we needed. Jen signs off on it. Awesome!
Here are screenshots:
BEFORE

AFTER

Comment #14
bleen commentedRTBC++ ... so much more betterer
Comment #15
webchickNice improvement.
Committed to HEAD. Thanks!