-
Notifications
You must be signed in to change notification settings - Fork 2.6k
Replace new tab pages for Private Windows with and without Tor #1507
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
Comments
Is there a mockup of what this task is that we can put in here? |
I suspect we need @cezaraugusto's help here if we're to make this task in time (he might already be working on it, not sure). |
Fairly sure this isn't making it in by the freeze tomorrow morning, so moving to the 1.0 project. |
Didn't realize there was activity in this, no problem moving back if we make the cut. |
I think this is still on track to make it. @cezaraugusto has a better handle on status but we're on it and progress seems positive. |
QA: note the following changes from the designs in this issue:
|
Verification Passed on
Went through verification using the following build under
Verification Passed on
|
@tomlowenthal as @srirambv mentioned above, the mockup from #1507 (comment) doesn't display the |
Nice catch on the |
Yeah that was an intentional omission I made on the fly cause the screens were needed so quickly. Too many big learn more buttons all over the page and in Windows with Tor it would have made it two big purple buttons going to the same article.. The Private Window learn more button was adjusted to a link inline with the copy as well instead of a big button (we could probably follow suit in Tor windows but I'd like to sit with it for a bit). |
Logged follow up issue #1667 |
Test plan
Verify the screens roughly match the mock-ups (below)
@tomlowenthal can help with verifying text
Description
In which @rossmoody is making the designs, @tomlowenthal is writing the text, with @riastradh-brave to implement.
The text was updated successfully, but these errors were encountered: