View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001055||OpenMPT||User Interface||public||2017-11-08 21:28||2017-12-10 16:27|
|Reporter||blimey||Assigned To||Saga Musix|
|Product Version||OpenMPT 1.27.02.00 / libopenmpt 0.3.2/0.3.3 (upgrade first)|
|Target Version||OpenMPT 1.27.03.00 / libopenmpt 0.3.4 (upgrade first)||Fixed in Version||OpenMPT 1.27.03.00 / libopenmpt 0.3.4 (upgrade first)|
|Summary||0001055: Can't paste BPM info from Excel due to input validation|
Usability problem: pasting a number from Excel into the "Initial Tempo" or "Loop Tempo" fields is rejected due to input validation.
|Steps To Reproduce|
Figure out the BPM in Excel - for me that's figuring out the BPM based on the sample frequency and length of sample
Can work around this by pasting the number into notepad and copying it to the clipboard buffer again, but it's inconvenient.
|Tags||No tags attached.|
|Has the bug occurred in previous versions?||Yes|
|Tested code revision (in case you know it)|
I'm afraid that newlines are generally not accepted by Windows numeric input fields. However, that should apply to any numeric input fields, not just tempo fields. Since tempo fields allow for fractional inputs, they have a different validation but its newline rejection is just identical to normal integer inputs.
If the newline could be rejected by trimming off whitespace, rather than completely rejecting the input that would be better IMHO.
Well as I said, any kind of numeric input field in Windows will reject newlines, and if we wanted to work around that, we would have to change the behaviour of every single numeric input field in OpenMPT. Maybe it's actually not the newline but some invisible character close to the newline that is causing the rejection, though?
Okay, I found that while newlines at the start of a pasted text are not acceptable in generic number edit fields, they are accepted at the end, which the decimal input field for tempo and various other things didn't like.
|2017-11-08 21:28||blimey||New Issue|
|2017-11-08 23:28||Saga Musix||Note Added: 0003322|
|2017-11-11 18:28||geoffswift||Note Added: 0003341|
|2017-11-11 18:38||Saga Musix||Note Added: 0003344|
|2017-11-18 18:07||Saga Musix||Assigned To||=> Saga Musix|
|2017-11-18 18:07||Saga Musix||Status||new => assigned|
|2017-11-18 18:08||Saga Musix||Target Version||=> OpenMPT 1.27.03.00 / libopenmpt 0.3.4 (upgrade first)|
|2017-11-18 18:08||Saga Musix||Note Added: 0003353|
|2017-11-18 18:11||Saga Musix||Status||assigned => feedback|
|2017-12-10 16:27||Saga Musix||Status||feedback => resolved|
|2017-12-10 16:27||Saga Musix||Resolution||open => fixed|
|2017-12-10 16:27||Saga Musix||Fixed in Version||=> OpenMPT 1.27.03.00 / libopenmpt 0.3.4 (upgrade first)|