Bug 69524 - EDITING Creating new style from selected graphic, adds properties for dimensions, and thus changes the size of images that it is applied too, which is wrong
Summary: EDITING Creating new style from selected graphic, adds properties for dimens...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version: 3.3.0 release
Hardware: Other All
: medium normal
Assignee: Not Assigned
QA Contact:
URL:
Whiteboard: preBibisect
Keywords:
Depends on:
Blocks:
 
Reported: 2013-09-18 12:17 UTC by Cor Nouws
Modified: 2015-01-03 21:14 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:

Attachments

Description Cor Nouws 2013-09-18 12:17:57 UTC 
To reproduce:

- writer document with some text
- insert graphic and change the border setting
- put focus in box Apply Style, where name 'Graphics' is shown, and type new name: 'Test1'
- set focus in text of document and insert another grafic, with different dimensions then the first one.
- while that graphic is selected, select 'Test1' from the box Apply Style

  > not only border is added, but also the size is changed!

I was pretty sure that somewhere  in the past, this did not happen. And found an old document that confirms that.

Grabbed the info from the styles.xml
"_5f_Icoontje" is the old one.
"_5f_KaderMagentaGrijs" is the one I made now in 4.1.1.2.



<style:style style:name="_5f_Icoontje" style:display-name="_Icoontje" style:family="graphic" style:parent-style-name="Graphics">
	<style:graphic-properties text:anchor-type="paragraph" svg:x="0cm" svg:y="0cm" style:wrap="dynamic" style:number-wrapped-paragraphs="no-limit" style:wrap-contour="false" style:vertical-pos="top" style:vertical-rel="paragraph-content" style:horizontal-pos="right" style:horizontal-rel="paragraph" fo:padding="0.049cm" fo:border="0.51pt solid #000080" style:shadow="none"/></style:style>


<style:style style:name="_5f_KaderMagentaGrijs" style:display-name="_KaderMagentaGrijs" style:family="graphic" style:parent-style-name="Graphics">
	<style:graphic-properties svg:width="4.657cm" fo:min-height="5.741cm" 	
	text:anchor-type="paragraph" svg:x="6.172cm" svg:y="0cm" style:wrap="dynamic" style:number-wrapped-paragraphs="no-limit" style:wrap-contour="false" style:vertical-pos="top" style:vertical-rel="paragraph-content" style:horizontal-pos="center" style:horizontal-rel="paragraph" fo:padding="0.049cm" fo:border="0.54pt solid #ff33ff" style:shadow="none"/></style:style>


Difference:
svg:width="4.657cm" fo:min-height="5.741cm"

(Also for Anchor type a vallue is added: svg:x="6.172cm"  - no idea if that cause a problem)


I see the problem already in 3.3.0. The style icoontje (properties above) has been made somewhere in 2009 or earlier...
Comment 1 Cor Nouws 2013-09-18 12:19:45 UTC 
As a work around, the user can
 - first apply the style
 - then open window Graphics of the graphic and click 'original dimensions'.

But it's so great the LibreOffice offers styles and lovely that they are so useful :)
Comment 2 Cor Nouws 2013-09-18 12:30:32 UTC 
And of course, create a new style in the window Styles and Formatting, based on Graphic, works too.
Though dimensions in the window are set to 0,36 cm, practical effec is that the style respects the size of the graphic.
Comment 3 Cor Nouws 2013-09-18 12:46:42 UTC 
Indeed, changing the mentioned properties in the styles.xml corrects the wrong behaviour.
Comment 4 Cor Nouws 2013-11-28 14:48:50 UTC 
regression from OOo, but still :(
Comment 5 Matthew Francis 2014-12-29 14:25:44 UTC 
Could you please clarify if you think this is an early regression since the fork from OOo (in which case, please add Whiteboard: preBibisect), or if it was in fact present before the fork (in which case set Version: "Inherited from OOo" and remove Keywords: regression)
Comment 6 Cor Nouws 2014-12-29 16:02:26 UTC 
(In reply to Matthew Francis from comment #5)
> Could you please clarify if you think this is an early regression since the
> fork from OOo 

As I wrote when submitting the issue: " I see the problem already in 3.3.0. The style icoontje (properties above) has been made somewhere in 2009 or earlier..."

So it's fair to remove 'regression' since the problem origens from OOo

Work Around: create new Style for images/frames in Dialog F11

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.