[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #26409 [Applications/Tor Browser]: Language prompt is shown twice at first start in Tor Browser based on ESR60
#26409: Language prompt is shown twice at first start in Tor Browser based on ESR60
-------------------------------------------------+-------------------------
Reporter: gk | Owner: tbb-
| team
Type: defect | Status:
| needs_review
Priority: High | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: ff60-esr, tbb-torbutton, | Actual Points:
AffectsTails, TorBrowserTeam201808R |
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------------------------+-------------------------
Comment (by intrigeri):
I've tested this in a running Tails system with 8.0a9 (by applying your
patches inside omni.ja and the torbutton xpi) and it worked just fine.
Thanks!
Code review passes modulo a couple nitpicks:
* `website/design/design.xml` and `website/design/index.html.en` still
refer to the obsolete `extensions.torbutton.spoof_english`; this probably
does not block merging these branches though since even without this fix,
the corresponding sections were obsolete already (they don't mention the
new Firefox locale spoofing feature that we now rely upon)
* the localized `torbutton.properties` files still have
`torbutton.popup.prompted_language`; maybe they're updated automatically
later during the build or release process?
I'm not a Torbutton developer so even though I'm quite at ease with this
part of the code, I'm leaving this is ''needs_review''. Let me know if I
should have been bold enough to make this ''merge_ready''.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/26409#comment:9>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs