Nitrokey / nitrokey-update-tool

A GUI tool for Windows and macOS for updating Nitrokey Storage firmware
https://www.nitrokey.com/
GNU General Public License v3.0
8 stars 0 forks source link

Inform about enabling the update mode on the device #1

Closed szszszsz closed 6 years ago

szszszsz commented 6 years ago

Show this message on tool's start:

This tool allows to update Nitrokey Storage's firmware. Please select
<I>Configure -> Enable Firmware Update</I> in the Nitrokey App [2] and
download the .hex file of the latest firmware version[1].

[1] https://github.com/Nitrokey/nitrokey-storage-firmware/releases/latest
[2] https://www.nitrokey.com/download
szszszsz commented 6 years ago

Fixed with https://github.com/Nitrokey/nitrokey-update-tool/commit/e404bf232c40e65f228389b6c27e1a37fb95400c. Current screen shot: screenshot from 2018-06-12 19-33-55

szszszsz commented 6 years ago

Additionally tool's closing is not possible while the update procedure is running.

jans23 commented 6 years ago
  1. I tried the 0.4 release under Windows and the instructions are not displayed. Here is how it looks for me: screenshot
  2. I suggest to remove the first four lines (NK Prod tool started... Source is available...) because it distracts users from reading the actual instructions (if displayed as desired)
  3. Please change the text "Device connected in update mode: false" to "Nitrokey Storage detected. Please enable firmware update mode in Nitrokey App first."
szszszsz commented 6 years ago

Sorry for not directly mentioning - #1 fix will be released in v0.5 (v0.4 is not containing it). I will apply the rest of the suggestions.

szszszsz commented 6 years ago

Current view (will be released in v0.5): screenshot from 2018-06-13 09-43-19

jans23 commented 6 years ago

Screenshot looks much better.

What does "see Links..." mean? I don't think it is clear to users where to look. Perhaps adding the link right there instead?

I suggest to replace bullet points with numbers (1, 2, 3, 4) to make it more clear that these are steps.

szszszsz commented 6 years ago

OK! Will be changed to numbers.

I have added a Links submenu to the application, whose items are opening the required URLs. I will add menu word to the description to indicate that better. If that would still not be obvious, I will add the URLs in the text as suggested.