Open
Bug 1235260
Opened 10 years ago
Updated 3 years ago
[gui] Mozregression incorrectly sets day if I paste date string after selecting text from end to beginning
Categories
(Testing :: mozregression, defect)
Testing
mozregression
Tracking
(Not tracked)
REOPENED
People
(Reporter: arni2033, Unassigned)
References
Details
>>> My Info: Win7_64, Mozregression GUI version 0.7.0 (also tested version 0.5.0)
STR:
1. Copy this string: "2009-01-01" in clipboard (without quotes)
2. Launch mozregression GUI, click "File" -> "Start new bisection", then click "Next" twice
3. Set date in the first text field (input?) to 2015-12-27
4. Make sure that the first text field is still focused
5.A) Press End key, then Shift+Home to select all text, then Ctrl+V to paste string from Step 1
5.B) Move mouse after the string but still within text field, hold left mouse button,
move mouse to the very start of the line, release left mouse button. Press Ctrl+V.
Result (A, B):
The year (2009) is set correctly, as well as the month (01). But the day in the text field
remains equal to previously set day (in my case, 27). So I get the date 2009-01-27
Expectations:
The date should be set correctly. In my case, 2009-01-01 instead of 2009-01-27.
Observation:
Looks like that happens for both inputs, only if pasted day is smaller than the day in input
Comment 1•10 years ago
|
||
I'm going to close this as wontfix since we will (in next release) ship a calendar instead of the date widgets(see bug 1231388).
Unfortunately I guess this mean that copy paste will be impossible now. If this is an issue, open a bug once the next release is available and we will try to find a workaround.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
"enhancements"... v_v
I'll just use GUI v.5 if I ever need a reliavle result with normal UX (even w/o background downloads)
Comment 3•10 years ago
|
||
(In reply to arni2033 from comment #2)
> "enhancements"... v_v
> I'll just use GUI v.5 if I ever need a reliavle result with normal UX (even
> w/o background downloads)
I'm sorry you feel that way.
The previous way of narrowing down a regression window was broken, because we always ended in mozilla-inbound, and this is not always the case. If you want to do that now, you still have the choice to choose the inbound branch and use dates to build the initial build range (you will be on inbound from the beginning). Even if it is not perfect now, I believe it is better. But as I said previously, I'm not against bringing that back as an optional choice, if that make sense and that is useful to someone.
For the calendar, I must say I wasn't much in favor of it. I feel like this is not convenient, I prefer to use the keyboard instead of having to click a lot. Having said that, I think we wanted this because the GUI was originally created for people that like this kind of easy things, otherwise they would use the command line mozregression. Well that was my point of view on this, I may be wrong. Maybe it is a bad idea to use this calendar, and we should not use it ? I'll discuss about that with the other contributors, see what they think.
Comment 4•10 years ago
|
||
Ok, let's bring this back!
We will bring back the date fields, I agree it is better than just calendars. We do that with bug 1246339, and this will allow calendars for those who still want it, but it won't change a thing for the current workflow (it is possible to click on the right of the date line edit to show the calendar, but otherwise it is not shown and the order of editing widgets is preserved).
arni2033, thanks for complaining about that. :) This was a bad decision, and I find it much better now.
So we can try to fix that bug now.
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•