616269 - Excessive space between the headings in EULA/ Custom License page

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[krupa raj[:krupa]]

Attachment: screenshot

Excessive space between the headings in EULA/ Custom License page

See 

https://addons.mozilla.org/en-US/firefox/addon/211331/eula/96968?src=addondetail 

https://addons.allizom.org/en-US/firefox/addon/3615/license/2.1.041

This is giving Fligtar a severe headache. Please fix.

Comment 1

[Jeff Balogh (:jbalogh)]

(In reply to comment #0)
> This is giving Fligtar a severe headache. Please fix.

Fligtar knows how to CSS.

Comment 2

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #1)
> (In reply to comment #0)
> > This is giving Fligtar a severe headache. Please fix.
> 
> Fligtar knows how to CSS.

CSS is not a verb.  Not even in g33k-speak.

Comment 3

[Justin Scott [:fligtar]]

why is this future? this looks horrible.

Comment 4

[Potch [:potch]]

This looks horrible even still now with the new header? I don't see a problem.

Comment 5

[Justin Scott [:fligtar]]

Yes, it still exists with the new header.

https://addons.allizom.org/en-US/firefox/addon/966/eula/79565?src=addondetail

Comment 6

[Wil Clouser [:clouserw]]

You're talking about the vertical space between "Apture Highlights" and "End-User License Agreement"?  You think that looks horrible?

Comment 7

[Potch [:potch]]

I'm not going to waste time guessing what the magic numbers to fix the spacing are, since I don't even see a problem with what's there, but you can play with it in Firebug and let me know what changes satisfy you.

Comment 8

[Justin Scott [:fligtar]]

Am I the only person who thinks there's a huge gap there?

Comment 9

[Wil Clouser [:clouserw]]

Getting rid of margin-bottom on the h2.name and margin-top on div.primary.h3 fixes this.

Comment 10

[Wil Clouser [:clouserw]]

fresh link: https://addons-dev.allizom.org/en-US/firefox/addon/tamper-data/eula/79565?src=addondetail

Comment 11

[Wil Clouser [:clouserw]]

this looks fixed in the new design.