[PR #111] [MERGED] [Linux] Add seperators to tray, add tray option to reopen app and settings, prevent duplicate instances of app from being opened #292

New issue

Closed

opened 2026-03-02 12:40:29 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 12:40:29 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/kavishdevar/librepods/pull/111
Author: @jazzysoggy
Created: 4/24/2025
Status: ✅ Merged
Merged: 4/30/2025
Merged by: @kavishdevar

Base: main ← Head: main

📝 Commits (3)

a51efe3 [Linux] Add reopen to tray options, enhance app tray, add ability to detect duplicate app instances, prevent duplicate app instances, and allow for original instance to be brought to front using the sockets
6376240 Add environmental variable check for phone MAC Address
a8f87f3 Proper handling of direct kill

📊 Changes

4 files changed (+160 additions, -3 deletions)

View changed files

📝 linux/Main.qml (+24 -0)
📝 linux/main.cpp (+117 -3)
📝 linux/trayiconmanager.cpp (+17 -0)
📝 linux/trayiconmanager.h (+2 -0)

📄 Description

This pull request would add additional functionality and quality of life enhancements to the Linux application:

Allows for an existing app instance to be reopened from the app tray to the main app menu or settings menu
Add separators between different options in tray to improve clarity
Add detection for existing app instances and the ability for a duplicate instance to trigger a reopening of the original app instance instead of spinning up a new instance

This improves functionality for an end user by adding in expected responses to app interactions.

Edit:
Additional changes made since original pull request:

Add error handling for socket and server
Use environment variable instead of preprocessor to detect
Clear variable on original instance quitting
Proper handling in case connection refused for duplicate client

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/kavishdevar/librepods/pull/111 **Author:** [@jazzysoggy](https://github.com/jazzysoggy) **Created:** 4/24/2025 **Status:** ✅ Merged **Merged:** 4/30/2025 **Merged by:** [@kavishdevar](https://github.com/kavishdevar) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (3) - [`a51efe3`](https://github.com/kavishdevar/librepods/commit/a51efe35dc749963a349217566e4c8f0a1adab35) [Linux] Add reopen to tray options, enhance app tray, add ability to detect duplicate app instances, prevent duplicate app instances, and allow for original instance to be brought to front using the sockets - [`6376240`](https://github.com/kavishdevar/librepods/commit/6376240ce0ba825e54e819f6f17f6265bdd58d94) Add environmental variable check for phone MAC Address - [`a8f87f3`](https://github.com/kavishdevar/librepods/commit/a8f87f37f6c71751c17bf4c0d0fe214f475f9a05) Proper handling of direct kill ### 📊 Changes **4 files changed** (+160 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `linux/Main.qml` (+24 -0) 📝 `linux/main.cpp` (+117 -3) 📝 `linux/trayiconmanager.cpp` (+17 -0) 📝 `linux/trayiconmanager.h` (+2 -0) </details> ### 📄 Description This pull request would add additional functionality and quality of life enhancements to the Linux application: - Allows for an existing app instance to be reopened from the app tray to the main app menu or settings menu - Add separators between different options in tray to improve clarity - Add detection for existing app instances and the ability for a duplicate instance to trigger a reopening of the original app instance instead of spinning up a new instance This improves functionality for an end user by adding in expected responses to app interactions. Edit: Additional changes made since original pull request: - Add error handling for socket and server - Use environment variable instead of preprocessor to detect - Clear variable on original instance quitting - Proper handling in case connection refused for duplicate client --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>