Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

Added Inverted TrayIcon & Gui Refresh interval metric in GUI #9

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

Tuphal
Copy link

@Tuphal Tuphal commented Sep 5, 2019

I did a fast test. It works after a restart of the program.
Please verify

@dabear
Copy link
Owner

dabear commented Sep 5, 2019

I'll check it out later this month and create a release

@dabear
Copy link
Owner

dabear commented Sep 5, 2019

Some of the designer related files have comments in german. That must be converted to english

Translated auto-generated comment
Translated auto-generated comment
@Tuphal
Copy link
Author

Tuphal commented Sep 6, 2019

I updated the two files with the auto-generated comments.

Settings for usage of Background image or Color at start
Settings for usage of Background image or Color at start
Settings for usage of Background image or Color at start
Background Image usage priorise
@Tuphal
Copy link
Author

Tuphal commented Sep 11, 2019

I encountered the problem, that my selected Background Image was not used after the start.
On every start had to open the settings and click "verify and continue" to use the image.

I made 2 radio buttons to let the user choose a background color or image.
Depending on this setting the load-script chooses the right setting.

@Tuphal
Copy link
Author

Tuphal commented Sep 11, 2019

The change of the background was not a problem of the app, but of my batch file.

Explanation: I have a batch file for all autostart processes.
In these I call start "" "D:\Tools\FloatingGlucose\FloatingGlucose.exe"
The background image is in the same directory, thats why I used in the FloatingGlucose Settings ".\background.jpg" and not the full path.
By calling the app through the batch, the app can not find the background image (I can not explain why). I changed the batch call and then it works:
cd /D D:\Tools\FloatingGlucose\ & start "" FloatingGlucose.exe

Therefore my last change is not needed, but maybe for a more clarified chosen background usefull

@Tuphal
Copy link
Author

Tuphal commented Sep 11, 2019

Added option to start and set window in the background.
Default setting is set to be always on top.

@dabear
Copy link
Owner

dabear commented Sep 11, 2019

This Pull request is becoming too large in featureset. Please Split out background and alwaysontop in two separate PRs.

@Tuphal
Copy link
Author

Tuphal commented Sep 12, 2019

sorry. I made two separate PRs. How can I undo the changes in this PR?

@dabear
Copy link
Owner

dabear commented Oct 7, 2019

Gui Refresh interval label can stay. I want the app to always have the same trayicon. I understand that might be an inconvenience. If so, please help me find another royalty free icon that would fit most desktops (win 8, win9, win10++)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants