Skip to content

feat: tray and minimize [JAG-26]#20

Merged
jagrat7 merged 1 commit intojagrat7:mainfrom
RainAfterDark:feat/tray
Mar 20, 2026
Merged

feat: tray and minimize [JAG-26]#20
jagrat7 merged 1 commit intojagrat7:mainfrom
RainAfterDark:feat/tray

Conversation

@RainAfterDark
Copy link
Copy Markdown
Collaborator

Implemented 3 settings:

  • Enable system tray
  • Minimize on startup (must enable launch at startup and system tray)
  • Minimize on close (must enable system tray)

Minimized options are disabled and greyed out when those conditions aren't met. Let me know if this seems intuitive.

@jagrat7
Copy link
Copy Markdown
Owner

jagrat7 commented Mar 9, 2026

it would be easier to add you as a collaborator so you can work on your branches, do you want me to add you?

@RainAfterDark
Copy link
Copy Markdown
Collaborator Author

it would be easier to add you as a collaborator so you can work on your branches, do you want me to add you?

Do you prefer that over PR drafts? I wouldn't mind

@jagrat7 jagrat7 changed the title feat: tray and minimize feat: tray and minimize [JAG-26] Mar 17, 2026
@RainAfterDark RainAfterDark marked this pull request as ready for review March 19, 2026 07:54
@RainAfterDark
Copy link
Copy Markdown
Collaborator Author

It's been a little while, I think I mislabled this as a draft. I'd like it to be reviewed if you have the time. I implemented a simple tray according to Electron's docs, and I wanna know if you're okay with the two new added settings and their enable/disable logic.

@jagrat7 jagrat7 self-requested a review March 20, 2026 06:27
Copy link
Copy Markdown
Owner

@jagrat7 jagrat7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jagrat7 jagrat7 merged commit b63b2d2 into jagrat7:main Mar 20, 2026
@RainAfterDark RainAfterDark mentioned this pull request Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants