Closed
Bug 496657
Opened 16 years ago
Closed 16 years ago
Tag Page: UI
Categories
(addons.mozilla.org Graveyard :: Public Pages, defect)
addons.mozilla.org Graveyard
Public Pages
Tracking
(Not tracked)
RESOLVED
FIXED
5.0.7
People
(Reporter: mikelee, Assigned: markwunsch)
References
Details
(Whiteboard: [webmocha])
Attachments
(3 files)
402.82 KB,
image/png
|
Details | |
1.10 KB,
patch
|
Details | Diff | Splinter Review | |
4.14 KB,
patch
|
Details | Diff | Splinter Review |
As indicated on the Tagging Spec
(http://docs.google.com/Doc?docid=dds6vwb4_1dxwx5vg3&hl=en)
1.10 - Tag page
A tag page should have an URL that is easy to understand, like "addons.mozilla.org/tag/photography". This page should contain a listing (similar to a search results page) for add-ons with a specific tag.
This bug involves the HTML/CSS/jQuery portion of this task.
Hi Nick,
As visual designs are completed for this UI flow (or if you're able to
articulate what the flow should do & look like), please add them to this bug.
Thanks.
Updated•16 years ago
|
Target Milestone: --- → 5.0.7
Comment 2•16 years ago
|
||
Neil, can you attach the designs you have for tagging (or link the bugs where you have them)?
Comment 3•16 years ago
|
||
All of the tagging design-related files are on bug #488517.
Thanks Neil. Do all of those images (tag icon & collections icon) exist already? If so, do you happen to know if they are in SVN or in the db? (Perhaps the second question is for Wil). Thanks!
Comment 5•16 years ago
|
||
(In reply to comment #4)
> Thanks Neil. Do all of those images (tag icon & collections icon) exist
> already? If so, do you happen to know if they are in SVN or in the db? (Perhaps
> the second question is for Wil). Thanks!
The small tag and collection icons are in /site/app/webroot/img/amo2009/icons/icons.png
Updated•16 years ago
|
Whiteboard: [webmocha]
Comment 6•16 years ago
|
||
As Wil said, as far as I know all of the image assets should be available already, but let me know if you're missing something.
Comment 7•16 years ago
|
||
So does this page look like like a SRP only there's a single tag on the left?
Yes. Except where it says, "Showing 1-10 of 401 results for bookmark manager",
perhaps it can say, "Showing 1-10 of 401 results for tag name".
So perhaps this bug is really a duplicate of:
https://bugzilla.mozilla.org/show_bug.cgi?id=496659
Comment 9•16 years ago
|
||
Mike is exactly right - the search subheading should change to say something along the lines of "Showing 1-10 of 401 items tagged with ##tag name##".
Sorry for not making this clear - it's something we discussed but it wasn't added to the wireframes or mock.
Reporter | ||
Comment 10•16 years ago
|
||
Oops - quick correction: this Tag Page is the same as the SERP page, but without the Refine Results (as said earlier in this bug) and without the "Search Results
Showing 1-10 of 401 results..." at the top.
Reporter | ||
Comment 11•16 years ago
|
||
FYI: the bug for the SERP UI is bug 496657. Both of these pages (Tags page & SERP page) will have a very similar UI. Both bug owners should coordinate with each other on these pages. And if someone is further along than the other, you can take on the corresponding bug too :)
Reporter | ||
Comment 12•16 years ago
|
||
Would help if I put in the correct SERP UI bug, huh? bug 496659
Comment 13•16 years ago
|
||
Given that this page is just a derivation of the other one, and there really should be just one owner since the code is the same, I'm sending this over to Mark.
Assignee: andrew → markwunsch
Reporter | ||
Comment 14•16 years ago
|
||
ETA: possibly tomorrow. Yem and Mark looking into this now.
Reporter | ||
Comment 15•16 years ago
|
||
Work in progress. Aiming to complete it tomorrow.
Reporter | ||
Comment 16•16 years ago
|
||
ETA moved to EOD Monday.
Comment 18•16 years ago
|
||
(In reply to comment #16)
> ETA moved to EOD Monday.
Status?
Assignee | ||
Comment 19•16 years ago
|
||
Has the same issues as bug 496659 -- needs to be localized
Reporter | ||
Comment 20•16 years ago
|
||
Notes from Mark:
I resolved my bugs involving SERP so unless something mysterious appears I'm trying to focus in on Tag Pages, which have caused me endless head scratching. I'm spinning my wheels about it.
Originally, since the UI for SERP and Tag pages were so similar, it made sense to not repeat the code and instead to make a route
amo/tag/tag_name => amo/search?tag=tag_name
But I ran into a bunch of obstacles with this approach. After talking it over, it makes more sense to do this the proper way (even if some code will be repeated) by creating this functionality in the Tag controller.
I'd like to have this knocked out at the very latest by eod tomorrow night.
Assignee | ||
Comment 21•16 years ago
|
||
This is how I designed the Tag Page to look like -- like the old-style SERP.
Assignee | ||
Comment 22•16 years ago
|
||
This is the new file for the VIEW for the display action of the tag controller (see bug 496655). Should go in app/views/tags/display.thtml.
Assignee | ||
Updated•16 years ago
|
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Attachment #386452 -
Attachment is patch: true
Attachment #386452 -
Attachment mime type: application/octet-stream → text/plain
Comment 23•16 years ago
|
||
I committed the page so we can test the other bugs but I'm reopening as this page is the old style.
At a minimum it should have the applications and the "sort by" dropdown at the top of it.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 24•16 years ago
|
||
Alright -- should there be a "Refine Results" similar to SERP as well? I'll work on this and work on some patches tonight.
Comment 25•16 years ago
|
||
Nope- no need for a refine UI- otherwise it might be too similar to a SERP.
Comment 26•16 years ago
|
||
You're probably already on it, but just so I don't forget: the title of the page needs to reflect the tag also
Assignee | ||
Comment 27•16 years ago
|
||
The app selector is now available, and you can sort the results. Default Sort is by popularity of addon.
Also the title of the page is "tag_name :: Add-ons for Firefox"
Assignee | ||
Updated•16 years ago
|
Status: REOPENED → RESOLVED
Closed: 16 years ago → 16 years ago
Resolution: --- → FIXED
Comment 28•16 years ago
|
||
Updated•9 years ago
|
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•