465439 - show tags in view page

Status: VERIFIED FIXED

(Websites :: communitystore.mozilla.org, enhancement)

Comment 1

[Ryan Doherty (:rdoherty)]

Attachment: Patch to add tags

This adds tags to the view page. Not sure on the design/layout yet, planning on getting updates for it once the basic functionality is committed.

Comment 2

[Ryan Doherty (:rdoherty)]

r20881

Tara, I've added the tags to the detailed view page if you want to check it out. If you have changes, let me know asap.

Comment 3

[Tara (musingt)]

Hey Ryan, where can I see the updated page?  Is there a new stage site?

Comment 4

[Ryan Doherty (:rdoherty)]

(In reply to comment #3)
> Hey Ryan, where can I see the updated page?  Is there a new stage site?

It's the same site it has always been:
http://community-store.stage.mozilla.com/

Comment 5

[Ryan Doherty (:rdoherty)]

And here's a design with tags:
http://community-store.stage.mozilla.com/gallery/view/37

Comment 6

[Tara (musingt)]

Hi there.  I think it looks great.  Only thing I'd change is the font size.  A few types smaller to allow for more tags without distorting the layout too much.

Comment 7

[Ryan Doherty (:rdoherty)]

r20904 Has a slightly smaller fontsize. I experimented with an even smaller fontsize, but it began to look out of place.

Comment 8

[Tara (musingt)]

Sure.  I'm just assuming that people will easily be able to apply many tags to a design, and I wonder what it would look like if we have 10-15 tags for example in that space.  I think tags are less important than the rest of the copy there, so smaller is okay as long as it doesn't look too odd.

Comment 9

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

Verified FIXED on http://community-store.stage.mozilla.com/gallery/view/37; if we want additional tweaks/features associated with tags, we'll file separate bugs.

Comment 10

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

Attachment: Screenshot with lots of tags

Two things about this screenshot:

1) proves the functionality works
2) exposes the height-constraints of the layout, which we'll file as a separate bug

Comment 11

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

(In reply to comment #10)
<snip>
> 2) exposes the height-constraints of the layout, which we'll file as a separate
> bug

Filed bug 469879.

Comment 12

[Ryan Doherty (:rdoherty)]

r20916 has the latest changes according to my discussion with Tara. This includes a smaller fontsize and a darker color for the main paragraph in the detailed view.

Comment 13

[Tara (musingt)]

Looks good to me.  I'll play around with it to see what longer lines of tags looks like.  Otherwise, thumbs up!