-
Notifications
You must be signed in to change notification settings - Fork 697
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[design refresh, rev2] Redesign footer, index page and language selector #4666
Conversation
(I'm basing this on the rev1 branch for now, which we'll hopefully merge into |
3b0761e
to
163113a
Compare
Codecov Report
@@ Coverage Diff @@
## logo-and-color-refresh-2019 #4666 +/- ##
==============================================================
Coverage ? 82.67%
==============================================================
Files ? 45
Lines ? 3122
Branches ? 338
==============================================================
Hits ? 2581
Misses ? 454
Partials ? 87 Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## develop #4666 +/- ##
========================================
Coverage 82.67% 82.67%
========================================
Files 45 45
Lines 3122 3122
Branches 338 338
========================================
Hits 2581 2581
Misses 454 454
Partials 87 87 Continue to review full report at Codecov.
|
cd30c12
to
064de4a
Compare
d25a7f9
to
a965c2a
Compare
This is technically ready for review if anyone wants to poke, but should be applied against |
a965c2a
to
13a44a5
Compare
13a44a5
to
9939eb9
Compare
Since nobody has reviewed yet and we're no longer in a huge rush, kicking this back into in development to combine with the index page rev2 (in reviewing that design, Nina suggested to alter some of the footer specs again; it's easiest to do that in the same PR). |
(Test failures are legit, functional tests make string checks that will need to be updated.) |
- Cleaner globe icon - Caret icon instead of glyph - Left-aligned menu items - Better align menu with logo - Mobile vs. desktop tweaks - Darker highlight color for accessibility
a27de5f
to
d9f333c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Main page design looks good on desktop and mobile Tor Browser (although the logo and language picker are displayed inconsistently (and awkwardly left-justified with a random HR) on subsequent pages).
- With the apparmor change in that last commit, the Source Interface renders each page with the footer in place
- The Journalist Interface displays correctly with no issues as a result of the SI change, and the footer isn't displayed on it.
As an aside, the footer language appears a bit stilted - "stewarded" isn't in common usage and it's more natural to refer to "the Freedom of the Press Foundation" rather than "Freedom of the Press Foundation" - the combination of the two gives the footer text a phishy vibe for me.
From a technical perspective, this is good to go. IMO the footer text could be reworded but that can happen separately.
That random HR in mobile looks like a recent design regression, will fix. The sizing of the language selector on subsequent pages should ideally be made consistent with the logo, but I view that as an enhancement that's OK to defer for now. (It's not entirely trivial given the different sizing specs used on index and subsequent pages.) |
Description
Update footer and index page to the second revision as specified for the SecureDrop 1.0.0 design refresh.
Resolves #4662
Resolves #4663
Status
Ready for review
Test plan
Why is the logo in the footer a bit different from the main logo?
We decided to use a monochromatic version of the logo in the footer to avoid too much repetition of the exact same image (favicon, potentially stock logo if not replaced, and footer). This monochromatic variant will be one of the approved variants post-logo refresh.
Screenshots
Desktop view
Mobile view