Skip to content
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

Replace new tab pages for Private Windows with and without Tor #1507

Closed
tildelowengrimm opened this issue Oct 11, 2018 · 12 comments
Closed

Replace new tab pages for Private Windows with and without Tor #1507

tildelowengrimm opened this issue Oct 11, 2018 · 12 comments

Comments

@tildelowengrimm
Copy link
Contributor

tildelowengrimm commented Oct 11, 2018

Test plan

Verify the screens roughly match the mock-ups (below)

  • Private window
  • Tor window

@tomlowenthal can help with verifying text

Description

In which @rossmoody is making the designs, @tomlowenthal is writing the text, with @riastradh-brave to implement.

@bbondy
Copy link
Member

bbondy commented Oct 12, 2018

Is there a mockup of what this task is that we can put in here?
Only a couple working days left until the freeze, thanks!

@rossmoody
Copy link
Contributor

rossmoody commented Oct 12, 2018

These are the latest designs for this initiative. They are good to go.

tor
private window
qwant layout

@bbondy
Copy link
Member

bbondy commented Oct 12, 2018

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).

@bbondy bbondy modified the milestones: 0.55.x, 1.x Backlog Oct 16, 2018
@bbondy
Copy link
Member

bbondy commented Oct 16, 2018

Fairly sure this isn't making it in by the freeze tomorrow morning, so moving to the 1.0 project.

@bbondy
Copy link
Member

bbondy commented Oct 16, 2018

Didn't realize there was activity in this, no problem moving back if we make the cut.

@rossmoody
Copy link
Contributor

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.

@bsclifton bsclifton modified the milestones: 1.x Backlog, 0.55.x Oct 16, 2018
@bsclifton bsclifton removed the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Oct 16, 2018
@tildelowengrimm
Copy link
Contributor Author

QA: note the following changes from the designs in this issue:

  • Final text is in this doc, not the mockups
  • Learn more overlay has been replaced with links to the knowledge base
  • For this revision, all browsers get the DDG version, there's no impl of the DDG-free one

@srirambv
Copy link
Contributor

srirambv commented Oct 17, 2018

Verification Passed on

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Linux
  • All support links to correct article on KB
  • Tor window More info is missing which is mentioned in the document but not implemented. cc: @tomlowenthal to confirm if this needs to be tracked in a new issue?

image

Went through verification using the following build under macOS 10.13.6 x64 - PASSED

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Mac OS X

Verification Passed on

Brave 0.55.16 Chromium: 70.0.3538.54 (Official Build) (64-bit)
Revision 4f8e578b6680574714e9ed3bb9f02922b4dde40d-refs/branch-heads/3538@{#937}
OS Windows
  • All support links to correct article on KB
  • Support links are not being loaded inline under Tor windows, @kjozwiak has created issue 1647

@kjozwiak
Copy link
Member

Tor window More info is missing which is mentioned in the document but not implemented. cc: @tomlowenthal to confirm if this needs to be tracked in a new issue?

@tomlowenthal as @srirambv mentioned above, the mockup from #1507 (comment) doesn't display the Learn more under the THIS IS A Private Window with Tor section but the https://docs.google.com/document/d/1-LvUwonWwXiFJvw1MjLt9fsCWktEiKiEM7cZ-UN6uPQ document mentions that there should be a [Learn more] underneath. Let us know which one is correct and we'll create a new issue if Learn more should have been there.

@tildelowengrimm
Copy link
Contributor Author

Nice catch on the learn more. Let's open a new issue to re-add that link for 56; I don't think it should block the 55 release since right now it's redundant with the text below. And there may be some other design tweaks for 56, so we can roll that right in.

@rossmoody
Copy link
Contributor

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).

@srirambv
Copy link
Contributor

Logged follow up issue #1667

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment