Bug 166229 - Bullet style does not change anymore with list indentation level
Summary: Bullet style does not change anymore with list indentation level
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
25.2.2.2 release
Hardware: All All
: medium minor
Assignee: Miklos Vajna
URL:
Whiteboard: target:25.8.0
Keywords: bibisected, bisected, regression
Depends on:
Blocks: Bullet-Number-Outline-Lists
  Show dependency treegraph
 
Reported: 2025-04-17 14:40 UTC by Henning
Modified: 2025-05-12 14:29 UTC (History)
4 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 Henning 2025-04-17 14:40:32 UTC
Description:
Before, when I created a bulleted list, the bullet's style used to follow the indentation level like so: filled circle, empty circle, filled square, etc.
Now, it stays a filled circle.
Even in an existing document with a bulleted list of the former style, when I create a new list it does not change the bullet style anymore with indentation.
The workaround for the time being is to copy a bit of an old list, which seems to retain the old style inside of it, and modify it.

I guess the overall template has changed, what I consider a problem:
- First, when looking at the style in the list templates, I can only edit each indentation level individually, afraid of maybe missing something else under the hood. I would like to take over the whole list template as it was before.
- Second, I don't want to need to edit the default style, because it will make LO Writer behave differently on different computers.

Steps to Reproduce:
1. create new document in Writer
2. create a bullet list
3. indent

Actual Results:
a. bullet style stays the same: a filled circle
b. in an existing document, a newly created list will differ in style from existing ones

Expected Results:
1. bullet style changes with indentation level
2. in an existing document, the style should remain as it is


Reproducible: Always


User Profile Reset: Yes

Additional Info:
Version: 25.2.2.2 (X86_64) / LibreOffice Community
Build ID: 7370d4be9e3cf6031a51beef54ff3bda878e3fac
CPU threads: 4; OS: Windows 10 X86_64 (10.0 build 19045); UI render: Skia/Raster; VCL: win
Locale: de-DE (en_DE); UI: de-DE
Calc: CL threaded
Comment 1 m_a_riosv 2025-04-18 10:20:19 UTC
I think the bug is with 
Bug 108697 - New default set of bullet characters
https://bugs.documentfoundation.org/show_bug.cgi?id=108697
@Heiko what do you think about?

Reproducible
Version: 25.2.0.3 (X86_64) / LibreOffice Community
Build ID: e1cf4a87eb02d755bce1a01209907ea5ddc8f069
CPU threads: 16; OS: Windows 11 X86_64 (10.0 build 26100); UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US
Calc: CL threaded


Latest version that works on the ones I have installed.
Version: 24.8.6.2 (X86_64) / LibreOffice Community
Build ID: 6d98ba145e9a8a39fc57bcc76981d1fb1316c60c
CPU threads: 16; OS: Windows 11 X86_64 (10.0 build 26100); UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US
Calc: CL threaded
Comment 2 Ezinne 2025-05-01 04:49:08 UTC
Git Bisect shows the regression started from:

commit 626357f53c934e7f57dc80c3c83ad080767961f3
Author: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
Date:   Thu Jul 11 12:48:29 2024 +0200

    Use configured bullet symbol when clicking bullets button
    
    Instead of the generic default symbol
    
    Change-Id: I0923d4ede5d7cc09d61bb57d5cb6dbf6f2af8058
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/170373
    Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
    Tested-by: Jenkins

diff --git a/sw/source/uibase/wrtsh/wrtsh1.cxx b/sw/source/uibase/wrtsh/wrtsh1.cxx
index e243beb22ea8..78170708ebad 100644
--- a/sw/source/uibase/wrtsh/wrtsh1.cxx
+++ b/sw/source/uibase/wrtsh/wrtsh1.cxx
@@ -1584,11 +1584,6 @@ void SwWrtShell::NumOrBulletOn(bool bNum)
         // Append the character template at the numbering.
         SwCharFormat* pChrFormat;
         SwDocShell* pDocSh = GetView().GetDocShell();
-        // #i63395#
-        // Only apply user defined default bullet font
-        const vcl::Font* pFnt = numfunc::IsDefBulletFontUserDefined()
-                           ? &numfunc::GetDefBulletFont()
-                           : nullptr;
 
         if (bNum)
         {
@@ -1615,13 +1610,14 @@ void SwWrtShell::NumOrBulletOn(bool bNum)
 
             if (! bNum)
             {
-                // #i63395#
-                // Only apply user defined default bullet font
-                if ( pFnt )
-                {
-                    aFormat.SetBulletFont( pFnt );
-                }
-                aFormat.SetBulletChar( numfunc::GetBulletChar(nLvl) );
+                uno::Sequence<OUString> aBulletSymbols(
+                    officecfg::Office::Common::BulletsNumbering::DefaultBullets::get());
+                uno::Sequence<OUString> aBulletSymbolsFonts(
+                    officecfg::Office::Common::BulletsNumbering::DefaultBulletsFonts::get());
+                aFormat.SetBulletChar(aBulletSymbols[0].toChar());
+                vcl::Font aFont;
+                aFont.SetFamilyName(aBulletSymbolsFonts[0]);
+                aFormat.SetBulletFont(&aFont);
                 aFormat.SetNumberingType(SVX_NUM_CHAR_SPECIAL);
                 // #i93908# clear suffix for bullet lists
                 aFormat.SetListFormat(u""_ustr, u""_ustr, nLvl);
Comment 3 Miklos Vajna 2025-05-08 11:37:37 UTC
I intend to look at this. Sounds like we always take the first item from that list, while the config has multiple items, so it would be possible to work with the config + have different characters at the same time.
Comment 4 Commit Notification 2025-05-12 14:27:24 UTC
Miklos Vajna committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/d32dd7675b36952dd38e6c9b7d280ff2c4008404

tdf#166229 sw: make bullet style change again with list indentation level

It will be available in 25.8.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.
Comment 5 Miklos Vajna 2025-05-12 14:29:11 UTC
I think this works now.