Post-YE campaign clean up

After the YE fundraising campaign we need to remove the banner from Tor Browser and figure out what to have instead.

The proposal so far is to remove the banner altogether and add a link to

e.g https://trac.torproject.org/projects/tor/attachment/ticket/27097/27097_4_screenshot.png

added GeorgKoppen201901 in Legacy / Trac TorBrowserTeam201901R in Legacy / Trac component::applications/tor browser in Legacy / Trac owner::tbb-team in Legacy / Trac priority::high in Legacy / Trac resolution::fixed in Legacy / Trac severity::minor in Legacy / Trac status::closed in Legacy / Trac type::task in Legacy / Trac labels

Will need to be localised also

Trac:
Cc: sstevenson, antonela, steph, gk to sstevenson, antonela, steph, gk, emmapeel

Trac:
Keywords: N/A deleted, TorBrowserTeam201901 added

I made a version with an icon (A) and a version without (B)

(A) https://marvelapp.com/a131e34/screen/52273353 (B) https://marvelapp.com/a131e34/screen/52273349

I prefer the non-icon one, but I see how the icon highlight it a bit more.

Assets are ready to be grabbed on /handoff and there is a PR from Arthur here https://github.com/arthuredelstein/torbutton/commit/27097+4

Trac:
Keywords: TorBrowserTeam201901 deleted, N/A added
Cc: sstevenson, antonela, steph, gk, emmapeel to sstevenson, antonela, steph, gk

Replying to antonela:

I made a version with an icon (A) and a version without (B)

(A) https://marvelapp.com/a131e34/screen/52273353 (B) https://marvelapp.com/a131e34/screen/52273349

I prefer the non-icon one, but I see how the icon highlight it a bit more.

Assets are ready to be grabbed on /handoff and there is a PR from Arthur here https://github.com/arthuredelstein/torbutton/commit/27097+4

That's for the old newsletter sign-up work. Did you mean to add a new pull request? Or did you mean we should essentially use the code there and just replace icons? Or...?

Trac:
Keywords: N/A deleted, TorBrowserTeam201901 added

Replying to gk:

That's for the old newsletter sign-up work. Did you mean to add a new pull request? Or did you mean we should essentially use the code there and just replace icons? Or...?

Yes, is the same. We can use that code and if we want the icon, just add the icon.

I prefer the icon because it makes it stand out more.

What do we plan to do on mobile? There is the donation banner available as well currently.

Isa would like to do the same thing on mobile - no banner, but have the newsletter sign up on the about:tor page.

Trac:
Keywords: N/A deleted, GeorgKoppen201901 added

Trac:

Trac:

Mobile


Desktop

The signup part looks good! We just talked this over in the comms meeting, and both versions should match. Can we also add the manual and the mission lines to TBA? Isa suggested making it responsive so it's easy to maintain.

I pushed bug_29035_v2 (https://gitweb.torproject.org/user/gk/torbutton.git/log/?h=bug_29035_v2) which contains a) the reverts of all the previous donations and the newsletter signup related commits and b) I added the signup link (the text is not vertically aligned, though, we which seems to be okay).

mcs/brade: I plan to group the revert commits by feature when squashing the branch but I thought having them as they are right now might make reviewing easier. Could you have a look at the desktop version already while I am working on the mobile one? (I need to adapt the stable maint-branch, too, which needs some time, hence the early review request).

Trac:
Keywords: TorBrowserTeam201901 deleted, TorBrowserTeam201901R added
Status: new to needs_review
Cc: sstevenson, antonela, steph, gk to sstevenson, antonela, steph, gk, mcs, brade

This was a little challenging to review, but Kathy and I think you got it right. Our only comment is that you forgot to add the new image (icon-newsletter.png).

Replying to mcs:

This was a little challenging to review, but Kathy and I think you got it right. Our only comment is that you forgot to add the new image (icon-newsletter.png).

Oops, fixed. Sorry for the hassle, I'll move forward regrouping the revert-commits and clean things up and then work on the Android part, thanks.

Trac:
Keywords: TorBrowserTeam201901R deleted, TorBrowserTeam201901 added
Status: needs_review to new

Replying to steph:

The signup part looks good! We just talked this over in the comms meeting, and both versions should match. Can we also add the manual and the mission lines to TBA? Isa suggested making it responsive so it's easy to maintain.

In principle, yes, that should be doable. We should think about how those additional lines should look like. If you look at the mobile part in comment:10 you see that already the single line for the news sign-up is broken into three lines on mobile. I doubt we want to clutter the whole mobile screen with all of the text from the manual, newsletter sign-up, and of project mission.

bug_29035_v3 has the patches for review. There are essentially three parts: 1) removing the donation banner related changes (commit 9b4c56f050a266bbcd04d6985ee46f9da612e0e7), 2) removing the newsletter sign-up banner changes (8d3c6f996c943399a1505646eee14ce81868c0c8), and 3) adding the code changes for the new sign-up message/link on desktop and mobile (commit e73ff049acb16d410083c9cad64da678aab30294).

I made it so that the whole sign-up related text and icon are on a single line on tablets, laptops, and desktops while on phones it has the three lines as outlined in comment:10. I am fine adjusting that as needed or making it more complex, like on a phone in landscape mode show the single line, too. I am not a web developer, so I don't have that strong opinions here.

A mobile bundle for testing can be found at:

https://people.torproject.org/~gk/testbuilds/tor-browser-tbb-nightly-android-armv7-multi-qa_29035.apk https://people.torproject.org/~gk/testbuilds/tor-browser-tbb-nightly-android-armv7-multi-qa_29035.apk.asc

steph: legacy/trac#28145 (moved) is the ticket for the remaining links on about:tor should be doable for the next release provided we are sure how this should look like.

Trac:
Status: new to needs_review
Keywords: TorBrowserTeam201901 deleted, TorBrowserTeam201901R added
Cc: sstevenson, antonela, steph, gk, mcs, brade to stevenson, antonela, steph, gk, mcs, brade, igt0