esitarski / CrossMgr

Cyclo Cross Management Application
MIT License
40 stars 20 forks source link

GUI improvements to Chip Reader Setup window #96

Closed kimble4 closed 1 year ago

kimble4 commented 1 year ago

As I've recently been training a new volunteer in how to use CrossMgr to time our races, I've seen some things in a new light...

Firstly, the configuration options at the top of the 'Chip Reader Setup' window are corrupted, at least in Windows 11: Screenshot of corrupted Chip Reader Setup window

I think it's been like this for a while, I just haven't noticed as there's been no reason to change the configuration.

Secondly, the RFID properties tab has a "Use RFID Reader" tickbox to enable the use of RFID during the race. However, in the 'Chip Reader Setup' window, this option is reproduced with the name "Accept RFID Reader Data During Race". As such, it's not immediately obvious that these are the same thing. It would be better if the naming was consistent.

Thirdly, in the 'Chip Reader Setup' window, the grey area to the right of "Accept RFID Reader Data During Race" is a clickable area that ticks and unticks the box. I have, on at least one occasion, clicked here in order to give the window focus, and inadvertently deactivated the RFID system before a race.

esitarski commented 1 year ago

Thanks for the comments. All your issues are fixed in the latest version.

  1. The JChipSetup dialog displays properly (wasn't my fault - another new wxPython bug).
  2. The text has been standardized to "Use RFID Reader" (the shorter text).
  3. As "Accept RFID Reader Data During Race" has been shortened to "Use RFID Reader", this should reduce the likelihood of clicking on it.

On Fri, Sep 23, 2022 at 12:15 PM Kim Wall @.***> wrote:

As I've recently been training a new volunteer in how to use CrossMgr to time our races, I've seen some things in a new light...

Firstly, the configuration options at the top of the 'Chip Reader Setup' window are corrupted: [image: Screenshot of corrupted Chip Reader Setup window] https://camo.githubusercontent.com/2c9b87cd63c5571979ddf39a41eea4418e6deef2842a50d2b3ce9a369391d777/68747470733a2f2f7777772e64756374696c65626973637569742e6e65742f67616c6c6572795f616c62756d732f424850432f53637265656e73686f745f323032325f30395f32335f3137303133322e706e67

I think it's been like this for a while, I just haven't noticed as there's been no reason to change the configuration.

Secondly, the RFID properties tab has a "Use RFID Reader" tickbox to enable the use of RFID during the race. However, in the 'Chip Reader Setup' window, this option is reproduced with the name "Accept RFID Reader Data During Race". As such, it's not immediately obvious that these are the same thing. It would be better if the naming was consistent.

Thirdly, in the 'Chip Reader Setup' window, the grey area to the right of "Accept RFID Reader Data During Race" is a clickable area that ticks and unticks the box. I have, on at least one occasion, clicked here in order to give the window focus, and inadvertently deactivated the RFID system before a race.

— Reply to this email directly, view it on GitHub https://github.com/esitarski/CrossMgr/issues/96, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABGXKKABWPBKHZY3RI5GULV7XJRRANCNFSM6AAAAAAQUD4M7I . You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Edward Sitarski

kimble4 commented 1 year ago

That all looks much better, and I can confirm that the grey area to the right of "Use RFID Reader" is no longer clickable.