Bug 79158 - dataloss:Paragraph style background colour ignored on read and lost on save
Summary: dataloss:Paragraph style background colour ignored on read and lost on save
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
4.3.0.0.beta1
Hardware: All Linux (All)
: highest major
Assignee: Miklos Vajna
URL:
Whiteboard: odf target:4.3.0 target:4.4.0
Keywords: bibisected, dataLoss, regression
: 79164 79775 80910 (view as bug list)
Depends on:
Blocks: mab4.3
  Show dependency treegraph
 
Reported: 2014-05-23 23:22 UTC by reporter_of_bugs
Modified: 2015-12-17 08:13 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description reporter_of_bugs 2014-05-23 23:22:07 UTC
Steps to replicate:
1.Open a new writer doc and type the following onto 3 separate lines( i.e paragraphs)

Apple 
Banana
Crowbar

2. select 'Banana' and Format->Paragraph..., then choose the 'Background' tab
3. select 'Gray 5' and press 'OK'
4. Press F11 to open the Styles & Formatting Window, select the rightmost button (tooltip:'New Style from Selection')
5. Give it the name 'GrBg' and press 'OK'
6. save and close the doc. Re-open it and note that the colour formatting has disappeared from the 'Banana' paragraph.

Expected Behaviour: when the style is applied to text the background will change colour and remain that colour when doc is saved & re-opened.

Additional information:
Ubuntu 14.04 64bit
Version: 4.3.0.0.beta1
Build ID: 2e39c7e59c8fc8b16a54c3d981dceef27fb0c07f
Version: 4.2.4.2
Build ID: 63150712c6d317d27ce2db16eb94c2f3d7b699f8

The same doc opened in 4.2.4.2 will show the paragraph background colour as expected.
A coloured background style created in a doc in 4.2.4.2 does not show when opened in 4.3.0.0.beta1.
Comment 1 Joel Madero 2014-05-24 03:19:41 UTC
Confirmed - regression
Ubuntu 14.04 x64
LibreOffice 4.3 Beta1 (broken)
LibreOffice 4.2.4.2 (works)

New (confirmed)
Major - this is something that has the potential of affecting a lot of users - it seems like styles just aren't sticking after save and reopen
Highest - MAB
 52472371d4cd1e7d386e3780b9e986fcb4ea1da2 is the first bad commit
commit 52472371d4cd1e7d386e3780b9e986fcb4ea1da2
Author: Bjoern Michaelsen <bjoern.michaelsen@canonical.com>
Date:   Tue May 20 10:16:41 2014 +0000

    source-hash-944c78ecb91608f4c3e9bab32fdbc90c67326525
    
    commit 944c78ecb91608f4c3e9bab32fdbc90c67326525
    Author:     Andrzej Hunt <andrzej.hunt@collabora.com>
    AuthorDate: Fri Apr 25 20:25:48 2014 +0100
    Commit:     Andrzej Hunt <andrzej.hunt@collabora.com>
    CommitDate: Sat Apr 26 08:24:36 2014 +0100
    
        fdo#75489 Pass toolbar context menu commands to toolbar for handling.
    
        The toolbar context menu is handled by the toolbar manager, however
        the items are provided and managed by the toolbar itself -- previously
        any events on the list of toolbar items in the context menu were silently
        discarded, whereas now we pass them on to the owning toolbar.
    
        Change-Id: Ia17718c3ff8acfba1b6d655022dd9469932f3493

:100644 100644 df7df9357f7ca1ecb4a9fb78f23774ae0075025c fa6afcb40a1990b220e346ee12647f81728f2f2b M	ccache.log
:100644 100644 8186336dc64fc91cff8a087490f54ca1584e4892 a03c81a56c3e0e34002c292723e3ab9cf4abf916 M	commitmsg
:100644 100644 9b48d132f39b397fc8d692d7daf7bf14e7c4b687 8673fadbb0d3fd57e58689b702c3b99a993cca9e M	make.log
:040000 040000 ae652e2ca7716809f637e49a6f79ce5f9cb529d1 ec6f85e2f9ba743959b62b182d705f755095f837 M	opt


# bad: [a92705c1fabafddd43d175a0714855cd22551232] source-hash-c15927f20d4727c3b8de68497b6949e72f9e6e9e
# good: [6ab7f53af36f13bbefdd4e4fcbd3d1ea432a77d9] source-hash-22029c7e17b4cb48acb058d47ec9c3b6b8b6b294
git bisect start 'latest' 'oldest'
# good: [bebf9d31c8fe9de96798484288a0fffc4d54917d] source-hash-09e5de8278dd8f13adcf614db35c8a8a04ba8e47
git bisect good bebf9d31c8fe9de96798484288a0fffc4d54917d
# good: [fac643700ecfabc737836bbed068776f34037d53] source-hash-40a61d93ade494fa98c23a9fd8776c8dadf8f30f
git bisect good fac643700ecfabc737836bbed068776f34037d53
# bad: [44e392733ca46fb0fde9664f8b33ff4da5295caa] source-hash-062e69f40b749aa8a6058c3e6ca328af86aeb45b
git bisect bad 44e392733ca46fb0fde9664f8b33ff4da5295caa
# good: [3e370810edadd040e63cef076594a2058603b344] source-hash-ddd4b159531e835539e86f76c56d1013de46a2bd
git bisect good 3e370810edadd040e63cef076594a2058603b344
# good: [3e370810edadd040e63cef076594a2058603b344] source-hash-ddd4b159531e835539e86f76c56d1013de46a2bd
git bisect good 3e370810edadd040e63cef076594a2058603b344
# skip: [c05d75aa2ab2065c8953a1649866074f2e0a75f1] source-hash-28fc951233a58fb4d8515ed4380140aaee0afd32
git bisect skip c05d75aa2ab2065c8953a1649866074f2e0a75f1
# good: [304c44989cd8e7d5074eeb53ca14c54b3b6e2e16] source-hash-f05861de995f8d4edb1a97c616d050f55ec04c32
git bisect good 304c44989cd8e7d5074eeb53ca14c54b3b6e2e16
# good: [1fd4b5c55c33b4df5d8b74e00fa49d45cf5927a5] source-hash-9f06e4bc3a56806061f759770f758ad3c7ddf09c
git bisect good 1fd4b5c55c33b4df5d8b74e00fa49d45cf5927a5
# skip: [495dc74dce0a63505cc339db9d2ee708f31d1e84] source-hash-4158d8843d50d50e9830c8dc24af8722de77f4af
git bisect skip 495dc74dce0a63505cc339db9d2ee708f31d1e84
# good: [ecbd42c0caaeb7db276bb0004a9f6a7133086bb1] source-hash-da1425650b0c3e0921d2c9d51d461aca99038650
git bisect good ecbd42c0caaeb7db276bb0004a9f6a7133086bb1
# good: [6efa4f17265c3af9378f78bb28570f75c3d88723] source-hash-2e41702ed4f224f195d33054e2e714f546b4442c
git bisect good 6efa4f17265c3af9378f78bb28570f75c3d88723
# bad: [52472371d4cd1e7d386e3780b9e986fcb4ea1da2] source-hash-944c78ecb91608f4c3e9bab32fdbc90c67326525
git bisect bad 52472371d4cd1e7d386e3780b9e986fcb4ea1da2
# first bad commit: [52472371d4cd1e7d386e3780b9e986fcb4ea1da2] source-hash-944c78ecb91608f4c3e9bab32fdbc90c67326525
Comment 2 Jean-Baptiste Faure 2014-06-04 07:09:31 UTC
The problem is worst: paragraph style background color is not only ignored when you open a document made with a previous version of LO, it is lost i you save your document with LO 4.3.0.0.beta1.
That is a very bad behavior when you use paragraph style background color to highlight important informations in professional documents.
So added keyword dataloss and updated the summary.

Best regards. JBF
Comment 3 V Stuart Foote 2014-06-12 12:29:46 UTC
*** Bug 79775 has been marked as a duplicate of this bug. ***
Comment 4 Xisco Faulí 2014-06-12 14:04:54 UTC
It looks like this regression was introduced by 4a0b5e569d070c286daefb0fdfe45c0dd15d241c. I reverted the commit localy and the grey backgrounds of https://bugs.freedesktop.org/attachment.cgi?id=100619 were displayed correctly.
Comment 5 Michael Stahl (allotropia) 2014-07-04 16:49:51 UTC
fixed on libreoffice-4-3 by commit dc17dad5fd7509f191718df8690e5847ab87669a

fixed on master, presumably by 7d9bb549d498d6beed2c4050c402d09643febdfa
or one of the follow-ups
Comment 6 Michael Stahl (allotropia) 2014-07-04 16:50:55 UTC
*** Bug 79164 has been marked as a duplicate of this bug. ***
Comment 7 Jean-Baptiste Faure 2014-07-05 05:14:05 UTC
Verified fixed under Ubuntu 14.04 x86-64 in LibreOffice 4.3.1.0.0+. 
Build ID: dc17dad5fd7509f191718df8690e5847ab87669a

Thank you very much.

Best regards. JBF
Comment 8 Cor Nouws 2014-07-05 17:57:08 UTC
*** Bug 80910 has been marked as a duplicate of this bug. ***
Comment 9 Michael Stahl (allotropia) 2014-07-10 18:52:21 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-4-3-0":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=18c105107b5664c3247274dbb629781606029634&h=libreoffice-4-3-0

fdo#78908 Revert "Merge back branch alg_writerframes to trunk"


It will be available already in LibreOffice 4.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.
Comment 10 Michael Stahl (allotropia) 2014-07-10 18:57:11 UTC
oops, that was VERIFIED before...
Comment 11 Robinson Tryon (qubit) 2015-12-17 08:13:53 UTC Comment hidden (obsolete)