Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#2309 closed Enhancement (wontfix)

close to tray

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

Description


Attachments (1)

transmission-1.73-alt-close-to-tray.patch (2.2 KB) - added by zerg 14 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 14 years ago by charles

  • Milestone changed from 1.73 to None Set

How on earth can we milestone it for a release that's already out the door? :)

comment:2 Changed 14 years ago by charles

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

zerg: I appreciate your submitting this patch. By default I try to encourage anyone who's submitting good code, and your patch looks fine.

However, in this particular case I'm going to close this ticket as wontfix. When Transmission has had close-to-tray in the past, it's invariably confused some users and caused more complaints than it's worth. The "officially supported" way of closing to the tray is by clicking on the tray icon.

I hope this doesn't scare you off from filing other Qt enhancements and/or submitting patches for them or other tickets. Thanks again for reading the code and for trying to improve Transmission.

comment:3 Changed 14 years ago by zerg

it's invariably confused some users

I'm non't believe, because:

  1. I was seemed of transmission always crash before I apply this patch :-)
  1. I observe many unexpirience users which close windows "to tray" or "to minimize"
  1. GTK version closes to tray

I hope this doesn't scare you

No, I'm developer and this is familiar to me :-)

comment:4 Changed 14 years ago by charles

  1. GTK version closes to tray

No it doesn't. close to tray was *removed* from the GTK+ version for this reason.

Note: See TracTickets for help on using tickets.