Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Needs these fixes [from JayzTwoCentz video] #1393

Open
Ironfist69 opened this issue Jul 30, 2024 · 30 comments
Open

Needs these fixes [from JayzTwoCentz video] #1393

Ironfist69 opened this issue Jul 30, 2024 · 30 comments

Comments

@Ironfist69
Copy link

Youtube video link

@hcabel
Copy link
Contributor

hcabel commented Jul 30, 2024

Here is a list of what he requested in his video:

I've already started fixing some of those here.
Please let me know if you want to take some of them.

@dcmbrown
Copy link

Dark mode is already an issue listed in #116. Would assume it's low in priority.

@SirVingeTheSecond
Copy link

@hcabel I believe he also requested a feature to make a custom defined interval for the scale of a data plot.

@meenzen
Copy link

meenzen commented Jul 30, 2024

.NET 9 may include new APIs for easy dark mode implementation

See dotnet/winforms#7641

@hcabel
Copy link
Contributor

hcabel commented Jul 30, 2024

@hcabel I believe he also requested a feature to make a custom defined interval for the scale of a data plot.

@SirVingeTheSecond I cannot find it, send me the time code and I'll update the list.

@khopland
Copy link

khopland commented Jul 30, 2024

@hcabel I believe he also requested a feature to make a custom defined interval for the scale of a data plot.

@SirVingeTheSecond I cannot find it, send me the time code and I'll update the list.

I think the time code is 12:43 (https://youtu.be/sX2ThUWNXio?t=763), but that is about the Y axis scale

@Chaosmeister
Copy link

Chaosmeister commented Jul 30, 2024

increase first start width / increase/resize column width - at 4:38

@tryptichon
Copy link

tryptichon commented Jul 30, 2024

I would suggest to break those bullet points from Jay down into single issues and backlink these here.

Assuming, that I am able to compile this project at my place I could try to tackle "Do not close dropdowns when checking/unchecking options". This can easily be handled by a small dialog instead of the menu structure. Menus always close when you click an item within.

@xmikedanielsx
Copy link
Contributor

@Ironfist69 @hcabel @tryptichon

Do not close dropdowns when checking/unchecking options
Pull Request (1395) Prevent Menu From Closing When Turning On or Off Sensors to Display

Created

@tryptichon
Copy link

tryptichon commented Jul 30, 2024

"Do not close dropdowns when checking/unchecking options":
Wow,, that was fast :-D. Is anybody still interested in a solution that uses a dialog instead of the menu?

@xmikedanielsx
Copy link
Contributor

@tryptichon
I plan on Redoing the menu entirely in another PR as It needs some love :) It will also allow for a lot of flexibility in options. Like log Location etc. Having a whole separate Options Screen with inputs that track to Reg or JSON file locally. Allowing user to maintain settings over time.

Also, Maybe we should turn this into a 1 Click Application or if someone has a license and cert sign the application due to the traction it's getting from famous YouTuber (Just a thought).

@tryptichon
Copy link

tryptichon commented Jul 30, 2024

@xmikedanielsx
Okay. A complete options screen would make a simple "Select Hardware" dialog obsolete, since this should then be part of the options. :) And it's the better solution IMHO anyway.

@SocietiDog
Copy link

SocietiDog commented Jul 30, 2024

Some of this stuff incl dark mode could easily be sorted using this open source github/NuGet packed. https:/MaterialDesignInXAML/MaterialDesignInXamlToolkit . I'm using it for 2 of my apps, and it works very well and very easy to set up for an existing .net project. And also in Visual studio, maybe make it into an .msi installer with they deployment project and I do this without any cert also. windows will just promt ppl with a warning.

@xmikedanielsx
Copy link
Contributor

@xmikedanielsx Okay. A complete options screen would make a simple "Select Hardware" dialog obsolete, since this part should then be part of the options. :) And it's the better solution IMHO anyway.

Yup, AGILE My Friend Agile! .. haha. Quick Fix until Good Fix :)

@xmikedanielsx
Copy link
Contributor

Some of this stuff incl dark mode could easily be sorted using this open source github/NuGet packed. https:/MaterialDesignInXAML/MaterialDesignInXamlToolkit . I'm using it for 2 of my apps, and it works very well and very easy to set up for an existing .net project.

If standard components were used for the tree view maybe. But it's nested in a bunch of custom components (aga Components). While possible It's a lot of work. Dark Mode is going to suck to do properly. UI will need to be torn apart it feels like.. I already started to take a stab at it and doing it a quick fix route. But it's trouble. Object. AllNodes Object.Nodes etc.. I am concentrating on other things atm so won't be for a little bit until I can focus on this again. But if you want to take it on by all means go for it.

As for the msi installer or ClickOnce application by all means go for it. I mentioned it above as well. Keep in mind, there is no true need for it to be clickonce unless it's going to be maintained and the updates are published on web outside of GitHub for purposes of auto update. If not people will have to keep uninstalling and installing. But if we have anyone that wants to volunteer webspace for it. We can publish all of that to the webspace and enable auto update for the people that want it. I think the current format (after rethinking) makes sense to just be a runtime app.

@tryptichon
Copy link

@xmikedanielsx
I would advise against the ClickOnce-methodology from Microsoft. ClickOnce-Applications are heavily coupled with their backend and might behave funky if they're used without a network - which might be a valid use case for this kind of software. Also maintaining ClickOnce-applications is a hassle.

The way I see it we can try to go one of two other routes:

  1. msi-installer, and the main program checks automatically for updates at the start. That's the way many other projects do it, i.e. HeidiSQL, KeePass and many more.
  2. The "evil" solution ;) Make the software available from the Windows Store. That's the way KODI does it and would have the advantage of updates being handled automatically every time a release is created. Although I do not have any experience with that route - maybe there are other hurdles I do not see here.

@SocietiDog
Copy link

Some of this stuff incl dark mode could easily be sorted using this open source github/NuGet packed. https:/MaterialDesignInXAML/MaterialDesignInXamlToolkit . I'm using it for 2 of my apps, and it works very well and very easy to set up for an existing .net project.

If standard components were used for the tree view maybe. But it's nested in a bunch of custom components (aga Components). While possible It's a lot of work. Dark Mode is going to suck to do properly. UI will need to be torn apart it feels like.. I already started to take a stab at it and doing it a quick fix route. But it's trouble. Object. AllNodes Object.Nodes etc.. I am concentrating on other things atm so won't be for a little bit until I can focus on this again. But if you want to take it on by all means go for it.

As for the msi installer or ClickOnce application by all means go for it. I mentioned it above as well. Keep in mind, there is no true need for it to be clickonce unless it's going to be maintained and the updates are published on web outside of GitHub for purposes of auto update. If not people will have to keep uninstalling and installing. But if we have anyone that wants to volunteer webspace for it. We can publish all of that to the webspace and enable auto update for the people that want it. I think the current format (after rethinking) makes sense to just be a runtime app.

I might take a swing at it. atm im just busy updating and adding a new addition to one of my apps. About the msi install I can see if no website ect, that might not the trouble worth. But my app dont have auto update. ppl just run a new version msi installer and it updates the already installed one. But to say I'm just a self learned C# hobby coder having some fun. I already downloaded the source to look at. If I get somewhere I will def hit you up in here. Great work btw. and Happy coding

@xmikedanielsx
Copy link
Contributor

@xmikedanielsx I would advise against the ClickOnce-methodology from Microsoft. ClickOnce-Applications are heavily coupled with their backend and might behave funky if they're used without a network - which might be a valid use case for this kind of software. Also maintaining ClickOnce-applications is a hassle.

The way I see it we can try to go one of two other routes:

  1. msi-installer, and the main program checks automatically for updates at the start. That's the way many other projects do it, i.e. HeidiSQL, KeePass and many more.
  2. The "evil" solution ;) Make the software available from the Windows Store. That's the way KODI does it and would have the advantage of updates being handled automatically every time a release is created. Although I do not have any experience with that route - maybe there are other hurdles I do not see here.

Yeah as I thought about it more. It makes more sense to be a runtime app or....... as you said (not me .. haha). Microsoft Store.

@SocietiDog
Copy link

Sorry was too fast to just use this NuGet packed. I thought the app was made in WPF and not windows forms.

@tryptichon
Copy link

@xmikedanielsx Okay. A complete options screen would make a simple "Select Hardware" dialog obsolete, since this part should then be part of the options. :) And it's the better solution IMHO anyway.

Yup, AGILE My Friend Agile! .. haha. Quick Fix until Good Fix :)

Well then ... throwing my "Quick Fix" into the ring. #1400

@xmikedanielsx
Copy link
Contributor

@tryptichon .. Well done. Looked at it. Not my favorite implementation as it is completely different from the rest of the menu and how it functions. But it does indeed solve his issue :)

@tryptichon
Copy link

@tryptichon .. Well done. Looked at it. Not my favorite implementation as it is completely different from the rest of the menu and how it functions. But it does indeed solve his issue :)

First steps towards the "Options Screen" you mentioned before. ;)

@tryptichon
Copy link

tryptichon commented Jul 30, 2024

@xmikedanielsx
I should add a keyboard shortcut for this menuitem: "Ctrl-J" ... just for Jay.

.... aaaaand done. 😁

@xmikedanielsx
Copy link
Contributor

@PhyxionNL - Quite a few outstanding PRs. Need help for QC or QA let the community know we can provide support. Let's keep the project moving ! . Go Devs Go! :)

@Tellinq
Copy link
Contributor

Tellinq commented Jul 31, 2024

Here is a list of what he requested in his video:

  • Dark mode (even simple one)
  • Do not close dropdowns when checking/unchecking options
  • Shift key to select/unselect multiple sensor
  • Record indicator when logging to csv file
  • Only log out checked sensor
  • One csv file per record session (instead of per day)
  • Change plot scale (y axis)
  • Auto resize sensor column width

I've already started fixing some of those here. Please let me know if you want to take some of them.

Little tip: If you wanted to keep your checklist up to date, instead of manually writing off the list, reference the issue (or better the applicable PR if present). If GitHub tracks this correctly, once closed the checkbox will be automatically ticked.

@Dumnersm580
Copy link

i can take on dark mode

@Tellinq
Copy link
Contributor

Tellinq commented Jul 31, 2024

i can take on dark mode

Looks like all that's left is just verifying that Dark Mode isn't in a "broken" state via PR #847. Currently I believe what's blocking the PR from being merged is this issue here: #847 (comment) - although I'm unable to replicate this... for now.

What we really need for traction is just QA testing this PR for any Dark Mode requests and try to break it if possible or further suggested improvements if any arise.

@hcabel
Copy link
Contributor

hcabel commented Jul 31, 2024

Here is a list of what he requested in his video:

  • Dark mode (even simple one)
  • Do not close dropdowns when checking/unchecking options
  • Shift key to select/unselect multiple sensor
  • Record indicator when logging to csv file
  • Only log out checked sensor
  • One csv file per record session (instead of per day)
  • Change plot scale (y axis)
  • Auto resize sensor column width

I've already started fixing some of those here. Please let me know if you want to take some of them.

Little tip: If you wanted to keep your checklist up to date, instead of manually writing off the list, reference the issue (or better the applicable PR if present). If GitHub tracks this correctly, once closed the checkbox will be automatically ticked.

Thanks for the tips, I didn't knew.
There is an urgent need for cleaning the issue on this repo, you've pointed out some duplicate, and I reckon properly tagging them is a must.
As a contributor do you have any right to do any of that? I would be happy to help but it's the first time I contribute to this repo 😅

@Tellinq
Copy link
Contributor

Tellinq commented Jul 31, 2024

Here is a list of what he requested in his video:

  • Dark mode (even simple one)
  • Do not close dropdowns when checking/unchecking options
  • Shift key to select/unselect multiple sensor
  • Record indicator when logging to csv file
  • Only log out checked sensor
  • One csv file per record session (instead of per day)
  • Change plot scale (y axis)
  • Auto resize sensor column width

I've already started fixing some of those here. Please let me know if you want to take some of them.

Little tip: If you wanted to keep your checklist up to date, instead of manually writing off the list, reference the issue (or better the applicable PR if present). If GitHub tracks this correctly, once closed the checkbox will be automatically ticked.

Thanks for the tips, I didn't knew. There is an urgent need for cleaning the issue on this repo, you've pointed out some duplicate, and I reckon properly tagging them is a must. As a contributor do you have any right to do any of that? I would be happy to help but it's the first time I contribute to this repo 😅

Since I'm merely someone who'll contribute via PRs, I don't have maintaining rights. However, I do agree that some tagging could be a bit useful. Via labels could be some use.

That being said, duplicates are bound to happen on any platform - it's a necessary evil that is impossible to avoid. Either by users desperately wanting something that they want to just write out what they want before utilizing the searching skills that they might have to search for duplicates first.

Usually when I maintain repositories (at least recent ones), one of the first few requirements I make users fill out is "What keywords have you used for potential duplicates?" - That way, if the user hasn't done that yet, they get that opportunity to search first, and then either upvote what they want, or offer further suggestions.

At the end of the day, end users who want to search for what's been already asked for will be through terminology that they are familiar with.

@hcabel
Copy link
Contributor

hcabel commented Jul 31, 2024

Here is a list of what he requested in his video:

  • Dark mode (even simple one)
  • Do not close dropdowns when checking/unchecking options
  • Shift key to select/unselect multiple sensor
  • Record indicator when logging to csv file
  • Only log out checked sensor
  • One csv file per record session (instead of per day)
  • Change plot scale (y axis)
  • Auto resize sensor column width

I've already started fixing some of those here. Please let me know if you want to take some of them.

Little tip: If you wanted to keep your checklist up to date, instead of manually writing off the list, reference the issue (or better the applicable PR if present). If GitHub tracks this correctly, once closed the checkbox will be automatically ticked.

Thanks for the tips, I didn't knew. There is an urgent need for cleaning the issue on this repo, you've pointed out some duplicate, and I reckon properly tagging them is a must. As a contributor do you have any right to do any of that? I would be happy to help but it's the first time I contribute to this repo 😅

Since I'm merely someone who'll contribute via PRs, I don't have maintaining rights. However, I do agree that some tagging could be a bit useful. Via labels could be some use.

That being said, duplicates are bound to happen on any platform - it's a necessary evil that is impossible to avoid. Either by users desperately wanting something that they want to just write out what they want before utilizing the searching skills that they might have to search for duplicates first.

Usually when I maintain repositories (at least recent ones), one of the first few requirements I make users fill out is "What keywords have you used for potential duplicates?" - That way, if the user hasn't done that yet, they get that opportunity to search first, and then either upvote what they want, or offer further suggestions.

At the end of the day, end users who want to search for what's been already asked for will be through terminology that they are familiar with.

100% agree with you.
What I meant was that it will be nice to have more people with some right to tag/label and even maybe close some issues.
So when a duplicates comes in, we can tag it duplicate, and close it with a message pointing to the original.
And just keep the issues clean or at least organized.

PhyxionNL added a commit that referenced this issue Aug 6, 2024
)

* Add CSV file rotation method

Add a new option to choose how the csv file is being rotated.
Before it was everyday, which meant that we had to make sure the csv file was deleted before recording a new session.
Now by default it's set to "Per session" meaning anytime you start a new session it create a new CSV file.
Unless the daily method is used, in this case it will behave like before (1 file per day)

* Scale up the first TreeView column ("Sensor") on resize

The first ("Sensor") column being the most important and most likely the one that need the more space as hardware names can be pretty long.
I've changed so that the first column is the one scaling up when the tree view is being resized (instead of the last one)

* Change treeview column resize behaviour

I've made so that the separator a controlling the size of the column to the left instead of the right.
This make a lot easier to scale up the last column, but worth for the first one.
Thought the previous commit made it so that the first column size is resize by the total with of the tree view.
Sounds weird like that, but feel much more natural than before, and there is no overflow anymore.

* Fix typo in daily file rotation menu item tool tip

Co-authored-by: PhyxionNL <[email protected]>

* Fix requested changes

---------

Co-authored-by: PhyxionNL <[email protected]>
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

No branches or pull requests