You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openoffice.apache.org by or...@apache.org on 2013/05/14 14:10:05 UTC
svn commit: r1482314 - in
/openoffice/trunk/main/svx/source/sidebar/paragraph:
ParaLineSpacingControl.cxx ParaLineSpacingControl.hxx ParaPropertyPanel.hrc
Author: orw
Date: Tue May 14 12:10:04 2013
New Revision: 1482314
URL: http://svn.apache.org/r1482314
Log:
122270: paragraph property panel, line spacing control: do not directly select item via cursor travelling; removing obsolete code
Modified:
openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx
openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.hxx
openoffice/trunk/main/svx/source/sidebar/paragraph/ParaPropertyPanel.hrc
Modified: openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx
URL: http://svn.apache.org/viewvc/openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx?rev=1482314&r1=1482313&r2=1482314&view=diff
==============================================================================
--- openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx (original)
+++ openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.cxx Tue May 14 12:10:04 2013
@@ -67,7 +67,6 @@ maSpacing1 (SVX_RES(IMG_SPACING1))
maSpacing115 (SVX_RES(IMG_SPACING115)),
maSpacing15 (SVX_RES(IMG_SPACING15)),
maSpacing2 (SVX_RES(IMG_SPACING2)),
-maLPCustom (SVX_RES(IMG_SPACINGLCUSTOM)),
maSelSpacing1 (SVX_RES(IMG_SEL_SPACING1)),
maSelSpacing115 (SVX_RES(IMG_SEL_SPACING115)),
maSelSpacing15 (SVX_RES(IMG_SEL_SPACING15)),
@@ -105,76 +104,64 @@ ParaLineSpacingControl::~ParaLineSpacing
void ParaLineSpacingControl::initial()
{
- //maLineSpacing.SetStyle( maLineSpacing.GetStyle()| WB_3DLOOK | WB_NO_DIRECTSELECT );
- //maLineSpacing.SetControlBackground(GetSettings().GetStyleSettings().GetMenuColor());
- //maLineSpacing.SetColor(GetSettings().GetStyleSettings().GetMenuColor());
- //maLineSpacing.SetMinFont();
-
- maLineSpacing.SetControlBackground(GetSettings().GetStyleSettings().GetHighContrastMode()?
- GetSettings().GetStyleSettings().GetMenuColor():
- sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
- maLineSpacing.SetColor(GetSettings().GetStyleSettings().GetHighContrastMode()?
- GetSettings().GetStyleSettings().GetMenuColor():
- sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
- maLineSpacing.SetBackground(GetSettings().GetStyleSettings().GetHighContrastMode()?
- GetSettings().GetStyleSettings().GetMenuColor():
- sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
- //maFTSpacing.SetBackground(GetSettings().GetStyleSettings().GetHighContrastMode()?
- //GetSettings().GetStyleSettings().GetMenuColor():
- //sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
- //maFTBy.SetBackground(GetSettings().GetStyleSettings().GetHighContrastMode()?
- //GetSettings().GetStyleSettings().GetMenuColor():
- //sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
-
- mpImg = new Image[4];
- mpImg[0] = maSpacing1;
- mpImg[1] = maSpacing115;
- mpImg[2] = maSpacing15;
- mpImg[3] = maSpacing2;
-
- mpImgSel = new Image[4];
- mpImgSel[0] = maSelSpacing1;
- mpImgSel[1] = maSelSpacing115;
- mpImgSel[2] = maSelSpacing15;
- mpImgSel[3] = maSelSpacing2;
-
- mpStr = new XubString[4];
- mpStr[0] = XubString(SVX_RES(STR_SPACING1));
- mpStr[1] = XubString(SVX_RES(STR_SPACING115));
- mpStr[2] = XubString(SVX_RES(STR_SPACING15));
- mpStr[3] = XubString(SVX_RES(STR_SPACING2));
-
- mpStrTip = new XubString[4];
- mpStrTip[0] = XubString(SVX_RES(STR_LS_SINGLE));
- mpStrTip[1] = XubString(SVX_RES(STR_LS_115));
- mpStrTip[2] = XubString(SVX_RES(STR_LS_15));
- mpStrTip[3] = XubString(SVX_RES(STR_LS_DOUBLE));
- for (int i=0;i<4;i++)
- maLineSpacing.AddItem(mpImg[i], &mpImgSel[i],mpStr[i],&mpStrTip[i]);
-
- maLineSpacing.AddItem( maImgCus, 0, maStrCus, 0 );
-
- SetAllNoSel();
- Link aLink = LINK(this, ParaLineSpacingControl,VSSelHdl );
- maLineSpacing.SetSelectHdl(aLink);
- maLineSpacing.StartSelection();
- maLineSpacing.Show();
+ maLineSpacing.SetStyle( maLineSpacing.GetStyle()| WB_3DLOOK | WB_NO_DIRECTSELECT );
- aLink = LINK( this, ParaLineSpacingControl, LineSPDistHdl_Impl );
- aLineDist.SetSelectHdl(aLink);
- aLineDist.SelectEntryPos( LLINESPACE_1 ) ;
- aLink = LINK( this, ParaLineSpacingControl, LineSPDistAtHdl_Impl );
- aLineDistAtPercentBox.SetModifyHdl( aLink );
- aLineDistAtMetricBox.SetModifyHdl( aLink );
-}
-void ParaLineSpacingControl::ToGetFocus(bool bType)
-{
- if(!bType)
- aLineDist.GrabFocus();
- else
- maLineSpacing.GrabFocus(); //wj
+ maLineSpacing.SetControlBackground(
+ GetSettings().GetStyleSettings().GetHighContrastMode()
+ ? GetSettings().GetStyleSettings().GetMenuColor()
+ : sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
+ maLineSpacing.SetColor(
+ GetSettings().GetStyleSettings().GetHighContrastMode()
+ ? GetSettings().GetStyleSettings().GetMenuColor()
+ : sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
+ maLineSpacing.SetBackground(
+ GetSettings().GetStyleSettings().GetHighContrastMode()
+ ? GetSettings().GetStyleSettings().GetMenuColor()
+ : sfx2::sidebar::Theme::GetColor( sfx2::sidebar::Theme::Paint_PanelBackground ));
+
+ mpImg = new Image[4];
+ mpImg[0] = maSpacing1;
+ mpImg[1] = maSpacing115;
+ mpImg[2] = maSpacing15;
+ mpImg[3] = maSpacing2;
+
+ mpImgSel = new Image[4];
+ mpImgSel[0] = maSelSpacing1;
+ mpImgSel[1] = maSelSpacing115;
+ mpImgSel[2] = maSelSpacing15;
+ mpImgSel[3] = maSelSpacing2;
+
+ mpStr = new XubString[4];
+ mpStr[0] = XubString(SVX_RES(STR_SPACING1));
+ mpStr[1] = XubString(SVX_RES(STR_SPACING115));
+ mpStr[2] = XubString(SVX_RES(STR_SPACING15));
+ mpStr[3] = XubString(SVX_RES(STR_SPACING2));
+
+ mpStrTip = new XubString[4];
+ mpStrTip[0] = XubString(SVX_RES(STR_LS_SINGLE));
+ mpStrTip[1] = XubString(SVX_RES(STR_LS_115));
+ mpStrTip[2] = XubString(SVX_RES(STR_LS_15));
+ mpStrTip[3] = XubString(SVX_RES(STR_LS_DOUBLE));
+ for (int i=0;i<4;i++)
+ maLineSpacing.AddItem(mpImg[i], &mpImgSel[i],mpStr[i],&mpStrTip[i]);
+
+ maLineSpacing.AddItem( maImgCus, 0, maStrCus, 0 );
+
+ SetAllNoSel();
+ Link aLink = LINK(this, ParaLineSpacingControl,VSSelHdl );
+ maLineSpacing.SetSelectHdl(aLink);
+ maLineSpacing.StartSelection();
+ maLineSpacing.Show();
+
+ aLink = LINK( this, ParaLineSpacingControl, LineSPDistHdl_Impl );
+ aLineDist.SetSelectHdl(aLink);
+ aLineDist.SelectEntryPos( LLINESPACE_1 ) ;
+ aLink = LINK( this, ParaLineSpacingControl, LineSPDistAtHdl_Impl );
+ aLineDistAtPercentBox.SetModifyHdl( aLink );
+ aLineDistAtMetricBox.SetModifyHdl( aLink );
}
+
void ParaLineSpacingControl::PopupModeEndCallback()
{
if( mbUseLineSPCustom )
@@ -484,24 +471,6 @@ void ParaLineSpacingControl::Rearrange(S
aLineDist.SaveValue();
- /*sal_uInt16 nID = pBox->GetCurItemId();
- pBox->SetItemDown(nID, sal_True);
-
- Size aFloatSz = GetOutputSizePixel();
-
- GetLineSPFloatWin()->SetSizePixel( aFloatSz );
-
- Point aPos=maLineSPTbx->GetPosPixel();
- aPos.setX(aPos.getX());
- aPos = OutputToScreenPixel( aPos );
- Size aSize = maLineSPTbx->GetSizePixel();
- Rectangle aRect( aPos, aSize );
-
- GetLineSPFloatWin()->StartPopupMode( aRect, FLOATWIN_POPUPMODE_DOWN | FLOATWIN_POPUPMODE_NOFOCUSCLOSE);
- GetLineSPFloatWin()->SetPopupModeFlags(GetLineSPFloatWin()->GetPopupModeFlags() | FLOATWIN_POPUPMODE_NOAPPFOCUSCLOSE );
-
- pLineSPPage->ToGetFocus(bValueSetFocus);
- */
sal_uInt16 uCount = aLineDist.GetEntryCount();
if( uCount == LLINESPACE_FIX + 1 )
{
Modified: openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.hxx
URL: http://svn.apache.org/viewvc/openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.hxx?rev=1482314&r1=1482313&r2=1482314&view=diff
==============================================================================
--- openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.hxx (original)
+++ openoffice/trunk/main/svx/source/sidebar/paragraph/ParaLineSpacingControl.hxx Tue May 14 12:10:04 2013
@@ -43,7 +43,6 @@ class ParaLineSpacingControl:public svx:
public:
ParaLineSpacingControl(Window* pParent, svx::sidebar::ParaPropertyPanel& rPanel);
~ParaLineSpacingControl();
- void ToGetFocus(bool bType);
void Rearrange(SfxItemState currSPState,FieldUnit currMetricUnit,SvxLineSpacingItem* currSPItem,const ::sfx2::sidebar::EnumContext currentContext);
//virtual void Paint(const Rectangle& rect);
@@ -84,7 +83,6 @@ private:
Image maSpacing115;
Image maSpacing15;
Image maSpacing2;
- Image maLPCustom;
Image maSelSpacing1;
Image maSelSpacing115;
Modified: openoffice/trunk/main/svx/source/sidebar/paragraph/ParaPropertyPanel.hrc
URL: http://svn.apache.org/viewvc/openoffice/trunk/main/svx/source/sidebar/paragraph/ParaPropertyPanel.hrc?rev=1482314&r1=1482313&r2=1482314&view=diff
==============================================================================
--- openoffice/trunk/main/svx/source/sidebar/paragraph/ParaPropertyPanel.hrc (original)
+++ openoffice/trunk/main/svx/source/sidebar/paragraph/ParaPropertyPanel.hrc Tue May 14 12:10:04 2013
@@ -119,7 +119,6 @@
#define IMG_SPACING115 19
#define IMG_SPACING15 20
#define IMG_SPACING2 21
-#define IMG_SPACINGLCUSTOM 22
#define IMG_SEL_SPACING1 23
#define IMG_SEL_SPACING115 24
#define IMG_SEL_SPACING15 25