[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-11-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

Rafael Lima  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |rafael.palma.l...@gmail.com
   |desktop.org |

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-11-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

--- Comment #10 from Rafael Lima  ---
> Not sure if all this helps ;-)

It helped me start from a different angle, then after a some time I managed to
find where it is. Thanks!

Actually the code that creates the default template is hard-coded in the method
SdDrawDocument::CreateLayoutTemplates() inside the file:
/core/sd/source/core/drawdoc4.cxx

I changed the default template and tested if it breaks compatibility with files
created with the current build. Fortunately the styles are saved inside the
.odp file and compatibility is not broken.

I'll submit the patch to Gerrit for review.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-11-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

--- Comment #9 from Heiko Tietze  ---
(In reply to Rafael Lima from comment #8)
> Hi! I am interested in fixing this issue. I just need some help with code
> pointers.

Described my approach in
https://design.blog.documentfoundation.org/2018/02/22/easyhacking-set-environment/

I search for the string "_Word wrap text in shape", find
cui/uiconfig/ui/textattrtabpage.ui, search for TSB_WORDWRAP_TEXT and find
m_xTsbWordWrapText, get its state from SDRATTR_TEXT_WORDWRAP. Atr this point it
becomes a bit time-consuming, for example this is misleading
svx/source/svdraw/svdattr.cxx has rPoolDefaults[SDRATTR_TEXT_WORDWRAP
-SDRATTR_START]=new SdrOnOffItem(SDRATTR_TEXT_WORDWRAP, true);

Not sure if all this helps ;-)

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-11-25 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

--- Comment #8 from Rafael Lima  ---
Hi! I am interested in fixing this issue. I just need some help with code
pointers.

I was browsing the code to search where the default template is located. At
first I was looking for some .otp file (Impress Template File), but could not
find any.

After some searching, it seems that the template is actually hard-coded at:
/core/xmloff/source/draw/XMLGraphicsDefaultStyle.cxx

Can anyone confirm if this is the file I should be working on to fix this
issue?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-07-02 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

Heiko Tietze  changed:

   What|Removed |Added

 CC|libreoffice-ux-advise@lists |heiko.tietze@documentfounda
   |.freedesktop.org|tion.org
   Keywords|needsUXEval |

--- Comment #7 from Heiko Tietze  ---
We discussed the RFE in the design meeting and agree that having text wrapped
by default makes sense. It might be an easyhack. Implementation should check
the patch to not affect existing documents.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-07-02 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

Cor Nouws  changed:

   What|Removed |Added

 CC||c...@nouenoff.nl
Version|7.0.0.0.beta1+  |Inherited From OOo

--- Comment #6 from Cor Nouws  ---
I think it is more occurring that people want text wrapped, also in shapes,
then run out. So IMO good to change this.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-06-30 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

V Stuart Foote  changed:

   What|Removed |Added

 CC||vstuart.fo...@utsa.edu

--- Comment #5 from V Stuart Foote  ---
Wait, for these controls on all Draw shape objects, their Text... dialog, Text
tab, Custom Shape text--doesn't the outcome of 'Word wrap text in shape'
interact with 'Resize shape to fit text' control?

Both are disabled by default for new draw shape text.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-06-30 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

Regina Henschel  changed:

   What|Removed |Added

 CC||rb.hensc...@t-online.de

--- Comment #4 from Regina Henschel  ---
If word wrap is enabled in the default-style, then it would likely solve bug
113926 too. [It is not duplicate, because different kind of shapes are
affected.]

ODF has no explicit default value, the underlying XSL has "wrap" as initial
value. Current MS Office uses "wrap" as default. I think, it would be good to
use "wrap" as default in LibreOffice too.

(In reply to Heiko Tietze from comment #3)
> Any potential side-effect of having word wrap on by default?

I don't know. But fixing the problem should include examining the
imports/exports, especially with OOXML and with binary MS Office formats,
whether they presume any default.

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 134369] Enable word-wrap by default for shapes

2020-06-30 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=134369

Heiko Tietze  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW
   Severity|normal  |enhancement
 CC||libreoffice-ux-advise@lists
   ||.freedesktop.org
   Keywords||needsUXEval
Summary|Text does not wrap in   |Enable word-wrap by default
   |Callouts in Impress |for shapes

--- Comment #3 from Heiko Tietze  ---
Any potential side-effect of having word wrap on by default?

-- 
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs