[Libreoffice-commits] core.git: officecfg/registry oox/source sd/qa sd/source sd/xml slideshow/source
jan iversen
jani at documentfoundation.org
Thu Jun 9 15:25:48 UTC 2016
officecfg/registry/data/org/openoffice/Office/UI/Effects.xcu | 33 -----------
oox/source/ppt/slidetransition.cxx | 11 ---
oox/source/ppt/slidetransitioncontext.cxx | 7 --
sd/qa/unit/data/AllTransitions.odp |binary
sd/qa/unit/export-tests.cxx | 3 -
sd/source/filter/eppt/pptx-epptbase.cxx | 11 ---
sd/source/filter/eppt/pptx-epptooxml.cxx | 6 --
sd/source/filter/ppt/pptanimations.hxx | 2
sd/source/filter/ppt/pptin.cxx | 12 ----
sd/xml/effects.xml | 16 -----
sd/xml/transitions.xml | 6 --
slideshow/source/engine/transitions/ellipsewipe.cxx | 25 ++------
slideshow/source/engine/transitions/ellipsewipe.hxx | 4 -
slideshow/source/engine/transitions/transitionfactorytab.cxx | 2
14 files changed, 11 insertions(+), 127 deletions(-)
New commits:
commit 4bcb4774a331cbe7edbb8d92bc6b628e79eee9a6
Author: jan iversen <jani at documentfoundation.org>
Date: Thu Jun 9 15:23:06 2016 +0000
Revert "Slideshow: Add 'Oval' Shape Transition variant"
The patch caused problems with the export filter tests, to check that you need to add --with-export-validation to your autopen.input
see https://cgit.freedesktop.org/libreoffice/contrib/dev-tools/tree/export-validation for more info on how to set it up
This reverts commit 248c5ea771255b54e64394458a321ccf829bbd02.
Change-Id: Ib3b8fa7bf80630feeca1f24dfb1ceb5a945d7162
Reviewed-on: https://gerrit.libreoffice.org/26114
Reviewed-by: jan iversen <jani at documentfoundation.org>
Tested-by: jan iversen <jani at documentfoundation.org>
diff --git a/officecfg/registry/data/org/openoffice/Office/UI/Effects.xcu b/officecfg/registry/data/org/openoffice/Office/UI/Effects.xcu
index 23824c1..765a0d5 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/Effects.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/Effects.xcu
@@ -50,11 +50,6 @@
<value xml:lang="en-US">Circle</value>
</prop>
</node>
- <node oor:name="ooo-entrance-oval" oor:op="replace">
- <prop oor:name="Label" oor:type="xs:string">
- <value xml:lang="en-US">Oval Vertical</value>
- </prop>
- </node>
<node oor:name="ooo-entrance-fly-in-slow" oor:op="replace">
<prop oor:name="Label" oor:type="xs:string">
<value xml:lang="en-US">Fly in Slow</value>
@@ -1733,16 +1728,6 @@
<value xml:lang="en-US">Circle</value>
</prop>
</node>
- <node oor:name="oval-horizontal" oor:op="replace">
- <prop oor:name="Label" oor:type="xs:string">
- <value xml:lang="en-US">Oval Horizontal</value>
- </prop>
- </node>
- <node oor:name="oval-vertical" oor:op="replace">
- <prop oor:name="Label" oor:type="xs:string">
- <value xml:lang="en-US">Oval Vertical</value>
- </prop>
- </node>
<node oor:name="diamond" oor:op="replace">
<prop oor:name="Label" oor:type="xs:string">
<value xml:lang="en-US">Diamond</value>
@@ -2066,22 +2051,6 @@
<value>circle</value>
</prop>
</node>
- <node oor:name="shape-oval-horizontal" oor:op="replace">
- <prop oor:name="Set" oor:type="xs:string">
- <value>shape</value>
- </prop>
- <prop oor:name="Variant" oor:type="xs:string">
- <value>oval-horizontal</value>
- </prop>
- </node>
- <node oor:name="shape-oval-vertical" oor:op="replace">
- <prop oor:name="Set" oor:type="xs:string">
- <value>shape</value>
- </prop>
- <prop oor:name="Variant" oor:type="xs:string">
- <value>oval-vertical</value>
- </prop>
- </node>
<node oor:name="shape-diamond" oor:op="replace">
<prop oor:name="Set" oor:type="xs:string">
<value>shape</value>
@@ -2414,7 +2383,7 @@
<value xml:lang="en-US">Basic</value>
</prop>
<prop oor:name="Effects" oor:type="oor:string-list">
- <value oor:separator=";">ooo-entrance-appear;ooo-entrance-fly-in;ooo-entrance-venetian-blinds;ooo-entrance-box;ooo-entrance-checkerboard;ooo-entrance-circle;ooo-entrance-oval;ooo-entrance-fly-in-slow;ooo-entrance-diamond;ooo-entrance-dissolve-in;ooo-entrance-flash-once;ooo-entrance-peek-in;ooo-entrance-plus;ooo-entrance-random-bars;ooo-entrance-split;ooo-entrance-diagonal-squares;ooo-entrance-wedge;ooo-entrance-wheel;ooo-entrance-wipe;ooo-entrance-random</value>
+ <value oor:separator=";">ooo-entrance-appear;ooo-entrance-fly-in;ooo-entrance-venetian-blinds;ooo-entrance-box;ooo-entrance-checkerboard;ooo-entrance-circle;ooo-entrance-fly-in-slow;ooo-entrance-diamond;ooo-entrance-dissolve-in;ooo-entrance-flash-once;ooo-entrance-peek-in;ooo-entrance-plus;ooo-entrance-random-bars;ooo-entrance-split;ooo-entrance-diagonal-squares;ooo-entrance-wedge;ooo-entrance-wheel;ooo-entrance-wipe;ooo-entrance-random</value>
</prop>
</node>
<node oor:name="special" oor:op="replace">
diff --git a/oox/source/ppt/slidetransition.cxx b/oox/source/ppt/slidetransition.cxx
index 49e24cf..9ae715f 100644
--- a/oox/source/ppt/slidetransition.cxx
+++ b/oox/source/ppt/slidetransition.cxx
@@ -371,17 +371,6 @@ namespace oox { namespace ppt {
mnTransitionType = TransitionType::ELLIPSEWIPE;
mnTransitionSubType = TransitionSubType::CIRCLE;
break;
- case PPT_TOKEN( oval ):
- mnTransitionType = TransitionType::ELLIPSEWIPE;
- if( ooxToOdpDirection( param1 ) == TransitionSubType::VERTICAL )
- {
- mnTransitionSubType = TransitionSubType::VERTICAL;
- }
- else
- {
- mnTransitionSubType = TransitionSubType::HORIZONTAL;
- }
- break;
case PPT_TOKEN( diamond ):
mnTransitionType = TransitionType::IRISWIPE;
mnTransitionSubType = TransitionSubType::DIAMOND;
diff --git a/oox/source/ppt/slidetransitioncontext.cxx b/oox/source/ppt/slidetransitioncontext.cxx
index 957d025..9b29653 100644
--- a/oox/source/ppt/slidetransitioncontext.cxx
+++ b/oox/source/ppt/slidetransitioncontext.cxx
@@ -125,13 +125,6 @@ SlideTransitionContext::~SlideTransitionContext() throw()
}
return this;
case PPT_TOKEN( circle ):
- case PPT_TOKEN( oval ) :
- if (!mbHasTransition)
- {
- mbHasTransition = true;
- maTransition.setOoxTransitionType( aElementToken, rAttribs.getToken( XML_dir, XML_vert), 0);
- }
- return this;
case PPT_TOKEN( diamond ):
case PPT_TOKEN( dissolve ):
case PPT_TOKEN( newsflash ):
diff --git a/sd/qa/unit/data/AllTransitions.odp b/sd/qa/unit/data/AllTransitions.odp
index 8922a07..dfb8d2a 100644
Binary files a/sd/qa/unit/data/AllTransitions.odp and b/sd/qa/unit/data/AllTransitions.odp differ
diff --git a/sd/qa/unit/export-tests.cxx b/sd/qa/unit/export-tests.cxx
index e562191..150e0b5 100644
--- a/sd/qa/unit/export-tests.cxx
+++ b/sd/qa/unit/export-tests.cxx
@@ -1502,9 +1502,6 @@ void SdExportTest::testExportTransitionsPPTX()
// NEWSFLASH
CPPUNIT_ASSERT(checkTransitionOnPage(xDoc, 74, TransitionType::ZOOM, TransitionSubType::ROTATEIN));
- //OVAL VERTICAL
- CPPUNIT_ASSERT(checkTransitionOnPage(xDoc, 76, TransitionType::ELLIPSEWIPE, TransitionSubType::VERTICAL));
-
xDocShRef->DoClose();
}
diff --git a/sd/source/filter/eppt/pptx-epptbase.cxx b/sd/source/filter/eppt/pptx-epptbase.cxx
index 8775200..7bd4b65 100644
--- a/sd/source/filter/eppt/pptx-epptbase.cxx
+++ b/sd/source/filter/eppt/pptx-epptbase.cxx
@@ -798,16 +798,7 @@ sal_Int8 PPTWriterBase::GetTransition( sal_Int16 nTransitionType, sal_Int16 nTra
break;
case TransitionType::ELLIPSEWIPE :
{
- switch( nTransitionSubtype ) {
- case TransitionSubType::VERTICAL:
- nPPTTransitionType = PPT_TRANSITION_TYPE_OVAL_VERTICAL;
- break;
- case TransitionSubType::HORIZONTAL:
- nPPTTransitionType = PPT_TRANSITION_TYPE_OVAL_HORIZONTAL;
- break;
- default:
- nPPTTransitionType = PPT_TRANSITION_TYPE_CIRCLE;
- }
+ nPPTTransitionType = PPT_TRANSITION_TYPE_CIRCLE;
}
break;
case TransitionType::FOURBOXWIPE :
diff --git a/sd/source/filter/eppt/pptx-epptooxml.cxx b/sd/source/filter/eppt/pptx-epptooxml.cxx
index 46c06cc..32f7171 100644
--- a/sd/source/filter/eppt/pptx-epptooxml.cxx
+++ b/sd/source/filter/eppt/pptx-epptooxml.cxx
@@ -696,12 +696,6 @@ void PowerPointExport::WriteTransition( const FSHelperPtr& pFS )
case PPT_TRANSITION_TYPE_CIRCLE:
nTransition = XML_circle;
break;
- case PPT_TRANSITION_TYPE_OVAL_VERTICAL:
- nTransition = XML_oval;
- break;
- case PPT_TRANSITION_TYPE_OVAL_HORIZONTAL:
- nTransition = XML_oval;
- break;
case PPT_TRANSITION_TYPE_COMB:
nTransition = XML_comb;
pDirection = (nDirection == 1) ? "vert" : "horz";
diff --git a/sd/source/filter/ppt/pptanimations.hxx b/sd/source/filter/ppt/pptanimations.hxx
index 534f7b6..599af58 100644
--- a/sd/source/filter/ppt/pptanimations.hxx
+++ b/sd/source/filter/ppt/pptanimations.hxx
@@ -60,8 +60,6 @@ namespace ppt
#define PPT_TRANSITION_TYPE_SMOOTHFADE 23 // Alpha Fade in MS-PPT Specs
#define PPT_TRANSITION_TYPE_WHEEL 26
#define PPT_TRANSITION_TYPE_CIRCLE 27
-#define PPT_TRANSITION_TYPE_OVAL_HORIZONTAL 28
-#define PPT_TRANSITION_TYPE_OVAL_VERTICAL 29
// atoms
#define DFF_msofbtAnimEvent 0xf125
diff --git a/sd/source/filter/ppt/pptin.cxx b/sd/source/filter/ppt/pptin.cxx
index 2955b71..a8f38b4 100644
--- a/sd/source/filter/ppt/pptin.cxx
+++ b/sd/source/filter/ppt/pptin.cxx
@@ -1720,18 +1720,6 @@ void ImplSdPPTImport::ImportPageEffect( SdPage* pPage, const bool bNewAnimations
pPage->setTransitionSubtype( animations::TransitionSubType::CIRCLE );
}
break;
- case PPT_TRANSITION_TYPE_OVAL_VERTICAL :
- {
- pPage->setTransitionType( animations::TransitionType::ELLIPSEWIPE );
- pPage->setTransitionSubtype( animations::TransitionSubType::VERTICAL );
- }
- break;
- case PPT_TRANSITION_TYPE_OVAL_HORIZONTAL :
- {
- pPage->setTransitionType( animations::TransitionType::ELLIPSEWIPE );
- pPage->setTransitionSubtype( animations::TransitionSubType::HORIZONTAL );
- }
- break;
case PPT_TRANSITION_TYPE_WEDGE :
{
pPage->setTransitionType( animations::TransitionType::FANWIPE );
diff --git a/sd/xml/effects.xml b/sd/xml/effects.xml
index b778938..bef5e05 100644
--- a/sd/xml/effects.xml
+++ b/sd/xml/effects.xml
@@ -91,22 +91,6 @@
</anim:par>
<anim:par smil:begin="indefinite" smil:fill="hold">
<anim:par smil:begin="0" smil:fill="hold">
- <anim:par pres:preset-property="Direction" smil:begin="0" smil:fill="hold" pres:node-type="on-click" pres:preset-class="entrance" pres:preset-id="ooo-entrance-oval" pres:preset-sub-type="in">
- <anim:set smil:begin="0" smil:dur="0.001" smil:fill="hold" smil:attributeName="visibility" smil:to="visible"/>
- <anim:transitionFilter smil:dur="2" smil:type="ellipseWipe" smil:subtype="vertical" smil:direction="reverse"/>
- </anim:par>
- </anim:par>
- </anim:par>
- <anim:par smil:begin="indefinite" smil:fill="hold">
- <anim:par smil:begin="0" smil:fill="hold">
- <anim:par pres:preset-property="Direction" smil:begin="0" smil:fill="hold" pres:node-type="on-click" pres:preset-class="entrance" pres:preset-id="ooo-entrance-oval" pres:preset-sub-type="out">
- <anim:set smil:begin="0" smil:dur="0.001" smil:fill="hold" smil:attributeName="visibility" smil:to="visible"/>
- <anim:transitionFilter smil:dur="2" smil:type="ellipseWipe" smil:subtype="vertical"/>
- </anim:par>
- </anim:par>
- </anim:par>
- <anim:par smil:begin="indefinite" smil:fill="hold">
- <anim:par smil:begin="0" smil:fill="hold">
<anim:par pres:preset-property="Direction" smil:begin="0" smil:fill="hold" pres:node-type="on-click" pres:preset-class="entrance" pres:preset-id="ooo-entrance-fly-in-slow" pres:preset-sub-type="from-bottom">
<anim:set smil:begin="0" smil:dur="0.001" smil:fill="hold" smil:attributeName="visibility" smil:to="visible"/>
<anim:animate smil:dur="5" smil:fill="hold" smil:attributeName="x" smil:values="x;x" smil:keyTimes="0;1"/>
diff --git a/sd/xml/transitions.xml b/sd/xml/transitions.xml
index cefcab3..8b96d79 100644
--- a/sd/xml/transitions.xml
+++ b/sd/xml/transitions.xml
@@ -107,12 +107,6 @@
<anim:par pres:preset-id="shape-circle">
<anim:transitionFilter smil:type="ellipseWipe" smil:subtype="circle"/>
</anim:par>
- <anim:par pres:preset-id="shape-oval-horizontal">
- <anim:transitionFilter smil:type="ellipseWipe" smil:subtype="horizontal"/>
- </anim:par>
- <anim:par pres:preset-id="shape-oval-vertical">
- <anim:transitionFilter smil:type="ellipseWipe" smil:subtype="vertical"/>
- </anim:par>
<!-- <anim:par pres:preset-id="circle-in"> -->
<!-- <anim:transitionFilter smil:type="ellipseWipe" smil:subtype="circle" smil:direction="reverse"/> -->
<!-- </anim:par> -->
diff --git a/slideshow/source/engine/transitions/ellipsewipe.cxx b/slideshow/source/engine/transitions/ellipsewipe.cxx
index c3dd9cc..35aa9ec 100644
--- a/slideshow/source/engine/transitions/ellipsewipe.cxx
+++ b/slideshow/source/engine/transitions/ellipsewipe.cxx
@@ -17,7 +17,7 @@
* the License at http://www.apache.org/licenses/LICENSE-2.0 .
*/
-#include <com/sun/star/animations/TransitionSubType.hpp>
+
#include <basegfx/numeric/ftools.hxx>
#include <basegfx/polygon/b2dpolygontools.hxx>
#include "ellipsewipe.hxx"
@@ -29,23 +29,12 @@ namespace internal {
::basegfx::B2DPolyPolygon EllipseWipe::operator () ( double t )
{
- ::basegfx::B2DPoint rCenter(0.5,0.5);
- double fRadius = ::basegfx::pruneScaleValue( t * M_SQRT2 / 2.0 );
-
- if( mnSubType == com::sun::star::animations::TransitionSubType::VERTICAL )
- {
- //oval :
- ::basegfx::B2DPolygon poly (
- ::basegfx::tools::createPolygonFromEllipse( rCenter, fRadius*2, fRadius ) ); //Horizontal Ellipse is rotated by 90 degress
- return ::basegfx::B2DPolyPolygon( poly );
- }
- else
- {
- // circle:
- ::basegfx::B2DPolygon poly(
- ::basegfx::tools::createPolygonFromCircle( rCenter, fRadius ) );
- return ::basegfx::B2DPolyPolygon( poly );
- }
+ // currently only circle:
+ ::basegfx::B2DPolygon poly(
+ ::basegfx::tools::createPolygonFromCircle(
+ ::basegfx::B2DPoint( 0.5, 0.5 ),
+ ::basegfx::pruneScaleValue( t * M_SQRT2 / 2.0 ) ) );
+ return ::basegfx::B2DPolyPolygon( poly );
}
}
diff --git a/slideshow/source/engine/transitions/ellipsewipe.hxx b/slideshow/source/engine/transitions/ellipsewipe.hxx
index c8d991b..d710cba 100644
--- a/slideshow/source/engine/transitions/ellipsewipe.hxx
+++ b/slideshow/source/engine/transitions/ellipsewipe.hxx
@@ -30,10 +30,8 @@ namespace internal {
class EllipseWipe : public ParametricPolyPolygon
{
public:
- explicit EllipseWipe( sal_Int32 nSubType ): mnSubType( nSubType ) {}
+ explicit EllipseWipe( sal_Int32 /*nTransitionSubType xxx todo */ ) {}
virtual ::basegfx::B2DPolyPolygon operator () ( double x ) override;
-private:
- sal_Int32 mnSubType;
};
}
diff --git a/slideshow/source/engine/transitions/transitionfactorytab.cxx b/slideshow/source/engine/transitions/transitionfactorytab.cxx
index 021c359..8f10a55 100644
--- a/slideshow/source/engine/transitions/transitionfactorytab.cxx
+++ b/slideshow/source/engine/transitions/transitionfactorytab.cxx
@@ -699,7 +699,7 @@ static const TransitionInfo lcl_transitionInfo[] =
1.0, // no scaling
TransitionInfo::REVERSEMETHOD_SUBTRACT_AND_INVERT,
true, // 'out' by parameter sweep inversion
- true // scale isotrophically to target size
+ false // scale isotrophically to target size
},
More information about the Libreoffice-commits
mailing list