Issue 124425

Summary: Program can’t decrease line width in toolbar from 0.01” to 0.00” using the decrease button. In the line width dialog box, this action can be executed.
Product: General Reporter: SimonSaysNoMore <thesimo65>
Component: uiAssignee: Armin Le Grand <Armin.Le.Grand>
Status: CLOSED FIXED QA Contact:
Severity: Minor    
Priority: P3 CC: Armin.Le.Grand, issues
Version: 3.3.0 or older (OOo)   
Target Milestone: 4.2.0   
Hardware: All   
OS: All   
Issue Type: DEFECT Latest Confirmation in: ---
Developer Difficulty: ---

Description SimonSaysNoMore 2014-03-14 05:30:27 UTC
When on dialog box for line width in menu bar, while your working scale is inches, you can increase and decrease the value with 0.02”. When you are decreasing and you are at line width value 0.01”, you can not decrease further (i.e. to 0.00”). When you select the line and do a mouse click you can open the line width dialog box (Line …) . In the line width dialog box you see the text field ‘Width’. If you click on the decrease button to decrease the value (i.e. 0.01” in this case), it jumps back to 0.00” (the lower boundary value). When you close the line width dialog box, the value of the line width displayed in the menu bar is set to 0.00” and the selected line is adjusted. 
I see this as an inconsistency bug.
The behavior of line width in menu bar must be consistent with behavior of line width in dialog box.
Comment 1 Oliver-Rainer Wittmann 2014-03-14 09:59:58 UTC
I can confirm the described behavior on Windows 7, too.
It is also reproducible in OOo 3.3.0, AOO 3.4.1 and AOO 4.0.0.
It is also reproducible with centimeter - e.g. 0,05cm
It is not restricted to Impress, the same behavior is observable in Writer, Draw and Calc - it is the same toolbar and same line dialog.
Comment 2 Armin Le Grand 2014-03-20 11:37:37 UTC
Cecking that - btw. there is now also the sidebar LineWidth control which allows to go back to zero, too. Indeed, the one in the 'Line and Filling' does not do the same. Getting some stuff with debug...
Comment 3 Armin Le Grand 2014-03-20 12:36:45 UTC
The problem is house-made in the SvxMetricField::Down() implementation. There is OS2-specific code which probably works around a UI error in OS2, but it is not #ifdef'ed for only OS2. The result is that reacing the min value is prohibited for all systems.
I have adapted this and also adapted the OS2 code (without being able to test it) to at least set the value to the min value and not just leave it unchanged when it falls below the minimum.
Comment 4 SVN Robot 2014-03-20 13:01:50 UTC
"alg" committed SVN revision 1579612 into trunk:
i124425 corrected SvxMetricField::Down() behaviour
Comment 5 Armin Le Grand 2014-03-20 13:02:29 UTC
Comitted, done.