From 41fa7244746819411f209110f13af43941f17ce8 Mon Sep 17 00:00:00 2001 From: Kelemen Gábor Date: Tue, 12 Nov 2019 15:03:04 +0100 Subject: tdf#108059 Introduce compatibility key for pie chart orientation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There is no similar setting in MSO, it just always uses clockwise. LO on the other hand confuses users by using counterclockwise which turns to clockwise upon save to OOXML and reload. Eliminate this behavior by introducing a compatibility key for OOXML-heavy environments that: - sets the orientation to clockwise when pie chart type is selected - hides the checkbox that would have no effect after a save to OOXML Default value is false to keep current behavior Change-Id: I9e12ecad8e9978f960c921a521461aee023a5063 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/82524 Tested-by: Jenkins Tested-by: László Németh Reviewed-by: Balazs Varga Reviewed-by: László Németh --- chart2/Library_chartcontroller.mk | 4 ++++ chart2/source/controller/dialogs/tp_PolarOptions.cxx | 4 +++- chart2/source/model/template/PieChartTypeTemplate.cxx | 11 ++++++++++- .../schema/org/openoffice/Office/Compatibility.xcs | 8 ++++++++ sw/qa/extras/uiwriter/uiwriter2.cxx | 17 +++++++++++++++++ 5 files changed, 42 insertions(+), 2 deletions(-) diff --git a/chart2/Library_chartcontroller.mk b/chart2/Library_chartcontroller.mk index 48115dd2a33b..05f031415213 100644 --- a/chart2/Library_chartcontroller.mk +++ b/chart2/Library_chartcontroller.mk @@ -22,6 +22,10 @@ $(eval $(call gb_Library_use_external,chartcontroller,boost_headers)) $(eval $(call gb_Library_use_sdk_api,chartcontroller)) +$(eval $(call gb_Library_use_custom_headers,chartcontroller,\ + officecfg/registry \ +)) + $(eval $(call gb_Library_use_libraries,chartcontroller,\ basegfx \ chartcore \ diff --git a/chart2/source/controller/dialogs/tp_PolarOptions.cxx b/chart2/source/controller/dialogs/tp_PolarOptions.cxx index 804c4221247a..e858813bb39f 100644 --- a/chart2/source/controller/dialogs/tp_PolarOptions.cxx +++ b/chart2/source/controller/dialogs/tp_PolarOptions.cxx @@ -22,6 +22,7 @@ #include #include +#include namespace chart { @@ -80,7 +81,8 @@ void PolarOptionsTabPage::Reset(const SfxItemSet* rInAttrs) { m_xFL_StartingAngle->hide(); } - if (rInAttrs->GetItemState(SCHATTR_CLOCKWISE, true, &pPoolItem) == SfxItemState::SET) + // tdf#108059 Hide clockwise orientation checkbox in OOXML-heavy environments it would be useless anyways + if (!officecfg::Office::Compatibility::View::ClockwisePieChartDirection::get() && rInAttrs->GetItemState(SCHATTR_CLOCKWISE, true, &pPoolItem) == SfxItemState::SET) { bool bCheck = static_cast< const SfxBoolItem * >( pPoolItem )->GetValue(); m_xCB_Clockwise->set_active(bCheck); diff --git a/chart2/source/model/template/PieChartTypeTemplate.cxx b/chart2/source/model/template/PieChartTypeTemplate.cxx index ec08764a7509..7f58b6e856e6 100644 --- a/chart2/source/model/template/PieChartTypeTemplate.cxx +++ b/chart2/source/model/template/PieChartTypeTemplate.cxx @@ -34,6 +34,7 @@ #include #include #include +#include #include @@ -237,7 +238,15 @@ void PieChartTypeTemplate::adaptScales( { chart2::ScaleData aScaleData( xAxis->getScaleData() ); AxisHelper::removeExplicitScaling( aScaleData ); - aScaleData.Orientation = chart2::AxisOrientation_MATHEMATICAL; + // tdf#108059 Create new pie/donut charts with clockwise orientation + if (!officecfg::Office::Compatibility::View::ClockwisePieChartDirection::get()) + { + aScaleData.Orientation = chart2::AxisOrientation_MATHEMATICAL; + } + else + { + aScaleData.Orientation = chart2::AxisOrientation_REVERSE; + } xAxis->setScaleData( aScaleData ); } diff --git a/officecfg/registry/schema/org/openoffice/Office/Compatibility.xcs b/officecfg/registry/schema/org/openoffice/Office/Compatibility.xcs index 5b89f9498005..87d139fb9116 100644 --- a/officecfg/registry/schema/org/openoffice/Office/Compatibility.xcs +++ b/officecfg/registry/schema/org/openoffice/Office/Compatibility.xcs @@ -193,6 +193,14 @@ true + + + + Specifies whether the default orientation of pie charts should be clockwise. In OOXML-heavy environments it might be confusing to see charts being mirrored after save, avoid that by setting this key to true. + + + false + diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx b/sw/qa/extras/uiwriter/uiwriter2.cxx index 8fb14a87c945..a6efc4a5ee60 100644 --- a/sw/qa/extras/uiwriter/uiwriter2.cxx +++ b/sw/qa/extras/uiwriter/uiwriter2.cxx @@ -2977,9 +2977,20 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf123218) } ~ReverseXAxisOrientationDoughnutChart() = delete; }; + + struct ClockwisePieChartDirection + : public comphelper::ConfigurationProperty + { + static OUString path() + { + return "/org.openoffice.Office.Compatibility/View/ClockwisePieChartDirection"; + } + ~ClockwisePieChartDirection() = delete; + }; auto batch = comphelper::ConfigurationChanges::create(); ReverseXAxisOrientationDoughnutChart::set(false, batch); + ClockwisePieChartDirection::set(true, batch); batch->commit(); createDoc(); @@ -3028,5 +3039,11 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf123218) CPPUNIT_ASSERT(xAxis.is()); chart2::ScaleData aScaleData = xAxis->getScaleData(); CPPUNIT_ASSERT_EQUAL(chart2::AxisOrientation_MATHEMATICAL, aScaleData.Orientation); + + // tdf#108059 test primary Y axis Orientation value + uno::Reference xYAxis = xCoord->getAxisByDimension(1, 0); + CPPUNIT_ASSERT(xYAxis.is()); + aScaleData = xYAxis->getScaleData(); + CPPUNIT_ASSERT_EQUAL(chart2::AxisOrientation_REVERSE, aScaleData.Orientation); } /* vim:set shiftwidth=4 softtabstop=4 expandtab: */ -- cgit