Opened 12 years ago

Closed 11 years ago

#2493 closed Enhancement (invalid)

Customize Icon

Reported by: zerg Owned by: charles
Priority: Normal Milestone: None Set
Component: Qt Client Version: 2.22
Severity: Normal Keywords:
Cc:

Description

Allow to customize window and system tray icon. It usable when more then one qtr instances running (see screenshot).

Attachments (3)

transmission-1.75-alt-custom-icon.patch (11.0 KB) - added by zerg 12 years ago.
transmission-custom-icon.png (44.5 KB) - added by zerg 12 years ago.
transmission-2.22-alt-custom-icon.patch (10.4 KB) - added by zerg 11 years ago.

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by zerg

Changed 12 years ago by zerg

comment:1 Changed 12 years ago by zerg

  • Component changed from Transmission to Qt Client
  • Owner set to charles

comment:2 Changed 12 years ago by charles

  • Resolution set to invalid
  • Status changed from new to closed

The patch is well-written, but it's 300+ new lines of code for a feature that few if any people will ever actually use, and it also breaks parity with the GTK+ client. I appreciate the patch but on balance think it's not something that belongs in the official releases.

Changed 11 years ago by zerg

comment:3 Changed 11 years ago by zerg

breaks parity with the GTK+ client

"Change Session" functionality breaks parity with the GTK+ client too. It's depend on it.

comment:4 Changed 11 years ago by zerg

  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Version changed from 1.75 to 2.22

comment:5 Changed 11 years ago by jordan

  • Resolution set to invalid
  • Status changed from reopened to closed

Ticket #2768 is already open for adding "change session" to the GTK+ client, but IMO the more important part is that this is 300+ lines of new code for a feature with an obscure use case.

In the case where you've got multiple copies of transmission-qt running and connected to different sessions, the titlebar already gives you the address + port of the connection.

Again, I appreciate the patch but don't think this feature makes sense.

Note: See TracTickets for help on using tickets.