builtbybel / ReadySunValley

Replacement for Microsoft PC Health Check app
https://www.builtbybel.com
MIT License
200 stars 21 forks source link

Improving Code and UI #22

Open Oleg-Chashko opened 2 years ago

Oleg-Chashko commented 2 years ago

Fixed the issue and tested the fix.

Belim commented 2 years ago

thanks for the suggestions 😃 can you take a look at the latest testing release https://github.com/builtbybel/ReadySunValley/issues/20

the left result panel has now an dynamic resize event. whats your reading? the min. form size has now also been reduced

Belim commented 2 years ago

I liked it, but there are questions: It is still desirable to do this.

  • 1.Correct the background:
125265142-d7831b00-e304-11eb-9669-decbe3dcb389

this should be fixed now. Check the feature again...

  • 2.Dynamic resize: - If you do dynamic resizing. Don't forget to remove the tooltip.
Screen Shot 2021-07-12 at 14 02 26

this panel has now autoresize feature. if the content does not fit in, it must be shortened.

  • 3.Why is the link clickable, but nothing happens when you click it? - If it's a prompt about what to do, it shouldn't be clickable.

absolutely right. i will add some explantations in upcoming versions here

Screen Shot 2021-07-12 at 14 11 37

Correct this, please: cusor -> cursor

😁 damn, I would overlook this for the next 177 years. thank you 😸

Oleg-Chashko commented 2 years ago
125613169-a7338480-1442-4823-865a-9fcd4d0521a7 125613205-a2a56b70-bc0e-4eaa-a1cb-450a5e8f2d7f 125685047-41d25e6d-966a-4923-b83b-599738a1afc3 125910243-437eda2f-d2e8-4d0a-a350-ee41f2e4350a 125910287-770ffe2f-5cb9-4e5a-a89e-0efd47ae675c 125910328-e9255530-91a3-41c3-bd73-9b9d686997e5
  1. The caption should be highlighted so that the user knows it is clickable. (I figured it out by accident myself, and that's because I was doing a localization for the app.)
  2. This window looks kind of creepy. :(
126083356-8041af14-d74a-418f-80a2-f844bec69940

Example:

2


Belim commented 2 years ago

the last test build have improved many things https://github.com/builtbybel/ReadySunValley/issues/20 😁

Belim commented 2 years ago

@Oleg-Chashko thanks, will look into this today

can you please attend to this https://github.com/builtbybel/ReadySunValley/issues/32 and give me feedback.

Belim commented 2 years ago
  1. If the user wants to see compatible Processors and Graphics cards, he clicks the button in your program, a window opens and in a clear form displays Processors and Graphics cards that are compatible with Windows 11.
  2. If the user wants to see the file diagnostics (rsv.txt), he clicks the button in your program, a window opens and in a clear form he sees more complete information about his system.
  3. All additional files (rsv.txt) and (supportedCPU.txt) are in the temp folder, where they belong in the Windows system.

P.S.: What do you say to that? And move the localization folders there as well? So they don't get under the user's feet?

I concur 💯 expect the localization files which could be tricky moving them to a separate dir satellite assemblies should be where they are. this is how they work, finding the resource according to the directory name.

data files could be maybe located to localappdata dir in a seperate ReadySunValley dir!?

Belim commented 2 years ago
  • In the future, I recommend that you redraw the application icon. Right now, it's Windows 8 and Big Sur macOS.
  • And also, increase the size a little bit, currently the icon is smaller than the other icons.
Screen Shot 2021-07-16 at 12 45 02

we might need a new icon. ideas? .. im open to your suggestions

Belim commented 2 years ago

i thought we would choose something under the open source license. but this is even better 😁

wuhaaa, looking amazing 🥰 can you provide it as .ico file?

Belim commented 2 years ago

we got a winner 🥇 it will be presented with upcoming release and the oscar ...um, credits goes to @Oleg-Chashko 😁

many thanks once again

the tile could still be a little larger proportioned Image 5

Belim commented 2 years ago
  1. Where is the name of the program?

still working on pushing it on app store which is a fucking rocket science, if you ask me this was only a test with a small tile. the big one inlcuding name will come if it should work out

Belim commented 2 years ago

Are you going to keep the bypass function or remove it? (Bypassing TPM/Secure boot restriction

um, yes will be kept

Belim commented 2 years ago
  1. The caption should be highlighted so that the user knows it is clickable. (I figured it out by accident myself, and that's because I was doing a localization for the app.)

great 💯 which language?

  1. This window looks kind of creepy. :(
Screen Shot 2021-07-18 at 23 54 09

i like to spread fear and panic 😁

Belim commented 2 years ago

Are you going to keep the bypass function or remove it? (Bypassing TPM/Secure boot restriction

um, yes

Then the question is, in the localization file you yourself will delete the code associated with the "bypass".

feature will be kept. had not completed my answer above

Belim commented 2 years ago

um, you 've closed it 😆 🤣 fjybwbGpmZ

Belim commented 2 years ago

In general, who puts these buttons next to each other? 🤬

Screen Shot 2021-07-19 at 19 58 32

the way i remember it was billy - the boy - gates 🤔

Belim commented 2 years ago

I took care of something new again 😬 https://github.com/builtbybel/ThisIsWin11

i will deal with this next 😠 appstore

after that coffee and then everything upstairs 😆

Belim commented 2 years ago

slightly great effort 🤔 btw, can we continue in a new thread? this one transcends all limits ... and i can't scroll down that far anymore. my wrist hurts 😭