transmission-remote-gui / transgui

🧲 A feature rich cross platform Transmission BitTorrent client. Faster and has more functionality than the built-in web GUI.
GNU General Public License v2.0
3.23k stars 280 forks source link

Made taskbar button follow main form monitor (Windows) #1427

Closed shole closed 1 year ago

shole commented 2 years ago

I have multiple monitors and by default transgui does not transfer taskbar button when used on a secondary monitor. There are also these two issues mentioning the problem. https://github.com/transmission-remote-gui/transgui/issues/1299 https://github.com/transmission-remote-gui/transgui/issues/1056

I resolved the issue with the simple suggestions in some of the linked forum articles in the issue threads. https://forum.lazarus.freepascal.org/index.php?topic=24926.0 This allows taskbar button to move onto other monitors. It's not perfect and sometimes I need to maximize and restore the window for the switch to happen, but at least now it's possible, while before it never happened.

On compilation I get the warning Warning: (5044) Symbol "MainFormOnTaskBar" is not portable. I don't know what the side effects of that are, so I wrapped the call in {$ifdef windows} just in case.

cr-gpt[bot] commented 1 year ago

Seems you are using me but didn't get OPENAI_API_KEY seted in Variables/Secrets for this repo. you could follow readme for more information

ghost commented 1 year ago
👇 Click on the image for a new way to code review #### [![Review these changes using an interactive CodeSee Map](https://s3.us-east-2.amazonaws.com/maps.codesee.io/images/github/transmission-remote-gui/transgui/1427/8f539d2e/89c4fa8882c3971e18ea314bec9d343bcda1fcfb.svg)](https://app.codesee.io/r/reviews?pr=1427&src=https%3A%2F%2Fgithub.com%2Ftransmission-remote-gui%2Ftransgui) #### Legend CodeSee Map legend