commit 2f43d95a0fb4bdd7121968ff9f365ffc325e5647
author Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de> Wed Nov 24 14:07:05 2021 +0100
committer Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de> Wed Nov 24 16:54:30 2021 +0100
tree 61e9f087300e5b7723c89aa50d9c615f57afdf45
parent 4b98cdc9ed51186cd0a7ef5041bf8e8e29420289

tdf#145828 Sidebar line styles: Default to "thin" width

Change-Id: I78d754d36c7ea41f7e3259be331d23b8a25c480f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125771
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>

sc/source/ui/sidebar/CellBorderStyleControl.cxx

diff --git a/sc/source/ui/sidebar/CellBorderStyleControl.cxx b/sc/source/ui/sidebar/CellBorderStyleControl.cxx
index d460b4d0..6749588 100644
--- a/sc/source/ui/sidebar/CellBorderStyleControl.cxx
+++ b/sc/source/ui/sidebar/CellBorderStyleControl.cxx
@@ -73,7 +73,7 @@ IMPL_LINK(CellBorderStylePopup, TB1SelectHdl, const OString&, rId, void)
{
    SvxBoxItem          aBorderOuter( SID_ATTR_BORDER_OUTER );
    SvxBoxInfoItem      aBorderInner( SID_ATTR_BORDER_INNER );
    editeng::SvxBorderLine theDefLine(nullptr, 1);
    editeng::SvxBorderLine theDefLine(nullptr, SvxBorderLineWidth::Thin);
    editeng::SvxBorderLine *pLeft = nullptr, *pRight = nullptr, *pTop = nullptr, *pBottom = nullptr;
    sal_uInt8 nValidFlags = 0;

@@ -132,7 +132,7 @@ IMPL_LINK(CellBorderStylePopup, TB2and3SelectHdl, const OString&, rId, void)
{
    if (rId == "diagup")
    {
        editeng::SvxBorderLine aTmp( nullptr, 1 );
        editeng::SvxBorderLine aTmp( nullptr, SvxBorderLineWidth::Thin );
        SvxLineItem     aLineItem( SID_ATTR_BORDER_DIAG_BLTR );
        aLineItem.SetLine( &aTmp );
        mpDispatcher->ExecuteList(
@@ -140,7 +140,7 @@ IMPL_LINK(CellBorderStylePopup, TB2and3SelectHdl, const OString&, rId, void)
    }
    else if (rId == "diagdown")
    {
        editeng::SvxBorderLine aTmp( nullptr, 1 );
        editeng::SvxBorderLine aTmp( nullptr, SvxBorderLineWidth::Thin );
        SvxLineItem     aLineItem( SID_ATTR_BORDER_DIAG_TLBR );
        aLineItem.SetLine( &aTmp );
        mpDispatcher->ExecuteList(
@@ -150,7 +150,7 @@ IMPL_LINK(CellBorderStylePopup, TB2and3SelectHdl, const OString&, rId, void)
    {
        SvxBoxItem          aBorderOuter( SID_ATTR_BORDER_OUTER );
        SvxBoxInfoItem      aBorderInner( SID_ATTR_BORDER_INNER );
        editeng::SvxBorderLine theDefLine(nullptr, 1);
        editeng::SvxBorderLine theDefLine(nullptr, SvxBorderLineWidth::Thin);
        editeng::SvxBorderLine       *pLeft = nullptr,
                            *pRight = nullptr,
                            *pTop = nullptr,
@@ -241,7 +241,7 @@ IMPL_LINK(CellBorderStylePopup, TB4SelectHdl, const OString&, rId, void)
    else if (rId == "topthickbottom")
    {
        pBottom.reset(new editeng::SvxBorderLine(nullptr, SvxBorderLineWidth::Thick));
        pTop.reset(new editeng::SvxBorderLine(nullptr, 1));
        pTop.reset(new editeng::SvxBorderLine(nullptr, SvxBorderLineWidth::Thin));
        nValidFlags |= FRM_VALID_BOTTOM|FRM_VALID_TOP;
    }
    else if (rId == "topdoublebottom")
@@ -249,7 +249,7 @@ IMPL_LINK(CellBorderStylePopup, TB4SelectHdl, const OString&, rId, void)
        pBottom.reset(new editeng::SvxBorderLine(nullptr));
        pBottom->GuessLinesWidths(SvxBorderLineStyle::DOUBLE, SvxBorderLineWidth::Hairline,
                                  SvxBorderLineWidth::Hairline, SvxBorderLineWidth::Thin);
        pTop.reset(new editeng::SvxBorderLine(nullptr, 1));
        pTop.reset(new editeng::SvxBorderLine(nullptr, SvxBorderLineWidth::Thin));
        nValidFlags |= FRM_VALID_BOTTOM|FRM_VALID_TOP;
    }