[Libreoffice-bugs] [Bug 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

2021-09-23 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=99492

--- Comment #15 from Justin L  ---
This painting of Table / Row / Cell is currently handled by recursive calls to
SwFrame::PaintSwFrameBackground.

It first paints the table (IsTabFrame()), then it GetLower() and paints the row
(IsRowFrame()), then it GetLower() and paints the cell (IsCellFrame()).

I expect the key to solving this is to cache/delay the CellFrame paintings, and
to handle them after all the rows have been printed. But that is starting to
get past my skill level.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

2021-09-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=99492

Justin L  changed:

   What|Removed |Added

 Resolution|FIXED   |---
   Assignee|jl...@mail.com  |libreoffice-b...@lists.free
   ||desktop.org
 Status|RESOLVED|REOPENED

--- Comment #14 from Justin L  ---
re-opening since the attempted commit caused a nasty regression as documented
in bug 144554. The fix has been reverted back to 7.1.7.

Make sure to review  attachment 175083 (mergedCellWithBackground.odt) and
attachment 175139 (mergedCellWithBackgroundC.docx) with any attempted fix.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

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

Justin L  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |jl...@mail.com
   |desktop.org |
 Status|NEW |ASSIGNED

--- Comment #9 from Justin L  ---
proposed fix at https://gerrit.libreoffice.org/c/core/+/99343

-- 
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 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

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

--- Comment #8 from Justin L  ---
Created attachment 163454
  --> https://bugs.documentfoundation.org/attachment.cgi?id=163454=edit
mergedCellColor.odt: only row background interferes with cell, not table
background.

-- 
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 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

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

--- Comment #7 from Justin L  ---
Code pointer: this commit probably stopped painting merged cells
commit 4207a09677e03f857dc9341616775be6c4e5a199
Author: Vladimir Glazounov on Wed Jun 4 08:50:59 2008 +
  INTEGRATION: CWS sw30bf04 (1.115.2); FILE MERGED
  2008/05/09 14:05:11 fme 1.115.2.1: #i80595# ww8 import should create new
tables

I can't tell why this was added. Reverting it fixes the problem.

-- 
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 99492] cell background colour ignored on all but the top grids of a vertically merged cell.

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

Justin L  changed:

   What|Removed |Added

Summary|LibreOffice is unable to|cell background colour
   |change background of table  |ignored on all but the top
   |copy/pasted from HTML   |grids of a vertically
   ||merged cell.

--- Comment #6 from Justin L  ---
I confirmed with a few bibisect spot-checks that this new definition of the
problem has also been inherited from OOo.

-- 
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