[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=15
   ||4704

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Mike Kaganski  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=15
   ||0530

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||9397

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Korrawit Pruegsanusak  changed:

   What|Removed |Added

 Attachment #128375|application/x-msdownload|application/pdf
  mime type||

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|BSA target:5.3.0|BSA target:5.3.0
   |target:5.4.0|target:5.4.0 target:6.2.0

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #35 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=8f30759a540c0b323b2fb6d873abd67f37da2ca2

tdf#41542 sw cleanup: m_bBorderDist was always true

It will be available in 6.2.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=11
   ||7449

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=10
   ||7946

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

 Status|RESOLVED|VERIFIED

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=10
   ||6786

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=10
   ||5294

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #34 from Justin L  ---
For a long time (at least since version 3.6) LibreOffice has reset the padding
values to zero when borders were removed, but some older documents may still
have padding values despite having no border - and thus may now render
differently as the padding is activated.

To remove the unwanted padding in LO <= 5.3, first add and apply a border, and
then remove it again.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=10
   ||4220

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

 CC||olivier.hallot@documentfoun
   ||dation.org

--- Comment #33 from Justin L  ---
*** Bug 78090 has been marked as a duplicate of this bug. ***

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #32 from Justin L  ---
Marking this bug as fixed in LibreOffice 5.4 (see comment 30 and comment 31).

To summarize the changes for 5.4
-user interface allows borderless padding in any format (odt/doc/docx etc.) for
Pages, Paragraphs, Characters, Headers, Frames, Images.

Summarized changes for 5.3
-layout displays borderless padding in any format (odt/doc/docx etc.)
-user interface allows borderless padding for FRAMES only in doc/docx only.
-any import/export issues identified were fixed for doc/docx.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Markus Mohrhard  changed:

   What|Removed |Added

 Whiteboard|BSA target:5.3.0|BSA target:5.3.0
   ||target:5.4.0

--- Comment #31 from Markus Mohrhard  ---
I forgot to update the bugzilla script. So the last commits are for 5.4.0 and
not for 5.3.0 unless they get pushed to the 5-3 branch.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #30 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=8a34ff14f18da0df261ae8f1ca3f23de157706a1

tdf#41542 globally allow padding without borders: UI

It will be available in 5.3.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #29 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=8eff1decd91cbfb10094c25d4cf1d2b434a4da72

tdf#41542 MSWordExport: accommodate image's borderless padding

It will be available in 5.3.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #28 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=67e385ac8dfd804015c8b6199865b71ab6e927b4

tdf#41542 MSWordExport: accommodate page's borderless padding

It will be available in 5.3.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #27 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=9130627e21dd7c52c5eee1acc4b71f86eb9f3118

tdf#41542 globally allow padding without borders: layout

It will be available in 5.3.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

 Blocks||33041


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=33041
[Bug 33041] Allow page background to cover the whole page, not only within page
borders/margins
-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
Libreoffice-ux-advise mailing list
Libreoffice-ux-advise@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-ux-advise

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=78
   ||090

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #26 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f013d4a1f4073cda735befd6e446bee35f3db65c

tdf#41542 PaddingWithoutBorders: allow UI changes if...

It will be available in 5.3.0.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #25 from Justin L  ---
Created attachment 128377
  --> https://bugs.documentfoundation.org/attachment.cgi?id=128377=edit
padding_demo.odt

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #24 from Justin L  ---
Created attachment 128375
  --> https://bugs.documentfoundation.org/attachment.cgi?id=128375=edit
padding_demo.pdf: page show page/frame/paragraph/character/header/image
borderless padding

proposed patch:  https://gerrit.libreoffice.org/30434 tdf41542 globally allow
padding without borders: layout

With this patch, I'm suggesting that in LO5.3 we allow the layout of borderless
padding, but leaving the UI component for 5.4 (see Comment 21). This patch
allows the layout demonstrated in padding_demo.pdf

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #23 from Regina Henschel  ---
@Heiko: Columns are difficult.

Case tables:
There exists the ODF style attribute 'table:border-model' with value
'collapsing' and 'separating'. A similar -but not referenced- attribute in CSS
and XSL is 'border-collapse' with values 'separate' and 'collapse'. In case of
'separate', the attribute 'border-spacing' in CSS (and perhaps
'border-separation' in XSL) determines the length of the gap. But a similar
attribute does not exist in ODF.
Therefore there is never a gap between the borders of adjacent table cells in
ODF, and we need not think about a name.

Case page-layout, section, and frame:

If there exists no single  elements, the attribute
'fo:column-gap' of  can be used. If there exists 
elements to make settings for each column, then the distance is set by the
attributes 'fo:start-indent' and 'fo:end-indent' of . These
attributes are based on the corresponding attributes in XSL.

The attributes 'fo:start-indent' of the first column and 'fo:end-indent' of the
last column work, although there exist no UI to set them. And there is no UI
for an unequal distribution of the total gap, but such work too.

The implementation in LibreOffice seems to be wrong, as it used 'fo:column-gap'
although  elements exist.

The attributes 'fo:space-before' and 'fo:space-after' (for the vertical
direction) are defined in ODF, but not implemented in LibreOffice.

In regard to the name:
It should not be 'padding' or 'margin' because those exist in addition for
page-layout, section, and frame and have different meanings.
'start-indent' and 'end-indent' would be fine, if there are plans to allow
different values left and right. 'start'/'end' and not 'left'/'right', because
it depends on the writing direction.
If we stay with the current UI, then 'gap' might be suitable, but it does not
correspond to the file format, in case e.g. three columns with different
'gap'-values are used, which needs to result in single 'indent' values.

The current word 'Spacing' as a generic term is not bad. So I suggest do not
change it for columns, as long as there are no changes in the UI.

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

--- Comment #22 from Heiko Tietze  ---
(In reply to Justin L from comment #21)
> Questions:
> -clarify what "rename the caption to margins" means.  I hope that doesn't
> mean change the tab name from "borders" to "margins". That would be a bad
> idea.

"Border" is fine.

> -confirm that character properties should also support this.

confirmed, same for the Page dialog
As stated in bug 103275 'Use "padding" for space inside boxes and "margin" for
outside distances.'
I wonder how we call the spacing between columns. Would say this is a true
_spacing_. @Regina?

> -is there anything that uses SVXPAGE_BORDER that should NOT allow
> borderless-padding?  (I didn't see anything else.)

No idea

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

[Libreoffice-ux-advise] [Bug 41542] FORMATTING: Allow "Spacing to contents" for edges without a line

[bugzilla-daemon]

https://bugs.documentfoundation.org/show_bug.cgi?id=41542

Justin L  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |jl...@mail.com
   |desktop.org |
Summary|FORMATTING: Text frames:|FORMATTING: Allow "Spacing
   |Allow "Spacing to contents" |to contents" for edges
   |for edges without a line|without a line

--- Comment #21 from Justin L  ---
Changed description to match the decision to implement borderless-padding for
textboxes/frames, pages, headers, paragraphs, ?characters?, images. (Tables
already worked that way.)

Waiting for LO 5.4 codebase to start before implementing the complete fix.
There will be a lot of verification required, as well as compatibility changes
for exporting to other formats.

Questions:
-clarify what "rename the caption to margins" means.  I hope that doesn't mean
change the tab name from "borders" to "margins". That would be a bad idea.
-confirm that character properties should also support this.
-is there anything that uses SVXPAGE_BORDER that should NOT allow
borderless-padding?  (I didn't see anything else.)

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