Opened 3 months ago

Closed 2 months ago

#33482 closed defect (fixed)

Update about:tor donate string

Reported by: antonela Owned by: mcs
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: ux-team, TorBrowserTeam202003R, l10n, tbb-9.5a10
Cc: tbb-team Actual Points: 0.1
Parent ID: Points: 0.5
Reviewer: acat Sponsor:

Description

Sarah suggested:
"On about:Tor instead of “Keep Tor Strong” how about something like, “Tor is free to use because of donations from people like you. Donate Now.”?" [i like it! -steph]

I attached a mockup.

Child Tickets

Attachments (1)

about_tor_95.png (858.2 KB) - added by antonela 3 months ago.

Download all attachments as: .zip

Change History (13)

Changed 3 months ago by antonela

Attachment: about_tor_95.png added

comment:1 Changed 3 months ago by sysrqb

Keywords: TorBrowserTeam202003 added
Status: assignedneeds_information

We have a lot on our plate, but this is a small/easy change. Should we update this "as soon as possible" or should we plan a specific release?

comment:2 Changed 3 months ago by sysrqb

Points: 0.5

comment:3 Changed 3 months ago by antonela

Keywords: l10n added

I think the next plausible release is good enough. We will need l10n for that new string. Adding l10n to give visibility to emmapeel.

comment:4 Changed 3 months ago by mcs

Cc: tbb-team added
Owner: changed from tbb-team to mcs
Status: needs_informationassigned

Kathy and I will take this ticket.

comment:5 Changed 3 months ago by mcs

Keywords: TorBrowserTeam202003R added; TorBrowserTeam202003 removed
Status: assignedneeds_review

Here is a patch for review:
https://gitweb.torproject.org/user/brade/torbutton.git/commit/?h=bug33482-01&id=01366d7b04b5e1d17ddd499f9ddcb24418a8dede

The other locales should get updated during the translation import process.

comment:6 Changed 3 months ago by emmapeel

looks good! bring it on! (i guess it will appear on /chrome/locale/en-US/aboutTor.dtd b/chrome/locale/en-US/aboutTor.dtd in the main repo)

comment:7 Changed 3 months ago by sstevenson

Looks good! Thanks for the quick movement on this!

comment:8 Changed 3 months ago by sysrqb

Reviewer: acat

comment:9 Changed 3 months ago by mcs

Actual Points: 0.1

comment:10 Changed 3 months ago by acat

The patch looks good to me.

comment:11 Changed 2 months ago by acat

Status: needs_reviewmerge_ready

comment:12 Changed 2 months ago by sysrqb

Keywords: tbb-9.5a10 added
Resolution: fixed
Status: merge_readyclosed

I merged this onto master with commit 248204701c25ed3250bd9b7ca827d2d0e41ad310. Thanks! It'll be in 9.5a10.

Note: See TracTickets for help on using tickets.