Padding values of div#content were (accidentially?) changed (in LESS rewrite?)
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	liangent
	Nov 22 2013, 7:06 AM

Description

For small screens, it now has:

padding: 1.5em 1.5em 1.5em 1.75em;

For big screens (982px+), it now has:

padding: 1.25em 1.5em 1.5em;

Some time ago, these values was:

padding: 1em; (small)
padding: 1.25em 1.5em 1.5em 1.5em; (big)

Anyway it's not reasonable to use bigger paddings now for smaller screens.

Version: unspecified
Severity: normal

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:34 AM

• bzimport set Reference to bz57401.

Fixed in I2551ab16. I couldn't find this bug when committing that, heh.

(In reply to comment #1)

Fixed in I2551ab16. I couldn't find this bug when committing that, heh.

So finally both paddings got enlarged, not changed back to original values?

Oh, hmm. So it would seem, yeah.

(In reply to comment #3)

Oh, hmm. So it would seem, yeah.

And it won't be changed back?

Unless you submit a patch ;) The change seems unintentional to me.

Change 111454 had a related patch set uploaded by Liangent:
(bug 57401) Fix padding values of div#content

Thanks.

Change 111454 merged by jenkins-bot:
Vector: Fix padding values of div#content