498082 - Multiple layout problems with manage collections add-ons page

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Collections, defect, P4)

Description

[Dave Garrett]

Attachment: screenshots with circled problems

1) Useless horizontal scrollbar
2) No padding between comment and "Remove" link
3) Added date works fine on Shiretoko but pushes things off to the side on Firefox 3

see attached

Comment 1

[Dave Garrett]

(if any of this warrants its own bug, please feel free to spinoff)

Comment 2

[Wil Clouser [:clouserw]]

-> neil

Comment 3

[Neil Lee [:neilio]]

Attachment: Patch to fix these issues

This should fix these up right.

Comment 4

[Wil Clouser [:clouserw]]

Comment on attachment 395400 [details] [diff] [review]
Patch to fix these issues

Looks great.  Can you commit this or do you need me to?

Comment 5

[Neil Lee [:neilio]]

Fixed: r49549.

Comment 6

[Dave Garrett]

Attachment: still has some issues, particularly on Firefox 3

Just tested on preview here:
https://preview.addons.mozilla.org/en-US/firefox/collections/edit/0dec6ef8-a985-7d9e-8817-61c4c6c0ad83

The scrollbar is gone in both versions and the wide right margin is gone on Firefox 3. However, the remove link is still very close to the text and Firefox 3 still has two layout glitches. See attached.

Instead of widening the padding next to the remove link, why not just make it stay at the bottom under everything? There's already room underneath given for the edit/delete publisher comment links so it should just be down there in the lower-right corner with the comment going over it all the way to the right.

Comment 7

[Dave Garrett]

Attachment: quick mockup of better layout

Quick mockup for what I said at the end of comment 6 in case I was as unclear as I think it was. ;)

Comment 8

[Neil Lee [:neilio]]

Looking at this and actually testing it out a fairly significant usability problem is evident - the "remove" link looks like it's tied to the publisher comment when it actually removes the add-on from the list. As there's no confirm for the remove action I'm thinking that moving this link is a good idea.

I propose something like this:

http://snaps.beatnikpad.com/remove-addon-20090820-092043.png

My suggestion is we push what we have for now as it fixes the issue for the majority of users, and we open a new bug to fix the positioning of this link.

clouserw, what say ye?

Comment 9

[Dave Garrett]

Changing the wording to "remove this add-on" as you suggest would definitely be better. Sticking it under the name could be good or with it reworded the lower-right corner would be good too. Either is better than currently.

Another bug for futzing with the remove link is fine by me, but what about the Firefox 3 specific layout glitches? (attachment 395484 [details]) Something is still causing breaking under Firefox 3.

As indicated by comment 1, I'm aware I filed 3 bugs in one here so if you want another for that too then I'll spinoff. Issue #1 is definitely fixed, though.

Comment 10

[Wil Clouser [:clouserw]]

Yeah, "remove this add-on" is far better.  I don't care where the link is.  Neil: code freeze isn't until midnight tonight (PST).  Do you have time to make these changes in 5.0.9?

Comment 11

[Neil Lee [:neilio]]

clouserw: should be fine - I'll ping on IRC if I run into any issues.

Comment 12

[Neil Lee [:neilio]]

Attachment: Moves the Remove link, tweaks the text

This patch moves the Remove link and updates the text at (in en-US for now) to "Remove this add-on".

Comment 13

[Wil Clouser [:clouserw]]

Comment on attachment 395778 [details] [diff] [review]
Moves the Remove link, tweaks the text

You'll have to change the text in addon_tags.thtml too.  It's line 263, same text change as the .po.

Comment 14

[Neil Lee [:neilio]]

Attachment: New patch that fixes the thtml file

Same as the last patch, but fixes the reference in the thtml for the new text.

Comment 15

[Fred Wenzel [:wenzel]]

I changed the actual string use (in the edit.thtml file) too, and committed neilio's patch to r49697.

Is this bug fixed now? If so, please close it, neilio.

Comment 16

[Dave Garrett]

The remove link is better worded and in a better place now, but there's still problems here I'm afraid to say. The 'R' in remove is now touching or overlapping the lower right corner of my icon when testing with the test link in comment 6. Also, in Firefox 3 all the layout errors are still present. (mispositioned date and the weird hole in the paragraph; see attachment 395484 [details])

Comment 17

[Neil Lee [:neilio]]

r49703 fixes the small margin issue. Dave, please break the Firefox 3 issue into a new bug.

Comment 18

[Dave Garrett]

(In reply to comment #17)
> r49703 fixes the small margin issue.

Looks perfect under FF3.5 now, thanks.

> Dave, please break the Firefox 3 issue into a new bug.

Done -> bug 511904