Skip to content

Conversation

slimline33
Copy link

  • BalloonTip Management: The BalloonTip is displayed only once when the window is minimized.
  • Error Handling for Icon: If the icon file is not found, an error message is shown.
  • Cleaned-Up Layout Logic: Visibility and update logic for layouts have been extracted into separate methods.
  • SerialPort Communication: Error handling for serial commands has been added.
  • Minimization to System Tray: When minimized, the program hides itself and moves to the system tray (right side of the taskbar).

- BalloonTip Management: The BalloonTip is displayed only once when the window is minimized.
- Error Handling for Icon: If the icon file is not found, an error message is shown.
- Cleaned-Up Layout Logic: Visibility and update logic for layouts have been extracted into separate methods.
- SerialPort Communication: Error handling for serial commands has been added.
- Minimization to System Tray: When minimized, the program hides itself and moves to the system tray (right side of the taskbar).
code optimierung
update to net6.0, ignore warnings, only windows
error in duplicate dll
@CoretechR
Copy link
Owner

Hi @slimline33, first of all, thank you so much for these improvements! I think that they are a great update to the project. And the Arduino sketch is much more compact and readable now!

I'm very sorry that it took so long to review your pull request. I had to set up the Arduino and VS environments to get it to run.

About the Companion App:

  • The tray icon is not part of the project: Icon = new Icon("path-to-icon.ico"), // Pfad zu deinem Icon. Could you try to add it to the project resources?
  • I like that the window minimizes to the system tray. But wouln't it make sense to have the app icon visible in the task bar while the app is open? So this would not always be active: this.ShowInTaskbar = false;

And for the Arduino sketch:

  • I was unable to compile this without dynamically allocating the buttons. Meaning Button* buttons[9] instead of Button buttons[9] and so on. Did you use a different library?

Und könntest du vielleicht deine Kommentare auf Englisch übersetzen? Zum Beispiel mit DeepL ginge das fix. Falls mal jemand außerhalb von Deutschland an dem Projekt weiterarbeiten will, wäre das vorteilhaft ;)

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