Download & Extend
Issues

No "padding: none"

Posted by droplet on June 23, 2012 at 12:59am
Project:Drupal core
Version:7.x-dev
Component:CSS
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed (fixed)
Issue tags:needs backport to D7, Quick fix

Issue Summary

.search-results li:last-child {
  border-bottom: none;
  padding-bottom: none;
  margin-bottom: 1em;
}

check it out:
http://jigsaw.w3.org/css-validator/validator
http://www.w3.org/TR/CSS2/box.html#padding-properties

AttachmentSizeStatusTest resultOperations
no-padding-none.patch404 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 37,023 pass(es).View details
Login or register to post comments

Comments

#1

Posted by RobLoach on June 23, 2012 at 1:00am
Priority:normal» minor
Status:needs review» reviewed & tested by the community

Easy one!

#2

Posted by RobLoach on June 23, 2012 at 1:04am
Status:reviewed & tested by the community» needs work

Hold on, I found another.

#3

Posted by RobLoach on June 23, 2012 at 1:06am
Status:needs work» reviewed & tested by the community

Nope! I was just seeing double :-) .

#4

Posted by droplet on June 23, 2012 at 1:10am

:)

#5

Posted by Dries on June 23, 2012 at 7:56pm
Version:8.x-dev» 7.x-dev

Committed to 8.x. Thanks. Moving to 7.x.

#6

Posted by droplet on June 23, 2012 at 9:38pm
Status:reviewed & tested by the community» needs review
AttachmentSizeStatusTest resultOperations
padding_none.patch384 bytesIdlePASSED: [[SimpleTest]]: [MySQL] 39,212 pass(es).View details

#7

Posted by System Message on June 23, 2012 at 10:16pm
Status:needs review» needs work

The last submitted patch, padding_none.patch, failed testing.

#8

Posted by droplet on June 23, 2012 at 10:19pm
Status:needs work» needs review

#6: padding_none.patch queued for re-testing.

#9

Posted by RobLoach on June 24, 2012 at 7:30am
Status:needs review» reviewed & tested by the community

#10

Posted by webchick on June 30, 2012 at 6:39pm
Status:reviewed & tested by the community» fixed

Normally we do not make markup/CSS changes to stable releases, but since this is isolated to Bartik I think it is ok.

Committed and pushed to 7.x. Thanks!

#11

Posted by cweagans on July 12, 2012 at 8:11am
Issue tags:+needs backport to D7

Fixing tags per http://drupal.org/node/1517250

#12

Posted by System Message on July 26, 2012 at 8:11am
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here