LCD-Smartie / LCDSmartie

The New Official LCD Smartie Repository (LCD Smartie Lazarus port)
GNU General Public License v2.0
32 stars 5 forks source link

Crash on screen copy function #15

Closed limbo666 closed 1 year ago

limbo666 commented 1 year ago

I'm heavily Using the "Copy to" "Move to" and "Swap with" buttons to manage my screens. Lots of times I'm facing the crash report Untitled-1 Indicated as Exception message : Range check error report.txt After a short investigation I found that the problem is caused when the user is typing the number instead using the numerical UpDown control on the interface (bad habit). Suggestion: since these textboxes are accepting any character to be typed, it would be a good idea to prevent typing anything else than numeric values and update the variables when a number is typed on the field

stokie-ant commented 1 year ago

Oh, that was silly. I'm fetching the index of the value not he actual value I've changed them to spinners that can only accept numbers as it makes more sense anyway. you can still type directly if you wish

limbo666 commented 1 year ago

Thank you for taking care of this... BTW I have few suggestions to make for developments here and there on the program. I would be great if we have a communication channel to exchange ideas. Let me know what is your preference. We can use the forums or whatever you think is convenient

stokie-ant commented 1 year ago

Suggestions are very welcome, I'll contact you on the LCDSmartie forum. That's probably the safest way