-
Notifications
You must be signed in to change notification settings - Fork 26
Update user interfaces to match designs #138
Comments
@jimporter the updated editor designs have been provided by @changecourse in Invision:
Let me know if/how you want to split this up, I'm happy to help where I can. |
I'm a bit curious about those icons. We have https://github.com/mozilla/tippy-top-sites, which we used for Activity Stream's favicons. Not sure if it'd be useful here. It's still just a random mix of "top sites" on the internet, so wouldn't work for lots of non en_US sites (or even Canadian versions of popular US sites). Also, would we need to build some sort of image proxy service, so we aren't loading site favicons directly and exposing any referer information? |
I think it's black and white as a way of indicating that it's a mockup. My plan is to store favicons in the datastore itself (but not anytime especially soon); this runs the risk of people having outdated icons, but it would also let people customize their icons, which I think would be pretty useful as a way of quickly disambiguating multiple accounts on one site |
Now that #154 is in place (thanks @jimporter) I will quickly walk @sandysage and @changecourse through where things are at and what, if anything, we should address before Alpha (versus after). 🚢 |
My guess would be that we want 1) strings, and 2) a little more polish on first-run. I don't think we'd have time to get first-run pixel-perfect, but I think we could make it look a little snazzier and get rid of the "Done!" page. I'll also write up a little thing with some more detail about why I think lock/unlock is the right language to use. |
Agreed! To help connect those dots...
I'll check in and see if we should flag anything including/beyond those two for release. |
I just met briefly with @sandysage and @changecourse and the plan is: Before Alpha
After Alpha(to be prioritized)
|
PR started at #168. Depending on how much I can get through this may need some extra assistance. |
Fonts, styles, spacing, labels, colors, etc.
To be provided...
The text was updated successfully, but these errors were encountered: