[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 Heiko Tietze changed: What|Removed |Added Keywords|needsUXEval |difficultyBeginner, ||easyHack, skillCpp, topicUI CC|libreoffice-ux-advise@lists |heiko.tietze@documentfounda |.freedesktop.org|tion.org --- Comment #20 from Heiko Tietze --- Fair enough. Code pointer for easy hack: In sc/source/ui/view/cellsh3.cxx, method ScopedVclPtrpDlg(pFact->CreateScMetricInputDlg(pTabViewShell->GetFrameWeld(), "ColWidthDialog", nCurHeight, STD_COL_WIDTH, eMetric, 2, MAX_COL_WIDTH)); change the value 2 after eMetric to 4. For consistency it would be good to do the same at ScopedVclPtrpDlg(pFact->CreateScMetricInputDlg(pTabViewShell->GetFrameWeld(), "RowHeightDialog", nCurHeight, ScGlobal::nStdRowHeight, eMetric, 2, MAX_ROW_HEIGHT)); -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 Nathan Shearer changed: What|Removed |Added Component|Writer |Calc --- Comment #19 from Nathan Shearer --- I must have chosen Writer by mistake when I opened the bug. I encountered it in Calc when I tried to make a spreadsheet that divided a page into exactly 8 columns. I have to disagree on the argument that rounding values and therefore changing the users explicit input is preferable because it can help reduce UI clutter. The UI does not need to change at all since the text fields will scroll as they do right now. Imagine if a Bank suggested that they should round your balance because it reduces clutter. I'm sure they would always round down too ;) -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 Heiko Tietze changed: What|Removed |Added CC||er...@redhat.com --- Comment #18 from Heiko Tietze --- (In reply to Nathan Shearer from comment #12) > Created attachment 169632 [details] > Column width rounding error when entering values This dialog is from Calc but the ticket is filed against Writer. This module shows five input fields for col widths and if we add decimal points it clutters heavily. I'd prefer a more precise rounding. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 --- Comment #17 from Nathan Shearer --- 1 Twip is defined as exactly 1/1440 of an inch. I would recommend removing the restriction on the UI altogether and allow unlimited decimals. Internally the software may round to the nearest twip since that seems to be a design limitation. Any other bugs caused by removing this restriction can also be discovered and tracked, resulting in an overall better software solution. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 --- Comment #16 from Regina Henschel --- (In reply to Heiko Tietze from comment #15) > (In reply to Regina Henschel from comment #14) > > I agree, that the input fields in the column size dialog should have more > > decimal places, so that it is possible to enter n/8 values. > > Would another decimal point in the UI solve the Twips problem? Not the Twips problem, but the original reported problem. Especially in an environment with US measurement system, you need at least 4 decimals. I personally see no need to restrict the column width input field at all. Currently I workaround the problem in using Millimeter as unit. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 Heiko Tietze changed: What|Removed |Added Severity|minor |enhancement --- Comment #15 from Heiko Tietze --- (In reply to Regina Henschel from comment #14) > I agree, that the input fields in the column size dialog should have more > decimal places, so that it is possible to enter n/8 values. Would another decimal point in the UI solve the Twips problem? -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
[Libreoffice-ux-advise] [Bug 101217] Setting the column width and height should not round the values causing compounding errors
https://bugs.documentfoundation.org/show_bug.cgi?id=101217 Regina Henschel changed: What|Removed |Added CC||libreoffice-ux-advise@lists ||.freedesktop.org, ||rb.hensc...@t-online.de Keywords||needsUXEval --- Comment #14 from Regina Henschel --- The 12mm to 11.99mm problem is not trivial. The internal unit is Twips as integer value. 1 Twips = 127/7200 mm and 1 mm = 7200/127 Twips. 12.00mm = 680.3149... Twips ≈ 680 Twips 680 Twips = 11.9944... mm ≈ 11.99 mm And on the other hand, for the next Twips value you get 681 Twips = 12.01208... mm ≈ 12.01 mm Twips is used, to get a better conversion with point: 1pt = 20 Twips. I agree, that the input fields in the column size dialog should have more decimal places, so that it is possible to enter n/8 values. -- You are receiving this mail because: You are on the CC list for the bug. ___ Libreoffice-ux-advise mailing list Libreoffice-ux-advise@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise
