Summary: | Impress and Draw use green (accent 1) default for shapes and lines | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Stéphane Guillou (stragu) <stephane.guillou> |
Component: | Draw | Assignee: | Not Assigned <libreoffice-bugs> |
Status: | VERIFIED FIXED | ||
Severity: | normal | CC: | aron.budea, heiko.tietze, momonasmon, quikee |
Priority: | medium | Keywords: | bibisected, bisected, regression |
Version: | 7.6.0.0 alpha0+ | ||
Hardware: | x86-64 (AMD64) | ||
OS: | Linux (All) | ||
See Also: | https://bugs.documentfoundation.org/show_bug.cgi?id=155066 | ||
Whiteboard: | target:24.2.0 target:7.6.0.0.beta2 | ||
Crash report or crash signature: | Regression By: | ||
Bug Depends on: | |||
Bug Blocks: | 107331, 108741 |
Description
Stéphane Guillou (stragu)
2023-04-30 08:27:28 UTC
so what the issue actually? Accent 1 as the default color or Writer still having the old one as default? Using accent 1 theme color as default for fill is mostly by mistake - Miklos added that if there is a theme associated (which it wasn't) it should use the accent 1 color in this cases. What I changed is just that we always have a theme associated with the document so using theme colors is possible. So how should this be fixed? To me, the main problem seems to be that themes are applied as DF, making the fill settings of the default drawing style useless, as they're overwritten anyway. I would prefer to revert the default accent 1 setting, unless we find a way to integrate that with styles. (In reply to Maxim Monastirsky from comment #2) > To me, the main problem seems to be that themes are applied as DF, making > the fill settings of the default drawing style useless, as they're > overwritten anyway. I would prefer to revert the default accent 1 setting, > unless we find a way to integrate that with styles. Themes are integrated with styles.. it's just a matter of setting it for the default drawing style, which indeed is an issue. We do it on top as DF for a new shape instead, probably because it was easier to do. (In reply to Tomaz Vajngerl from comment #3) > Themes are integrated with styles.. it's just a matter of setting it for the > default drawing style This doesn't seem to work for me. If I set the default drawing style to a theme color, and then modify the theme (in the master slide's properties dialog), then the new color is applied to existing shapes again as DF, while the style remains unchanged. Moreover - if I then create a second master page with a different theme color, then shapes placed there continue to use the theme color from the former master. So while it's indeed possible to select a theme color in the style editing dialog, it just sets the given color explicitly, instead of telling the style to get the actual color from the current master each time. (In reply to Maxim Monastirsky from comment #4) > (In reply to Tomaz Vajngerl from comment #3) > This doesn't seem to work for me. If I set the default drawing style to a > theme color, and then modify the theme (in the master slide's properties > dialog), then the new color is applied to existing shapes again as DF, while > the style remains unchanged. Moreover - if I then create a second master > page with a different theme color, then shapes placed there continue to use > the theme color from the former master. So while it's indeed possible to > select a theme color in the style editing dialog, it just sets the given > color explicitly, instead of telling the style to get the actual color from > the current master each time. Ah right - I see what you mean now and that needs to be fixed. My priority is Writer (where styles are taken into account) and later Calc, so I haven't yet looked into this yet. Tomaž Vajngerl committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/2afff27c5327bcb30f894d9a507131f2ad8f1dc6 tdf#155086 remove code that sets theme color to shapes as DF It will be available in 24.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. Tomaž Vajngerl committed a patch related to this issue. It has been pushed to "libreoffice-7-6": https://git.libreoffice.org/core/commit/cf36777e8df6ed7233e3c6c6fc0ba0a1ed505cfb tdf#155086 remove code that sets theme color to shapes as DF It will be available in 7.6.0.0.beta2. 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 Draw with: Version: 24.2.0.0.alpha0+ (X86_64) / LibreOffice Community Build ID: 66da786bdd4588b31755058acf46034c2056215c CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3 Locale: en-AU (en_AU.UTF-8); UI: en-US Calc: threaded Should this be set to fixed, Tomaž? It think so yes Thanks! Verified. |