sw/source/uibase/app/swmodul1.cxx |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 4b3eb51d4e2fb19c3c05a83b6890de070aa58ab7
Author:     Miklos Vajna <[email protected]>
AuthorDate: Fri Nov 28 08:25:08 2025 +0100
Commit:     Miklos Vajna <[email protected]>
CommitDate: Sat Nov 29 09:39:30 2025 +0100

    cool#13574 sw lok: increase saturation of non-standard redline render mode
    
    Factors considered:
    
    1) different redline authors have different colors, so losing the color
       complete is not ideal
    2) a result that is close to white or black is not ideal, either light or
       dark mode result will be unreadable
    3) we want a noticeable difference between the standard rendering and the
       ~omit style rendering
    
    The existing code did 1) and 2), but was weak on 3). So go with 25%
    saturation instead of 50% as a fix.
    
    Change-Id: I31891ba40f665e2fa16ef3450dd49ee9f7683a4b
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/194799
    Tested-by: Jenkins
    Reviewed-by: Miklos Vajna <[email protected]>

diff --git a/sw/source/uibase/app/swmodul1.cxx 
b/sw/source/uibase/app/swmodul1.cxx
index e361ed9a23ec..75d8909559b3 100644
--- a/sw/source/uibase/app/swmodul1.cxx
+++ b/sw/source/uibase/app/swmodul1.cxx
@@ -497,7 +497,8 @@ static void lcl_FillAuthorAttr( std::size_t nAuthor, 
SfxItemSet &rSet,
             sal_uInt16 nSaturation;
             sal_uInt16 nBrightness;
             aCol.RGBtoHSB(nHue, nSaturation, nBrightness);
-            nSaturation = nSaturation / 2;
+            // 25% saturation: balance between complete gray and 
hard-to-notice small difference.
+            nSaturation = nSaturation / 4;
             aCol = Color::HSBtoRGB(nHue, nSaturation, nBrightness);
         }
     }

Reply via email to