860402 - [fireplace] Add category dropdown

Status: VERIFIED FIXED

(Marketplace Graveyard :: Consumer Pages, defect)

Description

[Christopher Van Wiemeersch [:cvan]]

Mobile:
http://f.cl.ly/items/3s2g022z443R1y1C2s3G/[e-mail]
Upon :active, the background color of the category row should be blue (which we're calling "Sailor Blue" - #0096dc). And the checkmark should move to be on that focused category.

Tablet/Desktop:
http://f.cl.ly/items/2X1c060J1N2X292j0N2t/Screen%20Shot%202013-04-10%20at%2011.08.09%20AM.png Interactions are not completed; use your best judgement

Comment 1

[Matt Basta [:basta]]

The contents of the category dropdown will be provided by the API and cached in the model cache. This might be best done outside of the page wrapper and fetched with raw JS rather than via the builder.

Comment 2

[Matt Basta [:basta]]

To elaborate: the page should end up looking like this (pardon my pseudocode):

body
 header
  ...
 menu id="category-dropdown"
  ol
   li
    a class="category" href="{{ url('category', [category_slug]) }}"
      data-params={# whatever we put in here now on the homepage #}
     {{ category_title }}
 div id="page"
  ...
 ...

I'd recommend putting the category dropdown in its own template, just like the header and footer.

The only thing that should be hard-coded into the category list is "All Categories", since it's the default when you first start the Marketplace. All of the other categories should be requests.get()'d in marketplace.js.

We should re-requests.get() and re-render when reload_chrome is triggered (don't worry, it'll be cached) so that when a user's region changes, we'll show them the right operator shelf(s).

We'll also need to presumably detect (somehow) when the page loads and hide the dropdown on non-homepage and non-category pages. I.e.: when you search, there should be no category dropdown. Additionally, we should detect when you've changed categories (on loaded) and update the dropdown accordingly.

Comment 3

[[e-mail]]

https://github.com/mozilla/fireplace/commit/8be5109b77deb444021b09a55f26640bf1de553c

Comment 4

[Victor Carciu]

Verified as fixed in both Mobile and Desktop versions.