From c0baab9aa80bfa49b753d2d598a4fbc71ceb656e Mon Sep 17 00:00:00 2001 From: Justin Luth Date: Fri, 28 Oct 2016 19:40:23 +0300 Subject: [PATCH] reset border should clear all borders If no borders were originally defined, the reset button was failing to clear the last-selected border line. Eg. page border, click on two of the borders, then hit reset. One border remains defined, and didn't reset to the starting value of non-visible. Because one border was still "selected", it became visible again as SelStyleHdl_Impl reinitialized. Change-Id: I2d20ae1e02816d5b30fe126572b54e7083114fc3 Reviewed-on: https://gerrit.libreoffice.org/30365 Tested-by: Jenkins Reviewed-by: Justin Luth --- cui/source/tabpages/border.cxx | 2 ++ 1 file changed, 2 insertions(+) diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx index 2a2a32065d9a..05c8914abc5c 100644 --- a/cui/source/tabpages/border.cxx +++ b/cui/source/tabpages/border.cxx @@ -527,6 +527,8 @@ void SvxBorderTabPage::Reset( const SfxItemSet* rSet ) m_pFrameSel->HideAllBorders(); } + if( !m_pFrameSel->IsAnyBorderVisible() ) + m_pFrameSel->DeselectAllBorders(); // depict line (color) in controllers if unambiguous: