sc/qa/uitest/calc_tests/columns.py | 20 ++++++++++----------
sc/qa/uitest/calc_tests/rows.py | 12 ++++++------
sc/qa/uitest/calc_tests9/tdf126673.py | 2 +-
sc/source/ui/view/cellsh3.cxx | 8 ++------
4 files changed, 19 insertions(+), 23 deletions(-)
New commits:
commit f82f6a2714fbf7882eb1d77351574392ae8e4c27
Author: Heiko Tietze <tietze.he...@gmail.com>
AuthorDate: Fri Nov 26 11:30:26 2021 +0100
Commit: Heiko Tietze <heiko.tie...@documentfoundation.org>
CommitDate: Fri Nov 26 14:49:55 2021 +0100
Reverts tdf#144247 tdf#101217 - 4 digits in row height/col width
Reverts patch I29d90b413e6f6f6e8b73d30ac47d644a69d15046,
ad8edac43e73555bc2055514300c5b81a1bb04ea and partially
Ibdee348d84a8b96d8f698e7fc6db89eb24422f5d
Change-Id: Iba5fd2d2a65d813c2d5ba85e66a7c3e07b726ca5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125871
Tested-by: Jenkins
Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>
Reviewed-by: Heiko Tietze <heiko.tie...@documentfoundation.org>
diff --git a/sc/qa/uitest/calc_tests/columns.py
b/sc/qa/uitest/calc_tests/columns.py
index dafd825c76c8..19033f86ea7e 100644
--- a/sc/qa/uitest/calc_tests/columns.py
+++ b/sc/qa/uitest/calc_tests/columns.py
@@ -42,7 +42,7 @@ class CalcColumns(UITestCase):
#verify
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth",
close_button="cancel") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_column_width_two_columns(self):
@@ -67,12 +67,12 @@ class CalcColumns(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "C1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_column_width_copy(self):
@@ -103,7 +103,7 @@ class CalcColumns(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "C1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_column_hide_show(self):
@@ -195,32 +195,32 @@ class CalcColumns(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "C1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "D1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "2.0003
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "2.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "E1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "3.0004
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "3.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "F1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "G1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "2.0003
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "2.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "H1"}))
with
self.ui_test.execute_dialog_through_command(".uno:ColumnWidth") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "3.0004
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "3.00 cm")
# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/qa/uitest/calc_tests/rows.py b/sc/qa/uitest/calc_tests/rows.py
index 1e426623aaaa..4eeb9820ab5e 100644
--- a/sc/qa/uitest/calc_tests/rows.py
+++ b/sc/qa/uitest/calc_tests/rows.py
@@ -43,7 +43,7 @@ class CalcRows(UITestCase):
#verify
with self.ui_test.execute_dialog_through_command(".uno:RowHeight",
close_button="cancel") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_row_height_two_rows(self):
@@ -68,12 +68,12 @@ class CalcRows(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A1"}))
with self.ui_test.execute_dialog_through_command(".uno:RowHeight")
as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A3"}))
with self.ui_test.execute_dialog_through_command(".uno:RowHeight")
as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_tdf89140_row_height_copy(self):
@@ -105,7 +105,7 @@ class CalcRows(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A3"}))
with self.ui_test.execute_dialog_through_command(".uno:RowHeight")
as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
def test_row_hide_show(self):
@@ -180,12 +180,12 @@ class CalcRows(UITestCase):
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A3"}))
with self.ui_test.execute_dialog_through_command(".uno:RowHeight")
as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
gridwin.executeAction("SELECT", mkPropertyValues({"CELL": "A4"}))
with self.ui_test.execute_dialog_through_command(".uno:RowHeight")
as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.0001
cm")
+ self.assertEqual(get_state_as_dict(xvalue)["Text"], "1.00 cm")
# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/qa/uitest/calc_tests9/tdf126673.py
b/sc/qa/uitest/calc_tests9/tdf126673.py
index bcc2c49b69a0..e300942935e8 100644
--- a/sc/qa/uitest/calc_tests9/tdf126673.py
+++ b/sc/qa/uitest/calc_tests9/tdf126673.py
@@ -34,6 +34,6 @@ class tdf126673(UITestCase):
#row height
with self.ui_test.execute_dialog_through_command(".uno:RowHeight",
close_button="cancel") as xDialog:
xvalue = xDialog.getChild("value")
- self.assertEqual("1.2365 cm",
get_state_as_dict(xvalue)["Text"])
+ self.assertEqual("1.24 cm", get_state_as_dict(xvalue)["Text"])
# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/source/ui/view/cellsh3.cxx b/sc/source/ui/view/cellsh3.cxx
index 5f4275fd8572..f24c06e9a0d2 100644
--- a/sc/source/ui/view/cellsh3.cxx
+++ b/sc/source/ui/view/cellsh3.cxx
@@ -690,9 +690,7 @@ void ScCellShell::Execute( SfxRequest& rReq )
ScopedVclPtr<AbstractScMetricInputDlg>
pDlg(pFact->CreateScMetricInputDlg(
pTabViewShell->GetFrameWeld(), "RowHeightDialog",
nCurHeight, ScGlobal::nStdRowHeight,
- eMetric,
- nCurHeight == ScGlobal::nStdRowHeight ? 2 : 4, //use 4
digits for user-defined values
- MAX_ROW_HEIGHT));
+ eMetric, 2, MAX_ROW_HEIGHT));
if ( pDlg->Execute() == RET_OK )
{
@@ -790,9 +788,7 @@ void ScCellShell::Execute( SfxRequest& rReq )
ScAbstractDialogFactory* pFact =
ScAbstractDialogFactory::Create();
ScopedVclPtr<AbstractScMetricInputDlg>
pDlg(pFact->CreateScMetricInputDlg(
pTabViewShell->GetFrameWeld(), "ColWidthDialog",
nCurHeight,
- STD_COL_WIDTH, eMetric,
- nCurHeight == STD_COL_WIDTH ? 2 : 4, //use 4 digits
for user-defined values
- MAX_COL_WIDTH));
+ STD_COL_WIDTH, eMetric, 2, MAX_COL_WIDTH));
if ( pDlg->Execute() == RET_OK )
{
tools::Long nVal = pDlg->GetInputValue();
