Opened 5 months ago

Closed 2 months ago

Last modified 2 months ago

#26556 closed defect (fixed)

Tor Browser icon missing from .desktop

Reported by: jscott Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Minor Keywords: tbb-8.0-issues, tbb-regression, tbb-8.0.1-can, GeorgKoppen201809, TorBrowserTeam201809R
Cc: frheis2201, yaomtc, anon, enigmatic007 Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

I use the Tor Browser alpha on GNU/Linux having installed it with ./start-tor-browser.desktop --register-app long ago. The .desktop points to the icon being in Browser/browser/icons/mozicon128.png, but the Browser/browser/icons/ directory doesn't seem to exist anymore with the upgrade to 8.0a9.

mozicon128.png can't be found anywhere in the bundle at all, and this keeps the launcher from showing the icon after the upgrade.

Child Tickets

Change History (13)

comment:1 Changed 5 months ago by cypherpunks

I can reproduce this on an LXDE desktop.

comment:2 Changed 5 months ago by gk

Component: Applications/Tor bundles/installationApplications/Tor Browser

Moving to Tor Browser.

comment:3 Changed 2 months ago by gk

Keywords: tbb-8.0-issues tbb-regression added; ff60-esr removed

We got a report on the blog as well (https://blog.torproject.org/comment/276892#comment-276892).

Last edited 2 months ago by gk (previous) (diff)

comment:4 Changed 2 months ago by cypherpunks3

The icon is now in Browser/browser/chrome/icons/default/default128.png

comment:5 Changed 2 months ago by gk

Owner: changed from erinn to tbb-team
Status: newassigned

comment:6 Changed 2 months ago by gk

Keywords: tbb-8.0.1-can added

Marking for 8.0.1 can.

comment:7 Changed 2 months ago by gk

Cc: frheis2201 added

#27497 is a duplicate.

comment:8 Changed 2 months ago by gk

Cc: yaomtc added
Keywords: GeorgKoppen201809 added

#27676 is a duplicate.

comment:9 Changed 2 months ago by gk

Keywords: TorBrowserTeam201809R added
Status: assignedneeds_review

comment:10 Changed 2 months ago by mcs

r=mcs
The patch looks good to me and it seems to work.

comment:11 Changed 2 months ago by gk

Resolution: fixed
Status: needs_reviewclosed

Fixed on master (commit 5d99870ce97b06252c393c12f0c4d2bcf326d6c3) and maint-8.0 (commit eb6a0da7fc417f7f0b42c99a072e18056c8c40dc), thanks!

comment:12 Changed 2 months ago by gk

Cc: anon added

#27727 is a duplicate.

comment:13 Changed 2 months ago by gk

Cc: enigmatic007 added

#27779 is a duplicate.

Note: See TracTickets for help on using tickets.