| Summary: | UI: Missing Backcolor in Preview of conditional format dialog | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | riesslibo <lothar.becker> |
| Component: | Calc | Assignee: | Caolán McNamara <caolan.mcnamara> |
| Status: | VERIFIED FIXED | ||
| Severity: | normal | CC: | caolan.mcnamara, gponzo, heiko.tietze, xiscofauli |
| Priority: | medium | Keywords: | bibisected, bisected, regression |
| Version: | 7.1.1.2 release | ||
| Hardware: | All | ||
| OS: | All | ||
| See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=137059 | ||
| Whiteboard: | target:7.2.0 target:7.1.3 | ||
| Crash report or crash signature: | Regression By: | ||
| Attachments: |
screenshot of wrong preview in conditional format dialog
example for cond form dialog preview fault |
||
|
Description
riesslibo
2021-04-01 09:31:58 UTC
Created attachment 170890 [details]
screenshot of wrong preview in conditional format dialog
I can't reproduce it in Version: 7.2.0.0.alpha0+ / LibreOffice Community Build ID: c088d26578d1be352efa49bd164a8217627648de CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: x11 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded Thank you for reporting the bug. Please attach a sample document, as this makes it easier for us to verify the bug. I have set the bug's status to 'NEEDINFO'. Please change it back to 'UNCONFIRMED' once the requested document is provided. (Please note that the attachment will be public, remove any sensitive information before attaching it. See https://wiki.documentfoundation.org/QA/FAQ#How_can_I_eliminate_confidential_data_from_a_sample_document.3F for help on how to do so.) Created attachment 170891 [details]
example for cond form dialog preview fault
see attached exampleCellB3.ods
Open it, see B3 as conditional formatted with cell style Fault (in german "Fehler").
Now goto menu Format->Conditional Format -> Manage
Click on condition for B3, see that there is no preview in the condition with the cell style Fault ("Fehler")
Regression introduced by: https://cgit.freedesktop.org/libreoffice/core/commit/?id=6f3899b27156591e65f62649a92c727eb6f5dd03 author Heiko Tietze <tietze.heiko@gmail.com> 2020-10-29 15:03:44 +0100 committer Caolán McNamara <caolanm@redhat.com> 2020-11-03 22:02:24 +0100 commit 6f3899b27156591e65f62649a92c727eb6f5dd03 (patch) tree 742b41745e770fbf73ada4d5cea395a262219813 parent 2a7a62c09582ec24247022a94e929610d141a4c9 (diff) Resolves tdf#137059 - Use application colors for font preview Bisected with: bibisect-linux64-7.1 Adding Cc: to Heiko Tietze Same behaviour here. Version: 7.1.1.2 / LibreOffice Community Build ID: 10(Build:2) CPU threads: 8; OS: Linux 5.4; UI render: default; VCL: gtk3 Locale: it-IT (it_IT.UTF-8); UI: it-IT Ubuntu package version: 1:7.1.1~rc2-0ubuntu0.20.04.1~lo1 Calc: threaded @Caolán, i thought you might be interested in this issue I feel https://gerrit.libreoffice.org/c/core/+/114444 will continue to achieve the original goal of tdf#137059 but restore that an explicit set background in the input set of properties, as used in this example, is used as bg Caolán McNamara committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/41145efffe9d5af358a52b9e114bb01b98b3d7c2 tdf#141419 restore use any explicitly set background color in previews It will be available in 7.2.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. done in main, backport to 7-1 in gerrit Verified in Version: 7.2.0.0.alpha0+ / LibreOffice Community Build ID: 41145efffe9d5af358a52b9e114bb01b98b3d7c2 CPU threads: 4; OS: Linux 5.7; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Calc: threaded @Caolán, thanks for fixing this issue!! Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-1": https://git.libreoffice.org/core/commit/40868f520f9da24d3eaa5e0489da8657bc8383b4 tdf#141419 restore use any explicitly set background color in previews It will be available in 7.1.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Verified in nighly build Win10x64 from 24th April: works. Thanks for the work and the quick fix!! Caolán McNamara committed a patch related to this issue. It has been pushed to "libreoffice-7-1-3": https://git.libreoffice.org/core/commit/8aea4386148d65f204f1f4932997d82f99f714fa tdf#141419 restore use any explicitly set background color in previews It will be available in 7.1.3. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. |