[Libreoffice-commits] core.git: Branch 'libreoffice-5-2' - sw/source

2016-06-07 Thread Rodolfo Ribeiro Gomes
 sw/source/uibase/shells/tabsh.cxx |9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

New commits:
commit 0006c15d8a496b7544beb09ba4de6bde866c13b0
Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
Date:   Sun May 8 12:05:58 2016 -0300

tdf#34362 Setting table border from toolbar resets "spacing to content"

Change-Id: Iec51c02ab389015d268849aa0dda870c11323f46
    Signed-off-by: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/24764
Tested-by: Jenkins <c...@libreoffice.org>
Reviewed-by: Michael Stahl <mst...@redhat.com>
(cherry picked from commit ea7de7b2d3dc62b2ae73c2184398b227f8a85890)
Reviewed-on: https://gerrit.libreoffice.org/25982
Reviewed-by: Caolán McNamara <caol...@redhat.com>
Tested-by: Caolán McNamara <caol...@redhat.com>

diff --git a/sw/source/uibase/shells/tabsh.cxx 
b/sw/source/uibase/shells/tabsh.cxx
index ef4d98a..aee1703 100644
--- a/sw/source/uibase/shells/tabsh.cxx
+++ b/sw/source/uibase/shells/tabsh.cxx
@@ -46,6 +46,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -480,10 +481,12 @@ void SwTableShell::Execute(SfxRequest )
 if ( pArgs->GetItemState(RES_BOX, true, ) == 
SfxItemState::SET )
 {
 aBox = *static_cast(pBoxItem);
+sal_uInt16 nDefValue = MIN_BORDER_DIST;
 if ( !rReq.IsAPI() )
-aBox.SetDistance( 
std::max(rCoreBox.GetDistance(),sal_uInt16(55)) );
-else if ( aBox.GetDistance() < MIN_BORDER_DIST )
-aBox.SetDistance( 
std::max(rCoreBox.GetDistance(),(sal_uInt16)MIN_BORDER_DIST)  );
+nDefValue = 55;
+if ( !rReq.IsAPI() || aBox.GetDistance() < MIN_BORDER_DIST )
+for( SvxBoxItemLine k : o3tl::enumrange() )
+aBox.SetDistance( std::max(rCoreBox.GetDistance(k), 
nDefValue) , k );
 }
 else
 OSL_ENSURE( false, "where is BoxItem?" );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/source

2016-05-30 Thread Rodolfo Ribeiro Gomes
 sw/source/uibase/shells/tabsh.cxx |9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

New commits:
commit ea7de7b2d3dc62b2ae73c2184398b227f8a85890
Author: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
Date:   Sun May 8 12:05:58 2016 -0300

fdo#34362 Setting table border from toolbar resets "spacing to content"

Change-Id: Iec51c02ab389015d268849aa0dda870c11323f46
    Signed-off-by: Rodolfo Ribeiro Gomes <rodolf...@gmail.com>
Reviewed-on: https://gerrit.libreoffice.org/24764
Tested-by: Jenkins <c...@libreoffice.org>
Reviewed-by: Michael Stahl <mst...@redhat.com>

diff --git a/sw/source/uibase/shells/tabsh.cxx 
b/sw/source/uibase/shells/tabsh.cxx
index ac84b52..c8b9afd 100644
--- a/sw/source/uibase/shells/tabsh.cxx
+++ b/sw/source/uibase/shells/tabsh.cxx
@@ -46,6 +46,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -487,10 +488,12 @@ void SwTableShell::Execute(SfxRequest )
 if ( pArgs->GetItemState(RES_BOX, true, ) == 
SfxItemState::SET )
 {
 aBox = *static_cast(pBoxItem);
+sal_uInt16 nDefValue = MIN_BORDER_DIST;
 if ( !rReq.IsAPI() )
-aBox.SetDistance( 
std::max(rCoreBox.GetDistance(),sal_uInt16(55)) );
-else if ( aBox.GetDistance() < MIN_BORDER_DIST )
-aBox.SetDistance( 
std::max(rCoreBox.GetDistance(),(sal_uInt16)MIN_BORDER_DIST)  );
+nDefValue = 55;
+if ( !rReq.IsAPI() || aBox.GetDistance() < MIN_BORDER_DIST )
+for( SvxBoxItemLine k : o3tl::enumrange() )
+aBox.SetDistance( std::max(rCoreBox.GetDistance(k), 
nDefValue) , k );
 }
 else
 OSL_ENSURE( false, "where is BoxItem?" );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sd/uiconfig

2015-02-12 Thread Rodolfo Ribeiro Gomes
 sd/uiconfig/sdraw/statusbar/statusbar.xml |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 605494fd99ae8e54611899729783d1163cf9a5ba
Author: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Date:   Sat Feb 7 21:14:38 2015 -0200

fix tdf#85027 Add button to StatusBar: Fit page to window

Change-Id: Ibc6d2b8d3699caa0ba2baa51dbbf357f935fe5b2
Signed-off-by: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Reviewed-on: https://gerrit.libreoffice.org/14363
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sd/uiconfig/sdraw/statusbar/statusbar.xml 
b/sd/uiconfig/sdraw/statusbar/statusbar.xml
index 24224fe..78042ef 100644
--- a/sd/uiconfig/sdraw/statusbar/statusbar.xml
+++ b/sd/uiconfig/sdraw/statusbar/statusbar.xml
@@ -24,6 +24,7 @@
  statusbar:statusbaritem xlink:href=.uno:Signature statusbar:align=center 
statusbar:ownerdraw=true statusbar:width=16/
  statusbar:statusbaritem xlink:href=.uno:PageStatus statusbar:align=left 
statusbar:width=124/
  statusbar:statusbaritem xlink:href=.uno:LayoutStatus 
statusbar:align=left statusbar:autosize=true statusbar:width=54/
+ statusbar:statusbaritem xlink:href=.uno:ZoomPage statusbar:align=center 
statusbar:ownerdraw=true statusbar:width=9/
  statusbar:statusbaritem xlink:href=.uno:ZoomSlider 
statusbar:align=center statusbar:ownerdraw=true statusbar:width=130/
  statusbar:statusbaritem xlink:href=.uno:Zoom statusbar:align=center 
statusbar:width=36/
 /statusbar:statusbar
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sd/source

2015-02-12 Thread Rodolfo Ribeiro Gomes
 sd/source/ui/view/viewshel.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit 2d5d52c2abc9fa882b1350a522f2989933fcb67b
Author: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Date:   Mon Feb 2 03:45:51 2015 -0200

draw/impress - zoom in/out to the mouse position

Change-Id: I2bff851c683fdffe9a6074fedc2f2151650f79da
Signed-off-by: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Reviewed-on: https://gerrit.libreoffice.org/14307
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sd/source/ui/view/viewshel.cxx b/sd/source/ui/view/viewshel.cxx
index a78c68f..83dc9ca 100644
--- a/sd/source/ui/view/viewshel.cxx
+++ b/sd/source/ui/view/viewshel.cxx
@@ -648,6 +648,7 @@ bool ViewShell::HandleScrollCommand(const CommandEvent 
rCEvt, ::sd::Window* pWi
 {
 const long  nOldZoom = GetActiveWindow()-GetZoom();
 longnNewZoom;
+Point aOldMousePos = 
GetActiveWindow()-PixelToLogic(rCEvt.GetMousePosPixel());
 
 if( pData-GetDelta()  0L )
 nNewZoom = std::max( (long) pWin-GetMinZoom(), 
basegfx::zoomtools::zoomOut( nOldZoom ));
@@ -655,6 +656,10 @@ bool ViewShell::HandleScrollCommand(const CommandEvent 
rCEvt, ::sd::Window* pWi
 nNewZoom = std::min( (long) pWin-GetMaxZoom(), 
basegfx::zoomtools::zoomIn( nOldZoom ));
 
 SetZoom( nNewZoom );
+// Keep mouse at same doc point before zoom
+Point aNewMousePos = 
GetActiveWindow()-PixelToLogic(rCEvt.GetMousePosPixel());
+SetWinViewPos(GetWinViewPos() - (aNewMousePos - 
aOldMousePos), true);
+
 Invalidate( SID_ATTR_ZOOM );
 Invalidate( SID_ATTR_ZOOMSLIDER );
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sd/source

2015-02-12 Thread Rodolfo Ribeiro Gomes
 sd/source/ui/app/strings.src  |6 ++
 sd/source/ui/app/tmplctrl.cxx |3 +++
 sd/source/ui/inc/strings.hrc  |2 ++
 3 files changed, 11 insertions(+)

New commits:
commit 45b881801785d6bb6df1304732243e87c918c7c9
Author: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Date:   Tue Feb 3 02:08:25 2015 -0200

partial fix tdf#86611 (item #5) Draw statusbar tip

Change-Id: I5096c5ea1272e16969f9572ed40c60cdb9378b57
Signed-off-by: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Reviewed-on: https://gerrit.libreoffice.org/14308
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sd/source/ui/app/strings.src b/sd/source/ui/app/strings.src
index 3e11d18..991ee3a 100644
--- a/sd/source/ui/app/strings.src
+++ b/sd/source/ui/app/strings.src
@@ -995,6 +995,12 @@ String STR_PAGE_BACKGROUND_TXT
 {
 Text [ en-US ] = Background settings for all pages? ;
 };
+
+String STR_STATUSBAR_MASTERPAGE
+{
+Text [ en-US ] = Slide Master name. Right-click for list and double-click 
for dialog.;
+};
+
 String STR_TITLE_RENAMESLIDE
 {
 Text [ en-US ] = Rename Slide;
diff --git a/sd/source/ui/app/tmplctrl.cxx b/sd/source/ui/app/tmplctrl.cxx
index 15d8ec5..9f67052 100644
--- a/sd/source/ui/app/tmplctrl.cxx
+++ b/sd/source/ui/app/tmplctrl.cxx
@@ -28,6 +28,8 @@
 #include drawdoc.hxx
 #include sdattr.hrc
 #include app.hrc
+#include sdresid.hxx
+#include strings.hrc
 
 SFX_IMPL_STATUSBAR_CONTROL( SdTemplateControl, SfxStringItem );
 
@@ -64,6 +66,7 @@ SdTemplateControl::SdTemplateControl( sal_uInt16 _nSlotId,
   StatusBar rStb ) :
 SfxStatusBarControl( _nSlotId, _nId, rStb )
 {
+GetStatusBar().SetQuickHelpText(GetId(), 
SD_RESSTR(STR_STATUSBAR_MASTERPAGE));
 }
 
 SdTemplateControl::~SdTemplateControl()
diff --git a/sd/source/ui/inc/strings.hrc b/sd/source/ui/inc/strings.hrc
index 1fa282c..aa92fb3 100644
--- a/sd/source/ui/inc/strings.hrc
+++ b/sd/source/ui/inc/strings.hrc
@@ -336,6 +336,8 @@
 #define STR_PAGE_BACKGROUND_TXT (RID_APP_START+583)
 #define STR_PAGE_BACKGROUND_TITLE   (RID_APP_START+584)
 
+#define STR_STATUSBAR_MASTERPAGE(RID_APP_START+600)
+
 #define STR_TWAIN_NO_SOURCE (RID_APP_START+670)
 #define STR_UNDO_VECTORIZE  (RID_APP_START+671)
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-4' - include/svx svx/source

2015-02-09 Thread Rodolfo Ribeiro Gomes
 include/svx/zoomsliderctrl.hxx |1 +
 svx/source/stbctrls/zoomsliderctrl.cxx |   15 +--
 2 files changed, 14 insertions(+), 2 deletions(-)

New commits:
commit 0bc37cf20044fe8fee0cc7dd5af3ee58304a3a0f
Author: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Date:   Sat Jan 31 19:08:05 2015 -0200

tdf#47577  Zoom slider should only react to full click

Change-Id: If859754c956a634a96066ac407b70d6dd2759d53
Signed-off-by: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Reviewed-on: https://gerrit.libreoffice.org/14265
Tested-by: Jenkins c...@libreoffice.org
Reviewed-by: Michael Stahl mst...@redhat.com
(cherry picked from commit 53740bd71a0ae569da36b9293ef22842a0b53f1e)
Signed-off-by: Michael Stahl mst...@redhat.com

diff --git a/include/svx/zoomsliderctrl.hxx b/include/svx/zoomsliderctrl.hxx
index 7af69db..f35f825 100644
--- a/include/svx/zoomsliderctrl.hxx
+++ b/include/svx/zoomsliderctrl.hxx
@@ -49,6 +49,7 @@ public:
 virtual void  StateChanged( sal_uInt16 nSID, SfxItemState eState, const 
SfxPoolItem* pState ) SAL_OVERRIDE;
 virtual void  Paint( const UserDrawEvent rEvt ) SAL_OVERRIDE;
 virtual bool  MouseButtonDown( const MouseEvent  ) SAL_OVERRIDE;
+virtual bool  MouseButtonUp( const MouseEvent  ) SAL_OVERRIDE;
 virtual bool  MouseMove( const MouseEvent  rEvt ) SAL_OVERRIDE;
 };
 
diff --git a/svx/source/stbctrls/zoomsliderctrl.cxx 
b/svx/source/stbctrls/zoomsliderctrl.cxx
index 8ea241b..1379321 100644
--- a/svx/source/stbctrls/zoomsliderctrl.cxx
+++ b/svx/source/stbctrls/zoomsliderctrl.cxx
@@ -45,6 +45,7 @@ struct SvxZoomSliderControl::SvxZoomSliderControl_Impl
 ImagemaDecreaseButton;
 bool mbValuesSet;
 bool mbOmitPaint;
+bool mbDraggingStarted;
 
 SvxZoomSliderControl_Impl() :
 mnCurrentZoom( 0 ),
@@ -57,7 +58,8 @@ struct SvxZoomSliderControl::SvxZoomSliderControl_Impl
 maIncreaseButton(),
 maDecreaseButton(),
 mbValuesSet( false ),
-mbOmitPaint( false ) {}
+mbOmitPaint( false ),
+mbDraggingStarted( false ) {}
 };
 
 const long nSliderXOffset = 20;
@@ -330,7 +332,10 @@ bool SvxZoomSliderControl::MouseButtonDown( const 
MouseEvent  rEvt )
 mpImpl-mnCurrentZoom = basegfx::zoomtools::zoomIn( 
static_castint(mpImpl-mnCurrentZoom) );
 // click to slider
 else if( nXDiff = nSliderXOffset  nXDiff = aControlRect.GetWidth() - 
nSliderXOffset )
+{
 mpImpl-mnCurrentZoom = Offset2Zoom( nXDiff );
+mpImpl-mbDraggingStarted = true;
+}
 
 if ( mpImpl-mnCurrentZoom  mpImpl-mnMinZoom )
 mpImpl-mnCurrentZoom = mpImpl-mnMinZoom;
@@ -345,6 +350,12 @@ bool SvxZoomSliderControl::MouseButtonDown( const 
MouseEvent  rEvt )
 return true;
 }
 
+bool SvxZoomSliderControl::MouseButtonUp( const MouseEvent  )
+{
+mpImpl-mbDraggingStarted = false;
+return true;
+}
+
 bool SvxZoomSliderControl::MouseMove( const MouseEvent  rEvt )
 {
 if ( !mpImpl-mbValuesSet )
@@ -356,7 +367,7 @@ bool SvxZoomSliderControl::MouseMove( const MouseEvent  
rEvt )
 const sal_Int32 nXDiff = aPoint.X() - aControlRect.Left();
 
 // check mouse move with button pressed
-if ( 1 == nButtons )
+if ( 1 == nButtons  mpImpl-mbDraggingStarted )
 {
 if ( nXDiff = nSliderXOffset  nXDiff = aControlRect.GetWidth() - 
nSliderXOffset )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: include/sfx2 include/svx svx/source

2015-02-09 Thread Rodolfo Ribeiro Gomes
 include/sfx2/stbitem.hxx   |2 +-
 include/svx/zoomsliderctrl.hxx |1 +
 svx/source/stbctrls/zoomsliderctrl.cxx |   15 +--
 3 files changed, 15 insertions(+), 3 deletions(-)

New commits:
commit 53740bd71a0ae569da36b9293ef22842a0b53f1e
Author: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Date:   Sat Jan 31 19:08:05 2015 -0200

tdf#47577  Zoom slider should only react to full click

Change-Id: If859754c956a634a96066ac407b70d6dd2759d53
Signed-off-by: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br
Reviewed-on: https://gerrit.libreoffice.org/14265
Tested-by: Jenkins c...@libreoffice.org
Reviewed-by: Michael Stahl mst...@redhat.com

diff --git a/include/sfx2/stbitem.hxx b/include/sfx2/stbitem.hxx
index 07121f8..f5e146c 100644
--- a/include/sfx2/stbitem.hxx
+++ b/include/sfx2/stbitem.hxx
@@ -102,7 +102,7 @@ protected:
 virtual voidCommand( const CommandEvent rCEvt );
 virtual boolMouseButtonDown( const MouseEvent  );
 virtual boolMouseMove( const MouseEvent  );
-boolMouseButtonUp( const MouseEvent  );
+virtual boolMouseButtonUp( const MouseEvent  );
 virtual voidPaint( const UserDrawEvent rUDEvt );
 
 static sal_uInt16   convertAwtToVCLMouseButtons( sal_Int16 
nAwtMouseButtons );
diff --git a/include/svx/zoomsliderctrl.hxx b/include/svx/zoomsliderctrl.hxx
index 7af69db..f35f825 100644
--- a/include/svx/zoomsliderctrl.hxx
+++ b/include/svx/zoomsliderctrl.hxx
@@ -49,6 +49,7 @@ public:
 virtual void  StateChanged( sal_uInt16 nSID, SfxItemState eState, const 
SfxPoolItem* pState ) SAL_OVERRIDE;
 virtual void  Paint( const UserDrawEvent rEvt ) SAL_OVERRIDE;
 virtual bool  MouseButtonDown( const MouseEvent  ) SAL_OVERRIDE;
+virtual bool  MouseButtonUp( const MouseEvent  ) SAL_OVERRIDE;
 virtual bool  MouseMove( const MouseEvent  rEvt ) SAL_OVERRIDE;
 };
 
diff --git a/svx/source/stbctrls/zoomsliderctrl.cxx 
b/svx/source/stbctrls/zoomsliderctrl.cxx
index dcb0c55..c2e0f21 100644
--- a/svx/source/stbctrls/zoomsliderctrl.cxx
+++ b/svx/source/stbctrls/zoomsliderctrl.cxx
@@ -45,6 +45,7 @@ struct SvxZoomSliderControl::SvxZoomSliderControl_Impl
 ImagemaDecreaseButton;
 bool mbValuesSet;
 bool mbOmitPaint;
+bool mbDraggingStarted;
 
 SvxZoomSliderControl_Impl() :
 mnCurrentZoom( 0 ),
@@ -57,7 +58,8 @@ struct SvxZoomSliderControl::SvxZoomSliderControl_Impl
 maIncreaseButton(),
 maDecreaseButton(),
 mbValuesSet( false ),
-mbOmitPaint( false ) {}
+mbOmitPaint( false ),
+mbDraggingStarted( false ) {}
 };
 
 const long nSliderXOffset = 20;
@@ -331,7 +333,10 @@ bool SvxZoomSliderControl::MouseButtonDown( const 
MouseEvent  rEvt )
 mpImpl-mnCurrentZoom = basegfx::zoomtools::zoomIn( 
static_castint(mpImpl-mnCurrentZoom) );
 // click to slider
 else if( nXDiff = nSliderXOffset  nXDiff = aControlRect.GetWidth() - 
nSliderXOffset )
+{
 mpImpl-mnCurrentZoom = Offset2Zoom( nXDiff );
+mpImpl-mbDraggingStarted = true;
+}
 
 if ( mpImpl-mnCurrentZoom  mpImpl-mnMinZoom )
 mpImpl-mnCurrentZoom = mpImpl-mnMinZoom;
@@ -346,6 +351,12 @@ bool SvxZoomSliderControl::MouseButtonDown( const 
MouseEvent  rEvt )
 return true;
 }
 
+bool SvxZoomSliderControl::MouseButtonUp( const MouseEvent  )
+{
+mpImpl-mbDraggingStarted = false;
+return true;
+}
+
 bool SvxZoomSliderControl::MouseMove( const MouseEvent  rEvt )
 {
 if ( !mpImpl-mbValuesSet )
@@ -357,7 +368,7 @@ bool SvxZoomSliderControl::MouseMove( const MouseEvent  
rEvt )
 const sal_Int32 nXDiff = aPoint.X() - aControlRect.Left();
 
 // check mouse move with button pressed
-if ( 1 == nButtons )
+if ( 1 == nButtons  mpImpl-mbDraggingStarted )
 {
 if ( nXDiff = nSliderXOffset  nXDiff = aControlRect.GetWidth() - 
nSliderXOffset )
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/49/3949/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/95/6695/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/48/1348/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/49/3949/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/13/4113/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/26/6626/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/95/6695/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/13/4113/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/17/4117/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/52/1452/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/17/4117/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/76/4076/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/52/3952/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/13/4113/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/17/4117/4'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/52/3952/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/53/3953/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/76/4076/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/52/1452/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/53/3953/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/17/4117/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/29/6629/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/93/6693/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/06/8506/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/29/6629/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/8000/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/53/3953/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/43/5943/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/06/8506/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/37/6537/4'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/51/3951/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/51/3951/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/85/1785/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/85/1785/4'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/12/1412/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/51/3951/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/85/1785/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/16/6716/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/50/3950/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/37/6537/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/85/1785/5'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/85/1785/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/10/6610/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/12/1412/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/50/3950/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/16/6716/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/51/3951/4'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/07/6707/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/38/6538/3'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/07/6707/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/54/7454/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/38/6538/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/06/6706/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/06/6706/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/15/1415/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/43/6643/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/43/6643/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/10/6810/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/54/7454/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/16/1416/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/10/6810/2'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/25/1425/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/38/6538/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/16/1416/1'

2014-09-29 Thread Rodolfo Ribeiro Gomes

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: officecfg/registry

2014-02-18 Thread Rodolfo Ribeiro Gomes
 officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu |   10 
--
 1 file changed, 10 deletions(-)

New commits:
commit 2c9c54285b952b59b5262500089c9c1a7902deb2
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Tue Feb 11 21:49:08 2014 -0200

Removes bogus UNO command AcceptChange and RejectChange

These are not real commands. The real ones are:
- AcceptTracedChange
- RejectTracedChange

Change-Id: I352cea4a5ffec73947932264783b61ae044f25cf
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/8000
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
index 3cb9be4..0e871e4 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
@@ -272,16 +272,6 @@
   value xml:lang=en-USCopy Hyperlink Location/value
 /prop
   /node
-  node oor:name=.uno:AcceptChange oor:op=replace
-prop oor:name=Label oor:type=xs:string
-  value xml:lang=en-USAccept Change/value
-/prop
-  /node
-  node oor:name=.uno:RejectChange oor:op=replace
-prop oor:name=Label oor:type=xs:string
-  value xml:lang=en-USReject Change/value
-/prop
-  /node
   node oor:name=.uno:InsertBookmark oor:op=replace
 prop oor:name=Label oor:type=xs:string
   value xml:lang=en-USBookmar~k.../value
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: icon-themes/crystal icon-themes/galaxy icon-themes/hicontrast icon-themes/human icon-themes/industrial icon-themes/oxygen icon-themes/tango icon-themes/tango_testing

2014-02-10 Thread Rodolfo Ribeiro Gomes
 dev/null  |binary
 icon-themes/crystal/cmd/lc_zoomoptimal.png|binary
 icon-themes/crystal/cmd/sc_zoomoptimal.png|binary
 icon-themes/human/cmd/lc_zoomoptimal.png  |binary
 icon-themes/human/cmd/sc_zoomoptimal.png  |binary
 icon-themes/industrial/cmd/lc_zoomoptimal.png |binary
 icon-themes/industrial/cmd/lc_zoomoptimal.xcf.bz2 |binary
 icon-themes/industrial/cmd/sc_zoomoptimal.png |binary
 icon-themes/oxygen/cmd/lc_zoomoptimal.png |binary
 icon-themes/oxygen/cmd/sc_zoomoptimal.png |binary
 icon-themes/tango/cmd/lc_zoomoptimal.png  |binary
 icon-themes/tango/cmd/lc_zoomoptimal.xcf.bz2  |binary
 icon-themes/tango_testing/cmd/lc_zoomoptimal.png  |binary
 13 files changed

New commits:
commit aee6ad3022b4ba4a19e13cad81ab974ef4ba1724
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Fri May 31 07:52:55 2013 -0300

Remove unused icons after cleaning up Math zoom

After commit ffc2e5be1f712b09710e2096ad2f7eb81b80118d ,
these deleted icons are useless since other UNO commands are used instead:
  View050 - Zoom50Percent
  View100 - Zoom100Percent
  View200 - Zoom200Percent
  Adjust  - ZoomOptimal
  FitInWindow - ZoomOptimal

Sorry I forgot to remove then in that commit.

Note: As UNO command now is 'zoomoptimal' instead of 'adjust' and
people feel 'adjust' icon better, I deleted the disliked 'zoomoptimal'
icons and renamed 'adjust' icons to it.

Change-Id: I141f2c4ad2f63928efb15faa5f68bc9c6dabc144
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/4117
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/icon-themes/crystal/cmd/lc_view100.png 
b/icon-themes/crystal/cmd/lc_view100.png
deleted file mode 100644
index cc8ce15..000
Binary files a/icon-themes/crystal/cmd/lc_view100.png and /dev/null differ
diff --git a/icon-themes/crystal/cmd/lc_adjust.png 
b/icon-themes/crystal/cmd/lc_zoomoptimal.png
similarity index 100%
rename from icon-themes/crystal/cmd/lc_adjust.png
rename to icon-themes/crystal/cmd/lc_zoomoptimal.png
diff --git a/icon-themes/crystal/cmd/sc_view100.png 
b/icon-themes/crystal/cmd/sc_view100.png
deleted file mode 100644
index 8709d0a..000
Binary files a/icon-themes/crystal/cmd/sc_view100.png and /dev/null differ
diff --git a/icon-themes/crystal/cmd/sc_adjust.png 
b/icon-themes/crystal/cmd/sc_zoomoptimal.png
similarity index 100%
rename from icon-themes/crystal/cmd/sc_adjust.png
rename to icon-themes/crystal/cmd/sc_zoomoptimal.png
diff --git a/icon-themes/galaxy/cmd/lc_adjust.png 
b/icon-themes/galaxy/cmd/lc_adjust.png
deleted file mode 100644
index 228d958..000
Binary files a/icon-themes/galaxy/cmd/lc_adjust.png and /dev/null differ
diff --git a/icon-themes/galaxy/cmd/lc_view100.png 
b/icon-themes/galaxy/cmd/lc_view100.png
deleted file mode 100644
index 2f00d91..000
Binary files a/icon-themes/galaxy/cmd/lc_view100.png and /dev/null differ
diff --git a/icon-themes/galaxy/cmd/sc_adjust.png 
b/icon-themes/galaxy/cmd/sc_adjust.png
deleted file mode 100644
index d9d1a66..000
Binary files a/icon-themes/galaxy/cmd/sc_adjust.png and /dev/null differ
diff --git a/icon-themes/galaxy/cmd/sc_view100.png 
b/icon-themes/galaxy/cmd/sc_view100.png
deleted file mode 100644
index bfd1a58..000
Binary files a/icon-themes/galaxy/cmd/sc_view100.png and /dev/null differ
diff --git a/icon-themes/hicontrast/cmd/lc_adjust.png 
b/icon-themes/hicontrast/cmd/lc_adjust.png
deleted file mode 100644
index 60765d6..000
Binary files a/icon-themes/hicontrast/cmd/lc_adjust.png and /dev/null differ
diff --git a/icon-themes/hicontrast/cmd/lc_view100.png 
b/icon-themes/hicontrast/cmd/lc_view100.png
deleted file mode 100644
index 33af72d..000
Binary files a/icon-themes/hicontrast/cmd/lc_view100.png and /dev/null differ
diff --git a/icon-themes/hicontrast/cmd/sc_adjust.png 
b/icon-themes/hicontrast/cmd/sc_adjust.png
deleted file mode 100644
index fdaabf5..000
Binary files a/icon-themes/hicontrast/cmd/sc_adjust.png and /dev/null differ
diff --git a/icon-themes/hicontrast/cmd/sc_view100.png 
b/icon-themes/hicontrast/cmd/sc_view100.png
deleted file mode 100644
index 2254149..000
Binary files a/icon-themes/hicontrast/cmd/sc_view100.png and /dev/null differ
diff --git a/icon-themes/human/cmd/lc_adjust.png 
b/icon-themes/human/cmd/lc_adjust.png
deleted file mode 100644
index 4beea80..000
Binary files a/icon-themes/human/cmd/lc_adjust.png and /dev/null differ
diff --git a/icon-themes/human/cmd/lc_view100.png 
b/icon-themes/human/cmd/lc_view100.png
deleted file mode 100644
index 2c52084..000
Binary files a/icon-themes/human/cmd/lc_view100.png and /dev/null differ
diff --git a/icon-themes/human/cmd/lc_zoomoptimal.png 
b/icon-themes/human/cmd/lc_zoomoptimal.png
index 575284b..4beea80

[Libreoffice-commits] core.git: sw/source

2013-11-26 Thread Rodolfo Ribeiro Gomes
 sw/source/ui/dbui/mailmergechildwindow.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit eda8111fdf9b1c69935237106a73d1c53c2776e1
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon Nov 25 23:34:09 2013 -0200

Mailmerge shows wrong number of emails to be sent

The progress bar is ok, but the text is displayed with the total
number of registers instead of the amount selected to be used.

Change-Id: I81da0e7ba9f193366e12cbf81118e36a43bcaad4
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6810
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/ui/dbui/mailmergechildwindow.cxx 
b/sw/source/ui/dbui/mailmergechildwindow.cxx
index 682ab22..c1dc07a 100644
--- a/sw/source/ui/dbui/mailmergechildwindow.cxx
+++ b/sw/source/ui/dbui/mailmergechildwindow.cxx
@@ -648,7 +648,7 @@ void SwSendMailDialog::UpdateTransferStatus()
 {
 OUString sStatus( m_sTransferStatus );
 sStatus = sStatus.replaceFirst(%1, OUString::number(m_nSendCount) );
-sStatus = sStatus.replaceFirst(%2, 
OUString::number(m_pImpl-nDocumentCount));
+sStatus = sStatus.replaceFirst(%2, 
OUString::number(m_pImpl-aDescriptors.size()));
 m_aTransferStatusFT.SetText(sStatus);
 
 sStatus = m_sErrorStatus.replaceFirst(%1, 
OUString::number(m_nErrorCount) );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Branch 'libreoffice-4-2' - sw/source

2013-11-26 Thread Rodolfo Ribeiro Gomes
 sw/source/ui/dbui/mailmergechildwindow.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit aa3297af50b9a197428565bb927cf566a9198435
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon Nov 25 23:34:09 2013 -0200

Mailmerge shows wrong number of emails to be sent

The progress bar is ok, but the text is displayed with the total
number of registers instead of the amount selected to be used.

Change-Id: I81da0e7ba9f193366e12cbf81118e36a43bcaad4
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6810
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com
(cherry picked from commit eda8111fdf9b1c69935237106a73d1c53c2776e1)

diff --git a/sw/source/ui/dbui/mailmergechildwindow.cxx 
b/sw/source/ui/dbui/mailmergechildwindow.cxx
index 682ab22..c1dc07a 100644
--- a/sw/source/ui/dbui/mailmergechildwindow.cxx
+++ b/sw/source/ui/dbui/mailmergechildwindow.cxx
@@ -648,7 +648,7 @@ void SwSendMailDialog::UpdateTransferStatus()
 {
 OUString sStatus( m_sTransferStatus );
 sStatus = sStatus.replaceFirst(%1, OUString::number(m_nSendCount) );
-sStatus = sStatus.replaceFirst(%2, 
OUString::number(m_pImpl-nDocumentCount));
+sStatus = sStatus.replaceFirst(%2, 
OUString::number(m_pImpl-aDescriptors.size()));
 m_aTransferStatusFT.SetText(sStatus);
 
 sStatus = m_sErrorStatus.replaceFirst(%1, 
OUString::number(m_nErrorCount) );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: editeng/source

2013-11-20 Thread Rodolfo Ribeiro Gomes
 editeng/source/editeng/impedit.hxx  |4 -
 editeng/source/editeng/impedit3.cxx |  140 ++--
 editeng/source/editeng/impedit4.cxx |2 
 3 files changed, 73 insertions(+), 73 deletions(-)

New commits:
commit 142e89e6c2f5ae292cf70bffaf2d9dbc8c9a2ecd
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sun Sep 15 20:24:36 2013 -0300

sal_Bool to bool in ImpEditEngine

Change-Id: I76dc0343fd735104827b4803c5440570afe86174
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6693
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/editeng/source/editeng/impedit.hxx 
b/editeng/source/editeng/impedit.hxx
index 83704be..f6e2407 100644
--- a/editeng/source/editeng/impedit.hxx
+++ b/editeng/source/editeng/impedit.hxx
@@ -557,9 +557,9 @@ private:
 EditPaM ImpFastInsertText( EditPaM aPaM, const OUString rStr 
);
 EditPaM ImpFastInsertParagraph( sal_Int32 nPara );
 
-sal_BoolImpCheckRefMapMode();
+boolImpCheckRefMapMode();
 
-sal_BoolImplHasText() const;
+boolImplHasText() const;
 
 voidImpFindKashidas( ContentNode* pNode, sal_uInt16 
nStart, sal_uInt16 nEnd, std::vectorsal_uInt16 rArray );
 
diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index d100109..2375b41 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -87,7 +87,7 @@ using namespace ::com::sun::star::linguistic2;
 
 struct TabInfo
 {
-sal_BoolbValid;
+boolbValid;
 
 SvxTabStop  aTabStop;
 xub_StrLen  nCharPos;
@@ -95,7 +95,7 @@ struct TabInfo
 longnStartPosX;
 longnTabPos;
 
-TabInfo() { bValid = sal_False; }
+TabInfo() { bValid = false; }
 };
 
 Point Rotate( const Point rPoint, short nOrientation, const Point rOrigin )
@@ -246,7 +246,7 @@ static Point lcl_ImplCalcRotatedPos( Point rPos, Point 
rOrigin, double nSin, dou
 return aTranslatedPos;
 }
 
-static sal_Bool lcl_IsLigature( sal_Unicode cCh, sal_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
+static bool lcl_IsLigature( sal_Unicode cCh, sal_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
 {
 // Lam + Alef
 return ( 0x644 == cCh  0x627 == cNextCh ) ||
@@ -254,11 +254,11 @@ static sal_Bool lcl_IsLigature( sal_Unicode cCh, 
sal_Unicode cNextCh ) // For Ka
( 0x628 == cCh  0x631 == cNextCh );
 }
 
-static sal_Bool lcl_ConnectToPrev( sal_Unicode cCh, sal_Unicode cPrevCh )  // 
For Kashidas from sw/source/core/text/porlay.txt
+static bool lcl_ConnectToPrev( sal_Unicode cCh, sal_Unicode cPrevCh )  // For 
Kashidas from sw/source/core/text/porlay.txt
 {
 // Alef, Dal, Thal, Reh, Zain, and Waw do not connect to the left
-sal_Bool bRet = 0x627 != cPrevCh  0x62F != cPrevCh  0x630 != cPrevCh 
-0x631 != cPrevCh  0x632 != cPrevCh  0x648 != cPrevCh;
+bool bRet = 0x627 != cPrevCh  0x62F != cPrevCh  0x630 != cPrevCh 
+0x631 != cPrevCh  0x632 != cPrevCh  0x648 != cPrevCh;
 
 // check for ligatures cPrevChar + cChar
 if ( bRet )
@@ -369,12 +369,12 @@ void ImpEditEngine::FormatDoc()
 StartOnlineSpellTimer();
 
 long nY = 0;
-sal_Bool bGrow = sal_False;
+bool bGrow = false;
 
 Font aOldFont( GetRefDevice()-GetFont() );
 
 // Here already, so that not always in CreateLines...
-sal_Bool bMapChanged = ImpCheckRefMapMode();
+bool bMapChanged = ImpCheckRefMapMode();
 
 aInvalidRect = Rectangle();  // make empty
 for ( sal_Int32 nPara = 0; nPara  GetParaPortions().Count(); nPara++ )
@@ -405,7 +405,7 @@ void ImpEditEngine::FormatDoc()
 pPP-GetLines().Reset();
 }
 }
-bGrow = sal_True;
+bGrow = true;
 if ( IsCallParaInsertedOrDeleted() )
 GetEditEnginePtr()-ParagraphHeightChanged( nPara );
 pParaPortion-SetMustRepaint( sal_False );
@@ -490,17 +490,17 @@ void ImpEditEngine::FormatDoc()
 LeaveBlockNotifications();
 }
 
-sal_Bool ImpEditEngine::ImpCheckRefMapMode()
+bool ImpEditEngine::ImpCheckRefMapMode()
 {
-sal_Bool bChange = sal_False;
+bool bChange = false;
 
 if ( aStatus.DoFormat100() )
 {
 MapMode aMapMode( GetRefDevice()-GetMapMode() );
 if ( aMapMode.GetScaleX().GetNumerator() != 
aMapMode.GetScaleX().GetDenominator() )
-bChange = sal_True;
+bChange = true;
 else if ( aMapMode.GetScaleY().GetNumerator() != 
aMapMode.GetScaleY().GetDenominator() )
-bChange = sal_True;
+bChange = true;
 
 if ( bChange )
 {
@@ -584,8 +584,8 @@ sal_Bool ImpEditEngine::CreateLines

[Libreoffice-commits] core.git: include/vcl vcl/source

2013-11-20 Thread Rodolfo Ribeiro Gomes
 include/vcl/timer.hxx|   17 ++--
 vcl/source/app/timer.cxx |   92 ++-
 2 files changed, 34 insertions(+), 75 deletions(-)

New commits:
commit 205b77054a03e7d665167d6693ebaf476773aaf2
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sun Nov 17 00:35:56 2013 -0200

sal_Bool to bool in VCL Timer and minor cleanups

Change-Id: I72e3987af4509239b2e989483b7396512c17e48b
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6695
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/include/vcl/timer.hxx b/include/vcl/timer.hxx
index fc37a0e..ff9a4cf 100644
--- a/include/vcl/timer.hxx
+++ b/include/vcl/timer.hxx
@@ -27,17 +27,13 @@
 struct ImplTimerData;
 struct ImplSVData;
 
-// -
-// - Timer -
-// -
-
 class VCL_DLLPUBLIC Timer
 {
 protected:
 ImplTimerData*  mpTimerData;
-sal_uLong   mnTimeout;
-sal_BoolmbActive;
-sal_BoolmbAuto;
+sal_uLong   mnTimeout;
+boolmbActive;
+boolmbAuto;
 LinkmaTimeoutHdl;
 
 public:
@@ -51,8 +47,8 @@ public:
 voidStop();
 
 voidSetTimeout( sal_uLong nTimeout );
-sal_uLong   GetTimeout() const { return mnTimeout; }
-sal_BoolIsActive() const { return mbActive; }
+sal_uLong   GetTimeout() const { return mnTimeout; }
+sal_BoolIsActive() const { return mbActive ? sal_True : sal_False; 
}
 
 voidSetTimeoutHdl( const Link rLink ) { maTimeoutHdl = rLink; 
}
 const Link GetTimeoutHdl() const { return maTimeoutHdl; }
@@ -63,9 +59,6 @@ public:
 static void ImplTimerCallbackProc();
 };
 
-// -
-// - AutoTimer -
-// -
 
 class VCL_DLLPUBLIC AutoTimer : public Timer
 {
diff --git a/vcl/source/app/timer.cxx b/vcl/source/app/timer.cxx
index de5efca..dab9927 100644
--- a/vcl/source/app/timer.cxx
+++ b/vcl/source/app/timer.cxx
@@ -28,26 +28,18 @@
 #include salinst.hxx
 
 
-// ===
-
 #define MAX_TIMER_PERIOD((sal_uLong)0x)
 
-// -
-// - TimeManager-Types -
-// -
-
 struct ImplTimerData
 {
 ImplTimerData*  mpNext; // Pointer to the next Instance
-Timer*  mpSVTimer;  // Pointer to SV Timer instance
-sal_uLong   mnUpdateTime;   // Last Update Time
-sal_uLong   mnTimerUpdate;  // TimerCallbackProcs on stack
+Timer*  mpTimer;// Pointer to VCL Timer instance
+sal_uLong   mnUpdateTime;   // Last Update Time
+sal_uLong   mnTimerUpdate;  // TimerCallbackProcs on stack
 boolmbDelete;   // Was timer deleted during Update()?
 boolmbInTimeout;// Are we in a timeout handler?
 };
 
-// ===
-
 void Timer::ImplDeInitTimer()
 {
 ImplSVData* pSVData = ImplGetSVData();
@@ -58,10 +50,10 @@ void Timer::ImplDeInitTimer()
 do
 {
 ImplTimerData* pTempTimerData = pTimerData;
-if ( pTimerData-mpSVTimer )
+if ( pTimerData-mpTimer )
 {
-pTimerData-mpSVTimer-mbActive = sal_False;
-pTimerData-mpSVTimer-mpTimerData = NULL;
+pTimerData-mpTimer-mbActive = false;
+pTimerData-mpTimer-mpTimerData = NULL;
 }
 pTimerData = pTimerData-mpNext;
 delete pTempTimerData;
@@ -75,8 +67,6 @@ void Timer::ImplDeInitTimer()
 }
 }
 
-// ---
-
 static void ImplStartTimer( ImplSVData* pSVData, sal_uLong nMS )
 {
 if ( !nMS )
@@ -89,16 +79,14 @@ static void ImplStartTimer( ImplSVData* pSVData, sal_uLong 
nMS )
 }
 }
 
-// ---
-
 void Timer::ImplTimerCallbackProc()
 {
 ImplSVData* pSVData = ImplGetSVData();
 ImplTimerData*  pTimerData;
 ImplTimerData*  pPrevTimerData;
-sal_uLong   nMinPeriod = MAX_TIMER_PERIOD;
-sal_uLong   nDeltaTime;
-sal_uLong   nTime = Time::GetSystemTicks();
+sal_uLong   nMinPeriod = MAX_TIMER_PERIOD;
+sal_uLong   nDeltaTime;
+sal_uLong   nTime = Time::GetSystemTicks();
 
 if ( pSVData-mbNoCallTimer )
 return;
@@ -116,21 +104,21 @@ void Timer::ImplTimerCallbackProc()
  !pTimerData-mbDelete  !pTimerData-mbInTimeout )
 {
 // time has expired
-if ( (pTimerData-mnUpdateTime+pTimerData-mpSVTimer-mnTimeout) 
= nTime )
+if ( (pTimerData-mnUpdateTime+pTimerData-mpTimer-mnTimeout) = 
nTime )
 {
 // set new update

[Libreoffice-commits] core.git: include/svtools svtools/source

2013-11-20 Thread Rodolfo Ribeiro Gomes
 include/svtools/ruler.hxx|2 +-
 svtools/source/control/ruler.cxx |   24 
 2 files changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 912d6d171603cdc68c600ab713e22eeca2c1f58a
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon Nov 18 01:30:49 2013 -0200

fdo#65355 Ruler Comment control doesn't disappears

ImplDraw() should not be directly called because it avoids
overriden Paint() method.

Change-Id: I0f0f6b4af231c1ea50a23fb96f3c9018b5ce2d31
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6707
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/include/svtools/ruler.hxx b/include/svtools/ruler.hxx
index eff79e251..a6ae1e5 100644
--- a/include/svtools/ruler.hxx
+++ b/include/svtools/ruler.hxx
@@ -767,7 +767,7 @@ public:
 RulerSelection  GetHoverSelection() const { return maHoverSelection; }
 
 using Window::GetType;
-RulerType   GetType( const Point rPos, sal_uInt16* pAryPos = NULL ) 
const;
+RulerType   GetType( const Point rPos, sal_uInt16* pAryPos = NULL );
 
 voidSetNullOffset( long nPos );
 longGetNullOffset() const;
diff --git a/svtools/source/control/ruler.cxx b/svtools/source/control/ruler.cxx
index 5a0b32f..f6808a2 100644
--- a/svtools/source/control/ruler.cxx
+++ b/svtools/source/control/ruler.cxx
@@ -1868,7 +1868,7 @@ void Ruler::ImplDrag( const Point rPos )
 Drag();
 
 // and redraw
-ImplDraw();
+Paint(Rectangle());
 
 // reset the data as before cancel
 *mpDragData = aTempData;
@@ -1889,7 +1889,7 @@ void Ruler::ImplDrag( const Point rPos )
 
 // redraw
 if ( mbFormat )
-ImplDraw();
+Paint(Rectangle());
 }
 
 mnDragScroll = 0;
@@ -1921,7 +1921,7 @@ void Ruler::ImplEndDrag()
 mnStartDragPos  = 0;
 
 // redraw
-ImplDraw();
+Paint(Rectangle());
 }
 
 IMPL_LINK_NOARG(Ruler, ImplUpdateHdl)
@@ -1932,7 +1932,7 @@ IMPL_LINK_NOARG(Ruler, ImplUpdateHdl)
 if ( mnUpdateFlags  RULER_UPDATE_DRAW )
 {
 mnUpdateFlags = 0;
-ImplDraw();
+Paint(Rectangle());
 }
 else if ( mnUpdateFlags  RULER_UPDATE_LINES )
 {
@@ -1954,7 +1954,7 @@ void Ruler::MouseButtonDown( const MouseEvent rMEvt )
 // update ruler
 if ( mbFormat )
 {
-ImplDraw();
+Paint(Rectangle());
 mnUpdateFlags = ~RULER_UPDATE_DRAW;
 }
 
@@ -2062,7 +2062,7 @@ void Ruler::MouseMove( const MouseEvent rMEvt )
 
 if ( mbFormat )
 {
-ImplDraw();
+Paint(Rectangle());
 mnUpdateFlags = ~RULER_UPDATE_DRAW;
 }
 mpPreviousHitTest.swap(mpCurrentHitTest);
@@ -2189,7 +2189,7 @@ void Ruler::StateChanged( StateChangedType nType )
 else if ( nType == STATE_CHANGE_UPDATEMODE )
 {
 if ( IsReallyVisible()  IsUpdateMode() )
-ImplDraw();
+Paint(Rectangle());
 }
 else if ( (nType == STATE_CHANGE_ZOOM) ||
   (nType == STATE_CHANGE_CONTROLFONT) )
@@ -2291,7 +2291,7 @@ sal_Bool Ruler::StartDocDrag( const MouseEvent rMEvt, 
RulerType eDragType )
 // update ruler
 if ( mbFormat )
 {
-ImplDraw();
+Paint(Rectangle());
 mnUpdateFlags = ~RULER_UPDATE_DRAW;
 }
 
@@ -2350,15 +2350,15 @@ void Ruler::CancelDrag()
 }
 }
 
-RulerType Ruler::GetType( const Point rPos, sal_uInt16* pAryPos ) const
+RulerType Ruler::GetType( const Point rPos, sal_uInt16* pAryPos )
 {
 RulerSelection aHitTest;
 
 // update ruler
 if ( IsReallyVisible()  mbFormat )
 {
-((Ruler*)this)-ImplDraw();
-((Ruler*)this)-mnUpdateFlags = ~RULER_UPDATE_DRAW;
+Paint(Rectangle());
+mnUpdateFlags = ~RULER_UPDATE_DRAW;
 }
 
 ImplHitTest( rPos, aHitTest );
@@ -2790,7 +2790,7 @@ RulerUnitData Ruler::GetCurrentRulerUnit() const
 void Ruler::DrawTicks()
 {
 mbFormat = sal_True;
-ImplDraw();
+Paint(Rectangle());
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/source

2013-11-20 Thread Rodolfo Ribeiro Gomes
 sw/source/ui/misc/swruler.cxx |7 +++
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit 0b4b02d9b6cf5b939dc1522d8348e3f4d6f955d4
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Tue Nov 19 00:29:06 2013 -0200

Remove now useless RTL code-comment on SwCommentRuler

RTL work was done by someone else.

Change-Id: I8b8b923ff3e336427a2dadeccab7c7176d51b2fe
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6716
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/ui/misc/swruler.cxx b/sw/source/ui/misc/swruler.cxx
index 914231b..0697639 100644
--- a/sw/source/ui/misc/swruler.cxx
+++ b/sw/source/ui/misc/swruler.cxx
@@ -241,8 +241,6 @@ void SwCommentRuler::UpdateCommentHelpText()
 SetQuickHelpText( OUString( SW_RESSTR( nTooltipResId ) ) );
 }
 
-// NOTE: If sometime ahead SwSidebar moves from page right side due RTL layout,
-//   change how this rectangle is used by callers
 // TODO Make Ruler return its central rectangle instead of margins.
 Rectangle SwCommentRuler::GetCommentControlRegion()
 {
@@ -255,14 +253,15 @@ Rectangle SwCommentRuler::GetCommentControlRegion()
 if (!pPostItMgr)
 return Rectangle();
 
+unsigned long nSidebarWidth = pPostItMgr-GetSidebarWidth(true);
 //FIXME When the page width is larger then screen, the ruler is misplaced 
by one pixel
 if (GetTextRTL())
-   nLeft = GetPageOffset() - pPostItMgr-GetSidebarWidth(true) + 
GetBorderOffset();
+   nLeft = GetPageOffset() - nSidebarWidth + GetBorderOffset();
 else
nLeft = GetWinOffset() + GetPageOffset() + 
mpSwWin-LogicToPixel(Size(GetPageWidth(), 0)).Width();
 long nTop= 0 + 4; // Ruler::ImplDraw uses RULER_OFF (value: 3px) as 
offset, and Ruler::ImplFormat adds one extra pixel
 // Somehow pPostItMgr-GetSidebarBorderWidth() returns border width 
already doubled
-long nRight  = nLeft+ pPostItMgr-GetSidebarWidth(true) + 
pPostItMgr-GetSidebarBorderWidth(true);
+long nRight  = nLeft + nSidebarWidth + 
pPostItMgr-GetSidebarBorderWidth(true);
 long nBottom = nTop + GetRulerVirHeight() - 3;
 
 Rectangle aRect(nLeft, nTop, nRight, nBottom);
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/source

2013-11-20 Thread Rodolfo Ribeiro Gomes
 sw/source/ui/inc/swruler.hxx  |   19 ++
 sw/source/ui/misc/swruler.cxx |   54 --
 2 files changed, 60 insertions(+), 13 deletions(-)

New commits:
commit 55716b1ed969073f273c00baedb56d8f5de93761
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sun Nov 17 17:06:38 2013 -0200

Highlight fade in SwCommentRuler

It was not implemented, though designed.
No delay was used before start fading. I don't think it flickers.


https://wiki.documentfoundation.org/Design/Whiteboards/Comments_Ruler_Control#Behavior

Change-Id: I419220aac0a9cbfae328874cf6015d9530b93784
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6706
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/sw/source/ui/inc/swruler.hxx b/sw/source/ui/inc/swruler.hxx
index 10a23c1..aebf0b9 100644
--- a/sw/source/ui/inc/swruler.hxx
+++ b/sw/source/ui/inc/swruler.hxx
@@ -45,6 +45,8 @@ protected:
 SwViewShell * mpViewShell; // Shell to check if there is any comments 
on doc and their visibility
 SwEditWin * mpSwWin; // Used to get SwView to change the SideBar 
visibility
 boolmbIsHighlighted; // If comment control is highlighted (mouse 
is over it)
+Timer   maFadeTimer; // Timer for high/'low'light fading
+int mnFadeRate;  // From 0 to 100. 0 means not highlighted.
 VirtualDevice maVirDev;  // VirtualDevice of this window. Just for 
convenience.
 
 /**
@@ -77,7 +79,7 @@ protected:
  * Get the rectangle area that should be used to draw the comment control.
  *
  * It is horizontally aligned to the SideBar panel.
- * \param Rectangle The area where the comment control is.
+ * \return The area where the comment control is.
  */
 Rectangle GetCommentControlRegion();
 
@@ -98,6 +100,21 @@ protected:
  * Update the tooltip text.
  */
 void UpdateCommentHelpText();
+
+/**
+ * Get the proper color between two options, according to current status.
+ *
+ * The return color can be one of the given colors, or a merged one.
+ * It depends on highlight fadind status.
+ *
+ * \param rHighColor color used to highlight status
+ * \param rLowColor color used to normal status
+ * \return The proper color to used in moment
+ */
+Color GetFadedColor(const Color rHighColor, const Color rLowColor);
+
+/// Fade timer callback.
+DECL_LINK(FadeHandler, void *);
 };
 
 #endif
diff --git a/sw/source/ui/misc/swruler.cxx b/sw/source/ui/misc/swruler.cxx
index 0697639..fa396d0 100644
--- a/sw/source/ui/misc/swruler.cxx
+++ b/sw/source/ui/misc/swruler.cxx
@@ -10,7 +10,6 @@
 // FIX fdo#38246 https://bugs.libreoffice.org/show_bug.cgi?id=38246
 // Design proposal: 
https://wiki.documentfoundation.org/Design/Whiteboards/Comments_Ruler_Control
 // TODO Alpha blend border when it doesn't fit in window
-// TODO Delayed highlight fading when user moves mouse over and out the control
 
 #include swruler.hxx
 
@@ -40,8 +39,12 @@ SwCommentRuler::SwCommentRuler( SwViewShell* pViewSh, 
Window* pParent, SwEditWin
 , mpViewShell(pViewSh)
 , mpSwWin(pWin)
 , mbIsHighlighted(false)
+, mnFadeRate(0)
 , maVirDev( *this )
 {
+// Set fading timeout: 5 x 40ms = 200ms
+maFadeTimer.SetTimeout(40);
+maFadeTimer.SetTimeoutHdl( LINK( this, SwCommentRuler, FadeHandler ) );
 }
 
 // Destructor
@@ -57,6 +60,7 @@ void SwCommentRuler::Paint( const Rectangle rRect )
 DrawCommentControl();
 }
 
+
 void SwCommentRuler::DrawCommentControl()
 {
 const StyleSettings rStyleSettings = GetSettings().GetStyleSettings();
@@ -67,10 +71,8 @@ void SwCommentRuler::DrawCommentControl()
 
 // Paint comment control background
 // TODO Check if these are best colors to be used
-if ( mbIsHighlighted )
-maVirDev.SetFillColor( rStyleSettings.GetDarkShadowColor() );
-else
-maVirDev.SetFillColor( rStyleSettings.GetWorkspaceColor() );
+Color aBgColor = GetFadedColor( rStyleSettings.GetDarkShadowColor(), 
rStyleSettings.GetWorkspaceColor() );
+maVirDev.SetFillColor( aBgColor );
 
 if ( mbIsHighlighted || !bIsCollapsed )
 {
@@ -85,6 +87,7 @@ void SwCommentRuler::DrawCommentControl()
 
 maVirDev.DrawRect( Rectangle( Point(), aControlRect.GetSize() ) );
 
+// Label and arrow tip
 OUString aLabel( SW_RESSTR ( STR_COMMENTS_LABEL ) );
 // Get label and arrow coordinates
 Point aLabelPos;
@@ -130,15 +133,15 @@ void SwCommentRuler::DrawCommentControl()
 }
 
 // Draw label
-const Color rTextColor = mbIsHighlighted ? 
rStyleSettings.GetButtonTextColor() : rStyleSettings.GetDeactiveTextColor();
-maVirDev.SetTextColor( rTextColor );
+Color aTextColor = GetFadedColor( rStyleSettings.GetButtonTextColor(), 
rStyleSettings.GetDeactiveTextColor() );
+maVirDev.SetTextColor

[Libreoffice-commits] core.git: sfx2/source

2013-11-11 Thread Rodolfo Ribeiro Gomes
 sfx2/source/control/thumbnailview.cxx |  169 +-
 1 file changed, 86 insertions(+), 83 deletions(-)

New commits:
commit b29464a8fa58d776279c6da0247946eefad9be00
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon Nov 11 22:24:20 2013 -0200

less if-nesting in ThumnailView

Easier to read.

Change-Id: I1ceb551d8badffb211b1f444b61375a03cac1348
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6643
Reviewed-by: Noel Grandin noelgran...@gmail.com
Tested-by: Noel Grandin noelgran...@gmail.com

diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index 6279174..592c8b6 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -732,113 +732,116 @@ void ThumbnailView::MakeItemVisible( sal_uInt16 nItemId 
)
 
 void ThumbnailView::MouseButtonDown( const MouseEvent rMEvt )
 {
-if ( rMEvt.IsLeft() )
+if ( !rMEvt.IsLeft() )
 {
-size_t nPos = ImplGetItem(rMEvt.GetPosPixel());
-ThumbnailViewItem* pItem = ImplGetItem(nPos);
+Control::MouseButtonDown( rMEvt );
+return;
+}
 
-if (pItem)
-{
-if ( rMEvt.GetClicks() == 1 )
-{
-if (rMEvt.IsMod1())
-{
-//Keep selected item group state and just invert current 
desired one state
-pItem-setSelection(!pItem-isSelected());
+size_t nPos = ImplGetItem(rMEvt.GetPosPixel());
+ThumbnailViewItem* pItem = ImplGetItem(nPos);
 
-//This one becomes the selection range start position if 
it changes its state to selected otherwise resets it
-mpStartSelRange = pItem-isSelected() ? 
mFilteredItemList.begin() + nPos : mFilteredItemList.end();
-}
-else if (rMEvt.IsShift()  mpStartSelRange != 
mFilteredItemList.end())
-{
-std::pairsize_t,size_t aNewRange;
-aNewRange.first = mpStartSelRange - 
mFilteredItemList.begin();
-aNewRange.second = nPos;
+if ( !pItem )
+{
+deselectItems();
+Control::MouseButtonDown( rMEvt );
+return;
+}
 
-if (aNewRange.first  aNewRange.second)
-std::swap(aNewRange.first,aNewRange.second);
+if ( rMEvt.GetClicks() == 2 )
+{
+OnItemDblClicked(pItem);
+return;
+}
 
-//Deselect the ones outside of it
-for (size_t i = 0, n = mFilteredItemList.size(); i  n; 
++i)
-{
-ThumbnailViewItem *pCurItem  = mFilteredItemList[i];
+if ( rMEvt.GetClicks() == 1 )
+{
+if (rMEvt.IsMod1())
+{
+//Keep selected item group state and just invert current desired 
one state
+pItem-setSelection(!pItem-isSelected());
 
-if (pCurItem-isSelected()  (i  aNewRange.first || 
i  aNewRange.second))
-{
-pCurItem-setSelection(false);
+//This one becomes the selection range start position if it 
changes its state to selected otherwise resets it
+mpStartSelRange = pItem-isSelected() ? mFilteredItemList.begin() 
+ nPos : mFilteredItemList.end();
+}
+else if (rMEvt.IsShift()  mpStartSelRange != mFilteredItemList.end())
+{
+std::pairsize_t,size_t aNewRange;
+aNewRange.first = mpStartSelRange - mFilteredItemList.begin();
+aNewRange.second = nPos;
 
-if (pCurItem-isVisible())
-DrawItem(pCurItem);
+if (aNewRange.first  aNewRange.second)
+std::swap(aNewRange.first,aNewRange.second);
 
-maItemStateHdl.Call(pCurItem);
-}
-}
+//Deselect the ones outside of it
+for (size_t i = 0, n = mFilteredItemList.size(); i  n; ++i)
+{
+ThumbnailViewItem *pCurItem  = mFilteredItemList[i];
 
-size_t nSelPos = mpStartSelRange - 
mFilteredItemList.begin();
+if (pCurItem-isSelected()  (i  aNewRange.first || i  
aNewRange.second))
+{
+pCurItem-setSelection(false);
 
-//Select the items between start range and the selected 
item
-if (nSelPos != nPos)
-{
-int dir = nSelPos  nPos ? 1 : -1;
-size_t nCurPos = nSelPos + dir;
+if (pCurItem-isVisible())
+DrawItem(pCurItem);
+
+maItemStateHdl.Call(pCurItem);
+}
+}
+
+size_t nSelPos = mpStartSelRange

[Libreoffice-commits] core.git: sfx2/source

2013-11-10 Thread Rodolfo Ribeiro Gomes
 sfx2/source/control/recentdocsview.cxx |   10 +++---
 sfx2/source/control/thumbnailview.cxx  |7 ++-
 2 files changed, 5 insertions(+), 12 deletions(-)

New commits:
commit a5156a41633701fd0591640ba665fdb10221fe71
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sun Nov 10 10:46:42 2013 -0200

ThumbnailView: item found by mouse pos is always visible

-- so it does not need to check it again

Change-Id: Id6fc2732bde8dd7f6011828b6bc29d7947bf6312
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6629
Reviewed-by: Jan Holesovsky ke...@collabora.com
Tested-by: Jan Holesovsky ke...@collabora.com

diff --git a/sfx2/source/control/recentdocsview.cxx 
b/sfx2/source/control/recentdocsview.cxx
index 888c3ad..14e8f65 100644
--- a/sfx2/source/control/recentdocsview.cxx
+++ b/sfx2/source/control/recentdocsview.cxx
@@ -180,14 +180,10 @@ void RecentDocsView::MouseButtonDown( const MouseEvent 
rMEvt )
 size_t nPos = ImplGetItem(rMEvt.GetPosPixel());
 ThumbnailViewItem* pItem = ImplGetItem(nPos);
 
-if (pItem  pItem-isVisible())
+if (pItem)
 {
-Rectangle aRect(pItem-getDrawArea());
-if (aRect.IsInside(rMEvt.GetPosPixel()))
-{
-OnItemDblClicked(pItem);
-return;
-}
+OpenItem(pItem);
+return;
 }
 }
 ThumbnailView::MouseButtonDown( rMEvt );
diff --git a/sfx2/source/control/thumbnailview.cxx 
b/sfx2/source/control/thumbnailview.cxx
index e3e1b8f..6279174 100644
--- a/sfx2/source/control/thumbnailview.cxx
+++ b/sfx2/source/control/thumbnailview.cxx
@@ -737,7 +737,7 @@ void ThumbnailView::MouseButtonDown( const MouseEvent 
rMEvt )
 size_t nPos = ImplGetItem(rMEvt.GetPosPixel());
 ThumbnailViewItem* pItem = ImplGetItem(nPos);
 
-if (pItem  pItem-isVisible())
+if (pItem)
 {
 if ( rMEvt.GetClicks() == 1 )
 {
@@ -829,10 +829,7 @@ void ThumbnailView::MouseButtonDown( const MouseEvent 
rMEvt )
 }
 else if ( rMEvt.GetClicks() == 2 )
 {
-Rectangle aRect(pItem-getDrawArea());
-
-if (aRect.IsInside(rMEvt.GetPosPixel()))
-OnItemDblClicked(pItem);
+OnItemDblClicked(pItem);
 }
 
 return;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: include/sfx2 sfx2/source

2013-11-07 Thread Rodolfo Ribeiro Gomes
 include/sfx2/recentdocsview.hxx|2 ++
 sfx2/source/control/recentdocsview.cxx |   27 ++-
 2 files changed, 28 insertions(+), 1 deletion(-)

New commits:
commit a29c9eff781fd6bceee5078669a53c52086b5664
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Thu Nov 7 13:59:42 2013 -0200

single-click for open recent documents in new Start Center

Change-Id: I34c6a5f9057707cdd1e2f4f82024818e7c8a6630
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6610
Reviewed-by: Jan Holesovsky ke...@collabora.com
Tested-by: Jan Holesovsky ke...@collabora.com

diff --git a/include/sfx2/recentdocsview.hxx b/include/sfx2/recentdocsview.hxx
index 0552636..2edace5 100644
--- a/include/sfx2/recentdocsview.hxx
+++ b/include/sfx2/recentdocsview.hxx
@@ -58,7 +58,9 @@ public:
 DECL_STATIC_LINK( RecentDocsView, ExecuteHdl_Impl, LoadRecentFile* );
 
 protected:
+virtual void MouseButtonDown( const MouseEvent rMEvt );
 virtual void OnItemDblClicked(ThumbnailViewItem *pItem);
+void OpenItem( const ThumbnailViewItem *pItem );
 
 bool isAcceptedFile(const OUString rURL) const;
 
diff --git a/sfx2/source/control/recentdocsview.cxx 
b/sfx2/source/control/recentdocsview.cxx
index 5e3c796..706214e 100644
--- a/sfx2/source/control/recentdocsview.cxx
+++ b/sfx2/source/control/recentdocsview.cxx
@@ -175,9 +175,34 @@ void RecentDocsView::loadRecentDocs()
 Invalidate();
 }
 
+void RecentDocsView::MouseButtonDown( const MouseEvent rMEvt )
+{
+if ( rMEvt.IsLeft() )
+{
+size_t nPos = ImplGetItem(rMEvt.GetPosPixel());
+ThumbnailViewItem* pItem = ImplGetItem(nPos);
+
+if (pItem  pItem-isVisible())
+{
+Rectangle aRect(pItem-getDrawArea());
+if (aRect.IsInside(rMEvt.GetPosPixel()))
+{
+OnItemDblClicked(pItem);
+return;
+}
+}
+}
+ThumbnailView::MouseButtonDown( rMEvt );
+}
+
 void RecentDocsView::OnItemDblClicked(ThumbnailViewItem *pItem)
 {
-RecentDocsViewItem* pRecentItem = dynamic_castRecentDocsViewItem*(pItem);
+OpenItem( pItem );
+}
+
+void RecentDocsView::OpenItem( const ThumbnailViewItem *pItem )
+{
+const RecentDocsViewItem* pRecentItem = dynamic_castconst 
RecentDocsViewItem*(pItem);
 
 Reference XDispatch xDispatch;
 Reference XDispatchProvider xDispatchProvider;
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sc/source

2013-11-04 Thread Rodolfo Ribeiro Gomes
 sc/source/ui/inc/colrowba.hxx  |   32 ++-
 sc/source/ui/inc/hdrcont.hxx   |   24 +--
 sc/source/ui/view/colrowba.cxx |   33 +++
 sc/source/ui/view/hdrcont.cxx  |   85 +++--
 4 files changed, 73 insertions(+), 101 deletions(-)

New commits:
commit b64d19a6520871eaee908c79e9aaf76ee477e143
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sat Nov 2 23:37:34 2013 -0200

German translations and minor clean-up in ScHeaderControls

Some methods properly defined as const methods

Change-Id: I59bc371bbb3adc6df2b96b82a0968a8693622ff8
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6537
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com
Tested-by: Markus Mohrhard markus.mohrh...@googlemail.com

diff --git a/sc/source/ui/inc/colrowba.hxx b/sc/source/ui/inc/colrowba.hxx
index 2a0e976..2bb2c68 100644
--- a/sc/source/ui/inc/colrowba.hxx
+++ b/sc/source/ui/inc/colrowba.hxx
@@ -26,9 +26,6 @@
 class ScHeaderFunctionSet;
 class ScHeaderSelectionEngine;
 
-// ---
-
-
 class ScColBar : public ScHeaderControl
 {
 ScViewData*  pViewData;
@@ -40,19 +37,19 @@ public:
 ScHeaderFunctionSet* pFunc, 
ScHeaderSelectionEngine* pEng );
 ~ScColBar();
 
-virtual SCCOLROWGetPos();
-virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo );
-virtual OUStringGetEntryText( SCCOLROW nEntryNo );
+virtual SCCOLROWGetPos() const;
+virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo ) const;
+virtual OUStringGetEntryText( SCCOLROW nEntryNo ) const;
 
-virtual boolIsLayoutRTL();  // only for columns
+virtual boolIsLayoutRTL() const;
 
 virtual voidSetEntrySize( SCCOLROW nPos, sal_uInt16 nNewSize );
 virtual voidHideEntries( SCCOLROW nStart, SCCOLROW nEnd );
 
 virtual voidSetMarking( bool bSet );
 virtual voidSelectWindow();
-virtual boolIsDisabled();
-virtual boolResizeAllowed();
+virtual boolIsDisabled() const;
+virtual boolResizeAllowed() const;
 
 virtual voidDrawInvert( long nDragPos );
 
@@ -73,29 +70,26 @@ public:
 ScHeaderFunctionSet* pFunc, 
ScHeaderSelectionEngine* pEng );
 ~ScRowBar();
 
-virtual SCCOLROWGetPos();
-virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo );
-virtual OUStringGetEntryText( SCCOLROW nEntryNo );
+virtual SCCOLROWGetPos() const;
+virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo ) const;
+virtual OUStringGetEntryText( SCCOLROW nEntryNo ) const;
 
-virtual boolIsMirrored();   // only for columns
-virtual SCROW   GetHiddenCount( SCROW nEntryNo );   // only for columns
+virtual boolIsMirrored() const;
+virtual SCROW   GetHiddenCount( SCROW nEntryNo ) const;
 
 virtual voidSetEntrySize( SCCOLROW nPos, sal_uInt16 nNewSize );
 virtual voidHideEntries( SCCOLROW nStart, SCCOLROW nEnd );
 
 virtual voidSetMarking( bool bSet );
 virtual voidSelectWindow();
-virtual boolIsDisabled();
-virtual boolResizeAllowed();
+virtual boolIsDisabled() const;
+virtual boolResizeAllowed() const;
 
 virtual voidDrawInvert( long nDragPos );
 
 virtual OUStringGetDragHelp( long nVal );
 };
 
-
-
 #endif
 
-
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sc/source/ui/inc/hdrcont.hxx b/sc/source/ui/inc/hdrcont.hxx
index 3846cf8..9979af6 100644
--- a/sc/source/ui/inc/hdrcont.hxx
+++ b/sc/source/ui/inc/hdrcont.hxx
@@ -49,7 +49,7 @@ private:
 SCCOLROWnMarkEnd;
 boolbMarkRange;
 
-boolbDragging;  // Change size
+boolbDragging;  // Resizing
 SCCOLROWnDragNo;
 longnDragStart;
 longnDragPos;
@@ -57,8 +57,8 @@ private:
 
 boolbIgnoreMove;
 
-longGetScrPos( SCCOLROW nEntryNo );
-SCCOLROWGetMousePos( const MouseEvent rMEvt, bool rBorder );
+longGetScrPos( SCCOLROW nEntryNo ) const;
+SCCOLROWGetMousePos( const MouseEvent rMEvt, bool rBorder ) 
const;
 boolIsSelectionAllowed(SCCOLROW nPos) const;
 voidShowDragHelp();
 
@@ -80,21 +80,21 @@ protected:
 
 // new methods
 
-virtual SCCOLROWGetPos() = 0;   // current 
position (Scrolling)
-virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo ) = 0;  // width / 
height (Pixel)
-virtual OUString  GetEntryText( SCCOLROW nEntryNo ) = 0;
+virtual SCCOLROW

[Libreoffice-commits] core.git: sc/source

2013-11-04 Thread Rodolfo Ribeiro Gomes
 sc/source/ui/view/hdrcont.cxx |8 +++-
 1 file changed, 3 insertions(+), 5 deletions(-)

New commits:
commit 0432f847bba2e2d074d72a9ff794d680e0f46785
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sun Nov 3 22:57:26 2013 -0200

minor cleanup in ScHeaderControl: nCount is always greater than 0

Change-Id: Ia50265e397704131dc948bc2bcbf80a442054b71
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/6538
Reviewed-by: Markus Mohrhard markus.mohrh...@googlemail.com
Tested-by: Markus Mohrhard markus.mohrh...@googlemail.com

diff --git a/sc/source/ui/view/hdrcont.cxx b/sc/source/ui/view/hdrcont.cxx
index 93b5794..d536780 100644
--- a/sc/source/ui/view/hdrcont.cxx
+++ b/sc/source/ui/view/hdrcont.cxx
@@ -563,9 +563,9 @@ void ScHeaderControl::Paint( const Rectangle rRect )
 SCCOLROW ScHeaderControl::GetMousePos( const MouseEvent rMEvt, bool rBorder 
) const
 {
 boolbFound = false;
-SCCOLROWnCount = 1;
 SCCOLROWnPos = GetPos();
 SCCOLROWnHitNo = nPos;
+SCCOLROWnEntryNo = 1 + nPos;
 longnScrPos;
 longnMousePos = bVertical ? rMEvt.GetPosPixel().Y() : 
rMEvt.GetPosPixel().X();
 longnDif;
@@ -579,22 +579,20 @@ SCCOLROW ScHeaderControl::GetMousePos( const MouseEvent 
rMEvt, bool rBorder )
 nScrPos = GetScrPos( nPos ) - nLayoutSign;
 do
 {
-SCCOLROW nEntryNo = nCount + nPos;
-
 if (nEntryNo  nSize)
 nScrPos = nEndPos + nLayoutSign;
 else
 nScrPos += GetEntrySize( nEntryNo - 1 ) * nLayoutSign;  //! 
GetHiddenCount() ??
 
 nDif = nMousePos - nScrPos;
-if (nDif = -2  nDif = 2  nCount  0)
+if (nDif = -2  nDif = 2)
 {
 bFound = true;
 nHitNo=nEntryNo-1;
 }
 else if (nDif * nLayoutSign = 0  nEntryNo  nSize)
 nHitNo = nEntryNo;
-++nCount;
+++nEntryNo;
 }
 while ( nScrPos * nLayoutSign  nEndPos * nLayoutSign  nDif * 
nLayoutSign  0 );
 
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sc/source

2013-10-28 Thread Rodolfo Ribeiro Gomes
 sc/source/ui/inc/colrowba.hxx  |   18 +--
 sc/source/ui/inc/hdrcont.hxx   |   31 +--
 sc/source/ui/view/colrowba.cxx |   18 +--
 sc/source/ui/view/hdrcont.cxx  |   66 -
 4 files changed, 66 insertions(+), 67 deletions(-)

New commits:
commit 8a82cacc668268be2fd5471fe4357d52c4372241
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon Oct 28 21:15:56 2013 -0200

sal_Bool to bool in ScHeaderControl

Change-Id: I8f5b9207f5634e3ff2c55d7cf9372c8caad0ed23
Reviewed-on: https://gerrit.libreoffice.org/6471
Reviewed-by: Eike Rathke er...@redhat.com
Tested-by: Eike Rathke er...@redhat.com

diff --git a/sc/source/ui/inc/colrowba.hxx b/sc/source/ui/inc/colrowba.hxx
index a9b2b14..2a0e976 100644
--- a/sc/source/ui/inc/colrowba.hxx
+++ b/sc/source/ui/inc/colrowba.hxx
@@ -44,21 +44,21 @@ public:
 virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo );
 virtual OUStringGetEntryText( SCCOLROW nEntryNo );
 
-virtual sal_BoolIsLayoutRTL();  // only for 
columns
+virtual boolIsLayoutRTL();  // only for columns
 
 virtual voidSetEntrySize( SCCOLROW nPos, sal_uInt16 nNewSize );
 virtual voidHideEntries( SCCOLROW nStart, SCCOLROW nEnd );
 
-virtual voidSetMarking( sal_Bool bSet );
+virtual voidSetMarking( bool bSet );
 virtual voidSelectWindow();
-virtual sal_BoolIsDisabled();
-virtual sal_BoolResizeAllowed();
+virtual boolIsDisabled();
+virtual boolResizeAllowed();
 
 virtual voidDrawInvert( long nDragPos );
 
 virtual OUStringGetDragHelp( long nVal );
 
-sal_BoolUseNumericHeader() const;
+boolUseNumericHeader() const;
 };
 
 
@@ -77,16 +77,16 @@ public:
 virtual sal_uInt16  GetEntrySize( SCCOLROW nEntryNo );
 virtual OUStringGetEntryText( SCCOLROW nEntryNo );
 
-virtual sal_BoolIsMirrored();   // only for 
columns
+virtual boolIsMirrored();   // only for columns
 virtual SCROW   GetHiddenCount( SCROW nEntryNo );   // only for columns
 
 virtual voidSetEntrySize( SCCOLROW nPos, sal_uInt16 nNewSize );
 virtual voidHideEntries( SCCOLROW nStart, SCCOLROW nEnd );
 
-virtual voidSetMarking( sal_Bool bSet );
+virtual voidSetMarking( bool bSet );
 virtual voidSelectWindow();
-virtual sal_BoolIsDisabled();
-virtual sal_BoolResizeAllowed();
+virtual boolIsDisabled();
+virtual boolResizeAllowed();
 
 virtual voidDrawInvert( long nDragPos );
 
diff --git a/sc/source/ui/inc/hdrcont.hxx b/sc/source/ui/inc/hdrcont.hxx
index 31a4bf6..3846cf8 100644
--- a/sc/source/ui/inc/hdrcont.hxx
+++ b/sc/source/ui/inc/hdrcont.hxx
@@ -26,8 +26,7 @@
 
 #define HDR_SIZE_OPTIMUM0x
 
-
-// Size of the sliders
+// Size of the sliders
 #define HDR_SLIDERSIZE  2
 
 class ScHeaderControl : public Window
@@ -36,9 +35,9 @@ private:
 SelectionEngine*pSelEngine;
 FontaNormFont;
 FontaBoldFont;
-sal_BoolbBoldSet;
+boolbBoldSet;
 
-sal_BoolbVertical;  // Vertical = Row header
+boolbVertical;  // Vertical = Row header
 
 longnWidth;
 longnSmallWidth;
@@ -48,18 +47,18 @@ private:
 
 SCCOLROWnMarkStart;
 SCCOLROWnMarkEnd;
-sal_BoolbMarkRange;
+boolbMarkRange;
 
-sal_BoolbDragging;  // Change size
+boolbDragging;  // Change size
 SCCOLROWnDragNo;
 longnDragStart;
 longnDragPos;
-sal_BoolbDragMoved;
+boolbDragMoved;
 
-sal_BoolbIgnoreMove;
+boolbIgnoreMove;
 
 longGetScrPos( SCCOLROW nEntryNo );
-SCCOLROWGetMousePos( const MouseEvent rMEvt, sal_Bool rBorder );
+SCCOLROWGetMousePos( const MouseEvent rMEvt, bool rBorder );
 boolIsSelectionAllowed(SCCOLROW nPos) const;
 voidShowDragHelp();
 
@@ -86,16 +85,16 @@ protected:
 virtual OUString  GetEntryText( SCCOLROW nEntryNo ) = 0;
 
 virtual SCCOLROW GetHiddenCount( SCCOLROW nEntryNo );
-virtual sal_BoolIsLayoutRTL();
-virtual sal_BoolIsMirrored();
+virtual bool IsLayoutRTL();
+virtual bool IsMirrored();
 
 virtual voidSetEntrySize( SCCOLROW nPos, sal_uInt16 nNewWidth ) = 0;
 virtual voidHideEntries( SCCOLROW nStart, SCCOLROW nEnd ) = 0;
 
-virtual voidSetMarking( sal_Bool bSet );
+virtual voidSetMarking( bool

[Libreoffice-commits] core.git: vcl/source

2013-09-19 Thread Rodolfo Ribeiro Gomes
 vcl/source/gdi/impimagetree.cxx |5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

New commits:
commit 58d8a11a1b44d5f07c43f96c7e4ab03dc549ef27
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Sat Sep 14 19:27:50 2013 -0300

Theme icons are loaded only once

With the theme icon cleaning commit
d5151ab592367fde7db03fce81e0b76776d18216, duplicated
icons (same icon for multiple actions) doesn't need to
be stored in different files. The file links.txt inside
the theme pack can handle filename redirections.
So, it keeps the theme pack smaller. However, the
duplicates keep being loaded, because they use different
names.

With this patch, the names of those duplicates are mapped
also when querying for the icons. This way, an icon is
loaded only once, saving RAM.

Change-Id: Ieb5b8424c1515f0033276bf314cdef02a4e01862
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/5943
Reviewed-by: Caolán McNamara caol...@redhat.com
Tested-by: Caolán McNamara caol...@redhat.com

diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index 4d549f0..1c67667 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -270,13 +270,12 @@ bool ImplImageTree::checkStyleCacheLookup(
 bool ImplImageTree::iconCacheLookup(
 OUString const  name, bool localized, BitmapEx  bitmap)
 {
-IconCache::iterator i(m_iconCache.find(name));
+IconCache::iterator i(m_iconCache.find(getRealImageName(name)));
 if (i != m_iconCache.end()  i-second.first == localized) {
 bitmap = i-second.second;
 return true;
-} else {
-return false;
 }
+return false;
 }
 
 bool ImplImageTree::find(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Remove unused icons after cleaning up Math zoom

2013-05-31 Thread Rodolfo Ribeiro Gomes (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/4117

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/17/4117/1

Remove unused icons after cleaning up Math zoom

After commit ffc2e5be1f712b09710e2096ad2f7eb81b80118d ,
these deleted icons are useless since other UNO commands are used instead:
  View050 - Zoom50Percent
  View100 - Zoom100Percent
  View200 - Zoom200Percent
  Adjust  - ZoomOptimal
  FitInWindow - ZoomOptimal

Sorry I forgot to remove then in that commit.

Change-Id: I141f2c4ad2f63928efb15faa5f68bc9c6dabc144
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
---
D icon-themes/crystal/cmd/lc_adjust.png
D icon-themes/crystal/cmd/lc_view100.png
D icon-themes/crystal/cmd/sc_adjust.png
D icon-themes/crystal/cmd/sc_view100.png
D icon-themes/galaxy/cmd/lc_adjust.png
D icon-themes/galaxy/cmd/lc_view100.png
D icon-themes/galaxy/cmd/sc_adjust.png
D icon-themes/galaxy/cmd/sc_view100.png
D icon-themes/hicontrast/cmd/lc_adjust.png
D icon-themes/hicontrast/cmd/lc_view100.png
D icon-themes/hicontrast/cmd/sc_adjust.png
D icon-themes/hicontrast/cmd/sc_view100.png
D icon-themes/human/cmd/lc_adjust.png
D icon-themes/human/cmd/lc_view100.png
D icon-themes/human/cmd/sc_adjust.png
D icon-themes/human/cmd/sc_view100.png
D icon-themes/industrial/cmd/lc_adjust.png
D icon-themes/industrial/cmd/lc_view100.png
D icon-themes/industrial/cmd/sc_adjust.png
D icon-themes/industrial/cmd/sc_view100.png
D icon-themes/oxygen/cmd/lc_adjust.png
D icon-themes/oxygen/cmd/lc_view100.png
D icon-themes/oxygen/cmd/sc_adjust.png
D icon-themes/oxygen/cmd/sc_view100.png
D icon-themes/tango/cmd/lc_adjust.png
D icon-themes/tango/cmd/lc_view100.png
D icon-themes/tango_testing/cmd/lc_view100.png
27 files changed, 0 insertions(+), 0 deletions(-)



diff --git a/icon-themes/crystal/cmd/lc_adjust.png 
b/icon-themes/crystal/cmd/lc_adjust.png
deleted file mode 100644
index 5de38bf..000
--- a/icon-themes/crystal/cmd/lc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/crystal/cmd/lc_view100.png 
b/icon-themes/crystal/cmd/lc_view100.png
deleted file mode 100644
index cc8ce15..000
--- a/icon-themes/crystal/cmd/lc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/crystal/cmd/sc_adjust.png 
b/icon-themes/crystal/cmd/sc_adjust.png
deleted file mode 100644
index 9efbd54..000
--- a/icon-themes/crystal/cmd/sc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/crystal/cmd/sc_view100.png 
b/icon-themes/crystal/cmd/sc_view100.png
deleted file mode 100644
index 8709d0a..000
--- a/icon-themes/crystal/cmd/sc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/galaxy/cmd/lc_adjust.png 
b/icon-themes/galaxy/cmd/lc_adjust.png
deleted file mode 100644
index 228d958..000
--- a/icon-themes/galaxy/cmd/lc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/galaxy/cmd/lc_view100.png 
b/icon-themes/galaxy/cmd/lc_view100.png
deleted file mode 100644
index 2f00d91..000
--- a/icon-themes/galaxy/cmd/lc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/galaxy/cmd/sc_adjust.png 
b/icon-themes/galaxy/cmd/sc_adjust.png
deleted file mode 100644
index d9d1a66..000
--- a/icon-themes/galaxy/cmd/sc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/galaxy/cmd/sc_view100.png 
b/icon-themes/galaxy/cmd/sc_view100.png
deleted file mode 100644
index bfd1a58..000
--- a/icon-themes/galaxy/cmd/sc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/hicontrast/cmd/lc_adjust.png 
b/icon-themes/hicontrast/cmd/lc_adjust.png
deleted file mode 100644
index 60765d6..000
--- a/icon-themes/hicontrast/cmd/lc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/hicontrast/cmd/lc_view100.png 
b/icon-themes/hicontrast/cmd/lc_view100.png
deleted file mode 100644
index 33af72d..000
--- a/icon-themes/hicontrast/cmd/lc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/hicontrast/cmd/sc_adjust.png 
b/icon-themes/hicontrast/cmd/sc_adjust.png
deleted file mode 100644
index fdaabf5..000
--- a/icon-themes/hicontrast/cmd/sc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/hicontrast/cmd/sc_view100.png 
b/icon-themes/hicontrast/cmd/sc_view100.png
deleted file mode 100644
index 2254149..000
--- a/icon-themes/hicontrast/cmd/sc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/human/cmd/lc_adjust.png 
b/icon-themes/human/cmd/lc_adjust.png
deleted file mode 100644
index 4beea80..000
--- a/icon-themes/human/cmd/lc_adjust.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/human/cmd/lc_view100.png 
b/icon-themes/human/cmd/lc_view100.png
deleted file mode 100644
index 2c52084..000
--- a/icon-themes/human/cmd/lc_view100.png
+++ /dev/null
Binary files differ
diff --git a/icon-themes/human/cmd/sc_adjust.png 
b/icon-themes/human/cmd

[Libreoffice-commits] core.git: officecfg/registry sfx2/sdi sfx2/source starmath/inc starmath/qa starmath/sdi starmath/source starmath/uiconfig

2013-05-30 Thread Rodolfo Ribeiro Gomes
 officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu |2 
 sfx2/sdi/appslots.sdi|8 
 sfx2/source/appl/appserv.cxx |   16 +
 starmath/inc/starmath.hrc|5 
 starmath/qa/cppunit/test_starmath.cxx|9 
 starmath/sdi/smath.sdi   |  124 
--
 starmath/sdi/smslots.sdi |   26 --
 starmath/source/smres.src|8 
 starmath/source/view.cxx |   21 -
 starmath/uiconfig/smath/toolbar/toolbar.xml  |6 
 10 files changed, 35 insertions(+), 190 deletions(-)

New commits:
commit ffc2e5be1f712b09710e2096ad2f7eb81b80118d
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Tue May 28 11:22:05 2013 -0300

Clean zoom redundances in Math and fix fdo#55929

Zoom can be handled by sfx2 in many ways:
- 50%, 75%, 100%, 150%, 200%
- Optimal view (fit in window)
- Entire page
- Page width

The math module was doing the first two by itself. Remove it.

Strange enough, state methods for  zoom interface definitions on
sfx2's appslots.sdi were needed. I thought 'Container' property
in sfx.sdi should do the job. It seems to do nothing, though.
(The zoom should be disabled only if the object is an OLE/Container).

The Help-Ids from pop-up menu in Math/Formula were kept, because
they doesn't exist in sfx2.

Change-Id: Ie1ae413780551b34aa36b338f9a9df79a198319c
Reviewed-on: https://gerrit.libreoffice.org/4076
Reviewed-by: Luboš Luňák l.lu...@suse.cz
Tested-by: Luboš Luňák l.lu...@suse.cz

diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
index 46ed65a..bbed900 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/GenericCommands.xcu
@@ -1773,7 +1773,7 @@
   /node
   node oor:name=.uno:ZoomOptimal oor:op=replace
 prop oor:name=Label oor:type=xs:string
-  value xml:lang=en-USOptimal/value
+  value xml:lang=en-USOptimal view/value
 /prop
 prop oor:name=Properties oor:type=xs:int
   value1/value
diff --git a/sfx2/sdi/appslots.sdi b/sfx2/sdi/appslots.sdi
index aa1da36..320a75e 100644
--- a/sfx2/sdi/appslots.sdi
+++ b/sfx2/sdi/appslots.sdi
@@ -182,34 +182,42 @@ interface Application
 SID_ZOOM_ENTIRE_PAGE
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_OPTIMAL
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_PAGE_WIDTH
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_50_PERCENT
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_75_PERCENT
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_100_PERCENT
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_150_PERCENT
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_ZOOM_200_PERCENT
 [
 ExecMethod = MiscExec_Impl ;
+StateMethod = MiscState_Impl ;
 ]
 SID_HELP_TUTORIALS
 [
diff --git a/sfx2/source/appl/appserv.cxx b/sfx2/source/appl/appserv.cxx
index 80a380f..825125d 100644
--- a/sfx2/source/appl/appserv.cxx
+++ b/sfx2/source/appl/appserv.cxx
@@ -827,6 +827,22 @@ void SfxApplication::MiscState_Impl(SfxItemSet rSet)
 }
 break;
 
+case SID_ZOOM_50_PERCENT:
+case SID_ZOOM_75_PERCENT:
+case SID_ZOOM_100_PERCENT:
+case SID_ZOOM_150_PERCENT:
+case SID_ZOOM_200_PERCENT:
+case SID_ZOOM_OPTIMAL:
+case SID_ZOOM_ENTIRE_PAGE:
+case SID_ZOOM_PAGE_WIDTH:
+{
+const SfxPoolItem *pItem;
+SfxItemState aState = 
SfxViewFrame::Current()-GetDispatcher()-QueryState(SID_ATTR_ZOOM, pItem);
+if ( aState == SFX_ITEM_DISABLED )
+rSet.DisableItem( nWhich );
+}
+break;
+
 default:
 break;
 }
diff --git a/starmath/inc/starmath.hrc b/starmath/inc/starmath.hrc
index f233fa4..3b2e902 100644
--- a/starmath/inc/starmath.hrc
+++ b/starmath/inc/starmath.hrc
@@ -31,13 +31,9 @@
 #define SID_PREVMARK

[PATCH] Clean zoom redundances in Math and fix fdo#55929

2013-05-28 Thread Rodolfo Ribeiro Gomes (via Code Review)
index 7104423..00e9875 100644
--- a/starmath/sdi/smslots.sdi
+++ b/starmath/sdi/smslots.sdi
@@ -176,21 +176,6 @@
 ExecMethod = Execute ;
 StateMethod = GetState ;
 ]
-SID_VIEW050 //idlpp ole : no , status : no
-[
-ExecMethod = Execute ;
-StateMethod = GetState ;
-]
-SID_VIEW100 //idlpp ole : no , status : no
-[
-ExecMethod = Execute ;
-StateMethod = GetState ;
-]
-SID_VIEW200 //idlpp ole : no , status : no
-[
-ExecMethod = Execute ;
-StateMethod = GetState ;
-]
 SID_ZOOMIN //idlpp ole : no , status : no
 [
 ExecMethod = Execute ;
@@ -202,17 +187,6 @@
 StateMethod = GetState ;
 ]
 SID_DRAW //idlpp ole : no , status : no
-[
-ExecMethod = Execute ;
-StateMethod = GetState ;
-]
-SID_ADJUST //idlpp ole : no , status : no
-[
-ExecMethod = Execute ;
-StateMethod = GetState ;
-]
-//idlpp kein Menueeintrag , also keine Texte
-SID_FITINWINDOW //idlpp ole : no , status : no
 [
 ExecMethod = Execute ;
 StateMethod = GetState ;
diff --git a/starmath/source/smres.src b/starmath/source/smres.src
index 805b290..b5a776e 100644
--- a/starmath/source/smres.src
+++ b/starmath/source/smres.src
@@ -838,19 +838,19 @@
 {
 MenuItem
 {
-Identifier = SID_VIEW050 ;
+Identifier = SID_ZOOM_50_PERCENT;
 HelpId = CMD_SID_VIEW050 ;
 Text [ en-US ] = ~View 50% ;
 };
 MenuItem
 {
-Identifier = SID_VIEW100 ;
+Identifier = SID_ZOOM_100_PERCENT ;
 HelpId = CMD_SID_VIEW100 ;
 Text [ en-US ] = View ~100% ;
 };
 MenuItem
 {
-Identifier = SID_VIEW200 ;
+Identifier = SID_ZOOM_200_PERCENT ;
 HelpId = CMD_SID_VIEW200 ;
 Text [ en-US ] = View ~200% ;
 };
@@ -868,7 +868,7 @@
 };
 MenuItem
 {
-Identifier = SID_ADJUST ;
+Identifier = SID_ZOOM_OPTIMAL ;
 HelpId = CMD_SID_ADJUST ;
 Text [ en-US ] = ~Display All ;
 };
diff --git a/starmath/source/view.cxx b/starmath/source/view.cxx
index d144dde..e6179b8 100644
--- a/starmath/source/view.cxx
+++ b/starmath/source/view.cxx
@@ -1569,21 +1569,8 @@
 }
 break;
 
-case SID_ADJUST:
-case SID_FITINWINDOW:
+case SID_ZOOM_OPTIMAL:
 aGraphic.ZoomToFitInWindow();
-break;
-
-case SID_VIEW050:
-aGraphic.SetZoom(50);
-break;
-
-case SID_VIEW100:
-aGraphic.SetZoom(100);
-break;
-
-case SID_VIEW200:
-aGraphic.SetZoom(200);
 break;
 
 case SID_ZOOMIN:
@@ -1925,13 +1912,9 @@
 case SID_ATTR_ZOOM:
 rSet.Put(SvxZoomItem( SVX_ZOOM_PERCENT, aGraphic.GetZoom()));
 /* no break here */
-case SID_VIEW050:
-case SID_VIEW100:
-case SID_VIEW200:
-case SID_ADJUST:
 case SID_ZOOMIN:
 case SID_ZOOMOUT:
-case SID_FITINWINDOW:
+case SID_ZOOM_OPTIMAL:
 if ( GetViewFrame()-GetFrame().IsInPlace() )
 rSet.DisableItem( nWh );
 break;
diff --git a/starmath/uiconfig/smath/toolbar/toolbar.xml 
b/starmath/uiconfig/smath/toolbar/toolbar.xml
index 7ac3103..ae74d5e 100644
--- a/starmath/uiconfig/smath/toolbar/toolbar.xml
+++ b/starmath/uiconfig/smath/toolbar/toolbar.xml
@@ -20,10 +20,10 @@
 toolbar:toolbar xmlns:toolbar=http://openoffice.org/2001/toolbar; 
xmlns:xlink=http://www.w3.org/1999/xlink; toolbar:id=toolbar
  toolbar:toolbaritem xlink:href=.uno:ZoomIn toolbar:text=/
  toolbar:toolbaritem xlink:href=.uno:ZoomOut toolbar:text=/
- toolbar:toolbaritem xlink:href=.uno:View100 toolbar:text=/
- toolbar:toolbaritem xlink:href=.uno:Adjust toolbar:text=/
+ toolbar:toolbaritem xlink:href=.uno:Zoom100Percent toolbar:text=/
+ toolbar:toolbaritem xlink:href=.uno:ZoomOptimal toolbar:text=/
  toolbar:toolbarseparator/
  toolbar:toolbaritem xlink:href=.uno:Draw toolbar:text=/
  toolbar:toolbaritem xlink:href=.uno:FormelCursor toolbar:text=/
  toolbar:toolbaritem xlink:href=.uno:SymbolCatalogue toolbar:text=/
-/toolbar:toolbar
\ No newline at end of file
+/toolbar:toolbar

-- 
To view, visit https://gerrit.libreoffice.org/4076
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie1ae413780551b34aa36b338f9a9df79a198319c
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[Libreoffice-commits] core.git: vcl/source

2013-05-21 Thread Rodolfo Ribeiro Gomes
 vcl/source/gdi/impimagetree.cxx |   65 +---
 1 file changed, 22 insertions(+), 43 deletions(-)

New commits:
commit a2c9d4f8bbde97f175bae4df771273a61251f402
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Tue May 14 20:16:02 2013 -0300

Little cleanup in image loader internal functions

Change-Id: I6def704dc22d7a536849a9e4b51a08bf68778329
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/3950
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Tested-by: Fridrich Strba fridr...@documentfoundation.org

diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index 6e449de..ea7d7cf 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -64,13 +64,13 @@
 
 namespace {
 
-OUString createPath(
+static OUString createPath(
 OUString const  name, sal_Int32 pos, OUString const  locale)
 {
 return name.copy(0, pos + 1) + locale + name.copy(pos);
 }
 
-boost::shared_ptr SvStream  wrapFile(osl::File  file)
+static boost::shared_ptr SvStream  wrapFile(osl::File  file)
 {
 // This could use SvInputStream instead if that did not have a broken
 // SeekPos implementation for an XInputStream that is not also XSeekable
@@ -90,22 +90,7 @@ boost::shared_ptr SvStream  wrapFile(osl::File  file)
 return s;
 }
 
-void loadFromFile(
-osl::File  file,
-OUString const  path, BitmapEx  bitmap)
-{
-boost::shared_ptr SvStream  s(wrapFile(file));
-if (path.endsWith(.png))
-{
-vcl::PNGReader aPNGReader( *s );
-aPNGReader.SetIgnoreGammaChunk( sal_True );
-bitmap = aPNGReader.Read();
-} else {
-*s  bitmap;
-}
-}
-
-boost::shared_ptr SvStream  wrapStream(
+static boost::shared_ptr SvStream  wrapStream(
 css::uno::Reference css::io::XInputStream  const  stream)
 {
 // This could use SvInputStream instead if that did not have a broken
@@ -127,18 +112,17 @@ boost::shared_ptr SvStream  wrapStream(
 return s;
 }
 
-void loadFromStream(
-css::uno::Reference css::io::XInputStream  const  stream,
-OUString const  path, BitmapEx  bitmap)
+static void loadImageFromStream(
+boost::shared_ptr SvStream  pStream,
+OUString const  rPath, BitmapEx  rBitmap)
 {
-boost::shared_ptr SvStream  s(wrapStream(stream));
-if (path.endsWith(.png))
+if (rPath.endsWith(.png))
 {
-vcl::PNGReader aPNGReader( *s );
+vcl::PNGReader aPNGReader( *pStream );
 aPNGReader.SetIgnoreGammaChunk( sal_True );
-bitmap = aPNGReader.Read();
+rBitmap = aPNGReader.Read();
 } else {
-*s  bitmap;
+*pStream  rBitmap;
 }
 }
 
@@ -271,27 +255,22 @@ void ImplImageTree::setStyle(OUString const  style) {
 
 void ImplImageTree::resetPaths() {
 m_paths.clear();
+
+OUString url( $BRAND_BASE_DIR/share/config/ );
+rtl::Bootstrap::expandMacros(url);
+if ( m_style != default )
 {
-OUString url(
-$BRAND_BASE_DIR/share/config);
-rtl::Bootstrap::expandMacros(url);
 INetURLObject u(url);
 OSL_ASSERT(!u.HasError());
 bool ok = u.Append(images_ + m_style, INetURLObject::ENCODE_ALL);
 OSL_ASSERT(ok); (void) ok;
-m_paths.push_back(
-std::make_pair(
-u.GetMainURL(INetURLObject::NO_DECODE),
-css::uno::Reference css::container::XNameAccess ()));
-}
-if ( m_style == default )
-{
-OUString url( $BRAND_BASE_DIR/share/config/images);
-rtl::Bootstrap::expandMacros(url);
-m_paths.push_back(
-std::make_pair(
-url, css::uno::Reference css::container::XNameAccess ()));
+url = u.GetMainURL(INetURLObject::NO_DECODE);
 }
+else
+url += images;
+m_paths.push_back(
+std::make_pair(
+url, css::uno::Reference css::container::XNameAccess ()));
 }
 
 bool ImplImageTree::checkStyleCacheLookup(
@@ -329,7 +308,7 @@ bool ImplImageTree::find(
 {
 osl::File file(i-first + / + *j);
 if (file.open(osl_File_OpenFlag_Read) == 
::osl::FileBase::E_None) {
-loadFromFile(file, *j, bitmap);
+loadImageFromStream( wrapFile(file), *j, bitmap );
 file.close();
 return true;
 }
@@ -361,7 +340,7 @@ bool ImplImageTree::find(
 css::uno::Reference css::io::XInputStream  s;
 bool ok = i-second-getByName(*j) = s;
 OSL_ASSERT(ok); (void) ok;
-loadFromStream(s, *j, bitmap);
+loadImageFromStream( wrapStream(s), *j, bitmap );
 return true;
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman

[Libreoffice-commits] core.git: Changes to 'refs/tags/libreoffice-4-1-branch-point'

2013-05-21 Thread Rodolfo Ribeiro Gomes
Tag 'libreoffice-4-1-branch-point' created by Petr Mladek pmla...@suse.cz at 
2013-05-21 11:42 -0700

Tag libreoffice-4-1-branch-point
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.9 (GNU/Linux)

iQIcBAABAgAGBQJRm0+lAAoJEPQ0oe+v7q6jRW0P/j27uSMEZvtUvncQTs2cgxNb
f+F07hSYINR6X03DerTaqozW1oBAMB0mc4mLCBmlu0aHP5Acor8fCQQ4KOm9xZl/
B7ybtrIXfFzvPgVG3pb/mQmBveg5HQqHCqBT+v9n+NtOxcl6ST6nB+jtIH2hOXGL
UAXib+UXG5mVan4es5OXZJEEr+TQo8l4XAzBR4PnGaLE/0jhlnBpyvHMn+lfiskO
jTds1ZR/j3P+6fSVLMKTGXl0pppJrKp8p9ZUXYJBpW20Gk1Gnj2i8nSSL4gaaGcv
P7QcgmAZ3SV2YnAHi4O2iFP50/dBvnaf4xC2Avec0hCN3hAcLa49hKefH2ESxNqm
ctONYVJqzrOQFOKb6IVh+C3wZ0wE3p1Gz/8QFwqjJEAmXqW6fM1uDG8kQsqB/gr0
ga3xKrWYCoCFkSUyPknr6IvhOBABLVUhaKijlUAMXMWVWiNKvsTTwbUIXITecL4h
gx2HrOQ/z9/EXbkfA0L27P5YRawJREE3Wa+Etyj7Y8oaE9QufNDdxsF41e/jw125
UUMwcM8Lvlo82bTbD9eI4YIvgBlDZDHENYgvt4N8mXXUTeLYrJEtheNeI9AWoPbK
YeAVvRhz9lhIXabaAPgN5y8WpSE3Fmj3j5vCIRUCZQ3enJPyHPpvXwfYloJhxp2l
r8bM8+4s9oXVVpbcxau7
=48Vr
-END PGP SIGNATURE-

Changes since libreoffice-4-0-branch-point-10062:
---
 0 files changed
---
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-05-21 Thread Rodolfo Ribeiro Gomes
 vcl/inc/impimagetree.hxx|   11 +---
 vcl/source/gdi/impimagetree.cxx |  107 +---
 2 files changed, 53 insertions(+), 65 deletions(-)

New commits:
commit 330ce827f2fbc6499eddf2084f8401fcdee5103b
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Wed May 15 01:11:52 2013 -0300

Loader's ImageTree doesn't store multiple image paths anymore

Change-Id: Idd13039a76c46bfccad5f54e3357dde952508211
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/3951
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Tested-by: Fridrich Strba fridr...@documentfoundation.org

diff --git a/vcl/inc/impimagetree.hxx b/vcl/inc/impimagetree.hxx
index f42a039..0dc3110 100644
--- a/vcl/inc/impimagetree.hxx
+++ b/vcl/inc/impimagetree.hxx
@@ -65,11 +65,10 @@ private:
 OUString const  name, OUString const  style,
 BitmapEx  bitmap, bool localized);
 
-typedef std::list
-std::pair
-OUString,
-com::sun::star::uno::Reference
-com::sun::star::container::XNameAccessPaths;
+typedef std::pair
+OUString,
+com::sun::star::uno::Reference
+com::sun::star::container::XNameAccess   Path;
 
 typedef boost::unordered_map
 OUString, bool, OUStringHash  CheckStyleCache;
@@ -77,7 +76,7 @@ private:
 OUString, std::pair bool, BitmapEx , OUStringHash  IconCache;
 
 OUString m_style;
-Paths m_paths;
+Path m_path;
 CheckStyleCache m_checkStyleCache;
 IconCache m_iconCache;
 bool m_cacheIcons;
diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index ea7d7cf..935b8cf 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -145,23 +145,21 @@ bool ImplImageTree::checkStyle(OUString const  style)
 setStyle(style);
 
 exists = false;
-for (Paths::iterator i(m_paths.begin()); i != m_paths.end()  !exists; 
++i) {
-OUString aURL = i-first;
+OUString aURL = m_path.first;
 
-osl::File aZip(aURL + .zip);
-if (aZip.open(osl_File_OpenFlag_Read) == ::osl::FileBase::E_None) {
-aZip.close();
-exists = true;
-}
+osl::File aZip(aURL + .zip);
+if (aZip.open(osl_File_OpenFlag_Read) == ::osl::FileBase::E_None) {
+aZip.close();
+exists = true;
+}
 
-osl::Directory aLookaside(aURL);
-if (aLookaside.open() == ::osl::FileBase::E_None) {
-aLookaside.close();
-exists = true;
-m_cacheIcons = false;
-} else {
-m_cacheIcons = true;
-}
+osl::Directory aLookaside(aURL);
+if (aLookaside.open() == ::osl::FileBase::E_None) {
+aLookaside.close();
+exists = true;
+m_cacheIcons = false;
+} else {
+m_cacheIcons = true;
 }
 m_checkStyleCache[style] = exists;
 return exists;
@@ -239,7 +237,6 @@ bool ImplImageTree::doLoadImage(
 void ImplImageTree::shutDown() {
 m_style = OUString();
 // for safety; empty m_style means not initialized
-m_paths.clear();
 m_iconCache.clear();
 m_checkStyleCache.clear();
 }
@@ -254,8 +251,6 @@ void ImplImageTree::setStyle(OUString const  style) {
 }
 
 void ImplImageTree::resetPaths() {
-m_paths.clear();
-
 OUString url( $BRAND_BASE_DIR/share/config/ );
 rtl::Bootstrap::expandMacros(url);
 if ( m_style != default )
@@ -268,9 +263,8 @@ void ImplImageTree::resetPaths() {
 }
 else
 url += images;
-m_paths.push_back(
-std::make_pair(
-url, css::uno::Reference css::container::XNameAccess ()));
+m_path = std::make_pair(
+url, css::uno::Reference css::container::XNameAccess ());
 }
 
 bool ImplImageTree::checkStyleCacheLookup(
@@ -301,50 +295,45 @@ bool ImplImageTree::find(
 std::vector OUString  const  paths, BitmapEx  bitmap)
 {
 if (!m_cacheIcons) {
-for (Paths::iterator i(m_paths.begin()); i != m_paths.end(); ++i) {
-for (std::vector OUString ::const_reverse_iterator j(
- paths.rbegin());
- j != paths.rend(); ++j)
-{
-osl::File file(i-first + / + *j);
-if (file.open(osl_File_OpenFlag_Read) == 
::osl::FileBase::E_None) {
-loadImageFromStream( wrapFile(file), *j, bitmap );
-file.close();
-return true;
-}
-}
-}
-}
-
-for (Paths::iterator i(m_paths.begin()); i != m_paths.end();) {
-if (!i-second.is()) {
-try {
-i-second.set(
-css::packages::zip::ZipFileAccess::createWithURL(
-   comphelper::getProcessComponentContext(),
-   i-first + .zip),
-css::uno::UNO_QUERY_THROW);
-} catch (css

[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-05-21 Thread Rodolfo Ribeiro Gomes
 vcl/inc/impimagetree.hxx|2 --
 vcl/source/gdi/impimagetree.cxx |   13 -
 2 files changed, 15 deletions(-)

New commits:
commit e8c9a6af25a7ff476e33ad239b70414168aabe3a
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Thu May 16 23:28:04 2013 -0300

Removal of includes not needed anymore

Change-Id: I5dccad8629a59e92dc2864e2b3630732a631ee96
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/3952
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Tested-by: Fridrich Strba fridr...@documentfoundation.org

diff --git a/vcl/inc/impimagetree.hxx b/vcl/inc/impimagetree.hxx
index 0dc3110..d22e770 100644
--- a/vcl/inc/impimagetree.hxx
+++ b/vcl/inc/impimagetree.hxx
@@ -22,8 +22,6 @@
 
 #include sal/config.h
 
-#include list
-#include utility
 #include vector
 
 #include boost/unordered_map.hpp
diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index 935b8cf..bb0013f 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -20,11 +20,6 @@
 
 #include sal/config.h
 
-#include list
-#include memory
-#include utility
-#include vector
-#include boost/unordered_map.hpp
 #include boost/shared_ptr.hpp
 
 #include com/sun/star/container/XNameAccess.hpp
@@ -34,7 +29,6 @@
 #include com/sun/star/packages/zip/ZipFileAccess.hpp
 #include com/sun/star/uno/Any.hxx
 #include com/sun/star/uno/Exception.hpp
-#include com/sun/star/uno/Reference.hxx
 #include com/sun/star/uno/RuntimeException.hpp
 #include com/sun/star/uno/Sequence.hxx
 
@@ -44,13 +38,6 @@
 #include osl/diagnose.h
 
 #include rtl/bootstrap.hxx
-#include rtl/string.h
-#include rtl/textenc.h
-#include rtl/ustrbuf.hxx
-#include rtl/ustring.h
-#include rtl/ustring.hxx
-
-#include sal/types.h
 
 #include tools/stream.hxx
 #include tools/urlobj.hxx
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: vcl/inc vcl/source

2013-05-21 Thread Rodolfo Ribeiro Gomes
 vcl/inc/impimagetree.hxx|7 +++
 vcl/source/gdi/impimagetree.cxx |   88 ++--
 2 files changed, 92 insertions(+), 3 deletions(-)

New commits:
commit d5151ab592367fde7db03fce81e0b76776d18216
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Wed May 15 08:05:05 2013 -0300

Allow 'textual links' on icon theme packages

This would allow use of better icon names ( fdo#30425 ) and
minimizes space size needed by reuses of same icons bitmaps
with different filenames.

Icon packages shoud have a file named links.txt on root folder
in order to achieve that. It binds a name of a 'virtual' file to
an existing one, to avoid duplicates.

This file should contain one or more lines like this:
path/for/fakefile_that_reuse_a_bitmap.png path/for/existing_one.png

Implemented as discussed in:
http://lists.freedesktop.org/archives/libreoffice/2013-April/049650.html

Change-Id: Ia65a1ba18f93297fae47fa520104821f6f336694
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/3953
Reviewed-by: Fridrich Strba fridr...@documentfoundation.org
Tested-by: Fridrich Strba fridr...@documentfoundation.org

diff --git a/vcl/inc/impimagetree.hxx b/vcl/inc/impimagetree.hxx
index d22e770..11c003e 100644
--- a/vcl/inc/impimagetree.hxx
+++ b/vcl/inc/impimagetree.hxx
@@ -72,12 +72,15 @@ private:
 OUString, bool, OUStringHash  CheckStyleCache;
 typedef boost::unordered_map
 OUString, std::pair bool, BitmapEx , OUStringHash  IconCache;
+typedef boost::unordered_map
+OUString, OUString, OUStringHash  IconLinkCache;
 
 OUString m_style;
 Path m_path;
 CheckStyleCache m_checkStyleCache;
 IconCache m_iconCache;
 bool m_cacheIcons;
+IconLinkCache m_iconLinkCache;
 
 void setStyle(OUString const  style );
 
@@ -87,6 +90,10 @@ private:
 bool iconCacheLookup( OUString const  name, bool localized, BitmapEx  
bitmap );
 
 bool find(std::vector OUString  const  paths, BitmapEx  bitmap );
+
+void loadImageLinks();
+void parseLinkFile(boost::shared_ptr SvStream  stream);
+OUString const  getRealImageName(OUString const  name);
 };
 
 typedef salhelper::SingletonRef ImplImageTree  ImplImageTreeSingletonRef;
diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index bb0013f..857a080 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -194,7 +194,7 @@ bool ImplImageTree::doLoadImage(
 bitmap.SetEmpty();
 }
 std::vector OUString  paths;
-paths.push_back(name);
+paths.push_back(getRealImageName(name));
 if (localized) {
 sal_Int32 pos = name.lastIndexOf('/');
 if (pos != -1) {
@@ -203,7 +203,7 @@ bool ImplImageTree::doLoadImage(
 for (std::vector OUString ::const_reverse_iterator it( 
aFallbacks.rbegin());
 it != aFallbacks.rend(); ++it)
 {
-paths.push_back(createPath(name, pos, *it));
+paths.push_back( getRealImageName( createPath(name, pos, *it) 
) );
 }
 }
 }
@@ -226,6 +226,7 @@ void ImplImageTree::shutDown() {
 // for safety; empty m_style means not initialized
 m_iconCache.clear();
 m_checkStyleCache.clear();
+m_iconLinkCache.clear();
 }
 
 void ImplImageTree::setStyle(OUString const  style) {
@@ -234,6 +235,8 @@ void ImplImageTree::setStyle(OUString const  style) {
 m_style = style;
 resetPaths();
 m_iconCache.clear();
+m_iconLinkCache.clear();
+loadImageLinks();
 }
 }
 
@@ -282,7 +285,8 @@ bool ImplImageTree::find(
 std::vector OUString  const  paths, BitmapEx  bitmap)
 {
 if (!m_cacheIcons) {
-for (std::vector OUString ::const_reverse_iterator j(paths.rbegin());
+for (std::vector OUString ::const_reverse_iterator j(
+ paths.rbegin());
  j != paths.rend(); ++j)
 {
 osl::File file(m_path.first + / + *j);
@@ -325,4 +329,82 @@ bool ImplImageTree::find(
 return false;
 }
 
+void ImplImageTree::loadImageLinks()
+{
+const OUString aLinkFilename(links.txt);
+
+if (!m_cacheIcons)
+{
+osl::File file(m_path.first + / + aLinkFilename);
+if (file.open(osl_File_OpenFlag_Read) == ::osl::FileBase::E_None)
+{
+parseLinkFile( wrapFile(file) );
+file.close();
+return;
+}
+}
+
+if ( !m_path.second.is() )
+{
+css::uno::Sequence css::uno::Any  args(1);
+args[0] = m_path.first + .zip;
+try
+{
+m_path.second.set(
+
comphelper::getProcessServiceFactory()-createInstanceWithArguments(
+OUString( com.sun.star.packages.zip.ZipFileAccess),
+args),
+css::uno::UNO_QUERY_THROW

[Libreoffice-commits] core.git: vcl/source

2013-05-20 Thread Rodolfo Ribeiro Gomes
 vcl/source/gdi/impimagetree.cxx |   10 ++
 1 file changed, 2 insertions(+), 8 deletions(-)

New commits:
commit e673655ba784bd2993fb4e364c042fbc0e938d2d
Author: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Date:   Mon May 13 21:42:31 2013 -0300

Make some uses of OUStrings a little more readable

Change-Id: I2e85fa39abbff86918464aae67b4b9eacd3bf7a4
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
Reviewed-on: https://gerrit.libreoffice.org/3949
Reviewed-by: Luboš Luňák l.lu...@suse.cz
Tested-by: Luboš Luňák l.lu...@suse.cz

diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index a2cab8b..6e449de 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -67,10 +67,7 @@ namespace {
 OUString createPath(
 OUString const  name, sal_Int32 pos, OUString const  locale)
 {
-OUStringBuffer b(name.copy(0, pos + 1));
-b.append(locale);
-b.append(name.copy(pos));
-return b.makeStringAndClear();
+return name.copy(0, pos + 1) + locale + name.copy(pos);
 }
 
 boost::shared_ptr SvStream  wrapFile(osl::File  file)
@@ -280,10 +277,7 @@ void ImplImageTree::resetPaths() {
 rtl::Bootstrap::expandMacros(url);
 INetURLObject u(url);
 OSL_ASSERT(!u.HasError());
-OUStringBuffer b;
-b.appendAscii(images_);
-b.append(m_style);
-bool ok = u.Append(b.makeStringAndClear(), INetURLObject::ENCODE_ALL);
+bool ok = u.Append(images_ + m_style, INetURLObject::ENCODE_ALL);
 OSL_ASSERT(ok); (void) ok;
 m_paths.push_back(
 std::make_pair(
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[PATCH] Make some uses of OUStrings a little more readable

2013-05-17 Thread Rodolfo Ribeiro Gomes (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3949

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/49/3949/1

Make some uses of OUStrings a little more readable

Change-Id: I2e85fa39abbff86918464aae67b4b9eacd3bf7a4
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
---
M vcl/source/gdi/impimagetree.cxx
1 file changed, 2 insertions(+), 8 deletions(-)



diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index 7a2fbdd..bcdfd01 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -66,10 +66,7 @@
 OUString createPath(
 OUString const  name, sal_Int32 pos, OUString const  locale)
 {
-OUStringBuffer b(name.copy(0, pos + 1));
-b.append(locale);
-b.append(name.copy(pos));
-return b.makeStringAndClear();
+return name.copy(0, pos + 1) + locale + name.copy(pos);
 }
 
 boost::shared_ptr SvStream  wrapFile(osl::File  file)
@@ -279,10 +276,7 @@
 rtl::Bootstrap::expandMacros(url);
 INetURLObject u(url);
 OSL_ASSERT(!u.HasError());
-OUStringBuffer b;
-b.appendAscii(images_);
-b.append(m_style);
-bool ok = u.Append(b.makeStringAndClear(), INetURLObject::ENCODE_ALL);
+bool ok = u.Append(images_ + m_style, INetURLObject::ENCODE_ALL);
 OSL_ASSERT(ok); (void) ok;
 m_paths.push_back(
 std::make_pair(

-- 
To view, visit https://gerrit.libreoffice.org/3949
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2e85fa39abbff86918464aae67b4b9eacd3bf7a4
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Little cleanup in image loader internal functions

2013-05-17 Thread Rodolfo Ribeiro Gomes (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3950

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/50/3950/1

Little cleanup in image loader internal functions

Change-Id: I6def704dc22d7a536849a9e4b51a08bf68778329
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
---
M vcl/source/gdi/impimagetree.cxx
1 file changed, 22 insertions(+), 43 deletions(-)



diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index bcdfd01..97fa471 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -63,13 +63,13 @@
 
 namespace {
 
-OUString createPath(
+static OUString createPath(
 OUString const  name, sal_Int32 pos, OUString const  locale)
 {
 return name.copy(0, pos + 1) + locale + name.copy(pos);
 }
 
-boost::shared_ptr SvStream  wrapFile(osl::File  file)
+static boost::shared_ptr SvStream  wrapFile(osl::File  file)
 {
 // This could use SvInputStream instead if that did not have a broken
 // SeekPos implementation for an XInputStream that is not also XSeekable
@@ -89,22 +89,7 @@
 return s;
 }
 
-void loadFromFile(
-osl::File  file,
-OUString const  path, BitmapEx  bitmap)
-{
-boost::shared_ptr SvStream  s(wrapFile(file));
-if (path.endsWith(.png))
-{
-vcl::PNGReader aPNGReader( *s );
-aPNGReader.SetIgnoreGammaChunk( sal_True );
-bitmap = aPNGReader.Read();
-} else {
-*s  bitmap;
-}
-}
-
-boost::shared_ptr SvStream  wrapStream(
+static boost::shared_ptr SvStream  wrapStream(
 css::uno::Reference css::io::XInputStream  const  stream)
 {
 // This could use SvInputStream instead if that did not have a broken
@@ -126,18 +111,17 @@
 return s;
 }
 
-void loadFromStream(
-css::uno::Reference css::io::XInputStream  const  stream,
-OUString const  path, BitmapEx  bitmap)
+static void loadImageFromStream(
+boost::shared_ptr SvStream  pStream,
+OUString const  rPath, BitmapEx  rBitmap)
 {
-boost::shared_ptr SvStream  s(wrapStream(stream));
-if (path.endsWith(.png))
+if (rPath.endsWith(.png))
 {
-vcl::PNGReader aPNGReader( *s );
+vcl::PNGReader aPNGReader( *pStream );
 aPNGReader.SetIgnoreGammaChunk( sal_True );
-bitmap = aPNGReader.Read();
+rBitmap = aPNGReader.Read();
 } else {
-*s  bitmap;
+*pStream  rBitmap;
 }
 }
 
@@ -270,27 +254,22 @@
 
 void ImplImageTree::resetPaths() {
 m_paths.clear();
+
+OUString url( $BRAND_BASE_DIR/share/config/ );
+rtl::Bootstrap::expandMacros(url);
+if ( m_style != default )
 {
-OUString url(
-$BRAND_BASE_DIR/share/config);
-rtl::Bootstrap::expandMacros(url);
 INetURLObject u(url);
 OSL_ASSERT(!u.HasError());
 bool ok = u.Append(images_ + m_style, INetURLObject::ENCODE_ALL);
 OSL_ASSERT(ok); (void) ok;
-m_paths.push_back(
-std::make_pair(
-u.GetMainURL(INetURLObject::NO_DECODE),
-css::uno::Reference css::container::XNameAccess ()));
+url = u.GetMainURL(INetURLObject::NO_DECODE);
 }
-if ( m_style == default )
-{
-OUString url( $BRAND_BASE_DIR/share/config/images);
-rtl::Bootstrap::expandMacros(url);
-m_paths.push_back(
-std::make_pair(
-url, css::uno::Reference css::container::XNameAccess ()));
-}
+else
+url += images;
+m_paths.push_back(
+std::make_pair(
+url, css::uno::Reference css::container::XNameAccess ()));
 }
 
 bool ImplImageTree::checkStyleCacheLookup(
@@ -328,7 +307,7 @@
 {
 osl::File file(i-first + / + *j);
 if (file.open(osl_File_OpenFlag_Read) == 
::osl::FileBase::E_None) {
-loadFromFile(file, *j, bitmap);
+loadImageFromStream( wrapFile(file), *j, bitmap );
 file.close();
 return true;
 }
@@ -362,7 +341,7 @@
 css::uno::Reference css::io::XInputStream  s;
 bool ok = i-second-getByName(*j) = s;
 OSL_ASSERT(ok); (void) ok;
-loadFromStream(s, *j, bitmap);
+loadImageFromStream( wrapStream(s), *j, bitmap );
 return true;
 }
 }

-- 
To view, visit https://gerrit.libreoffice.org/3950
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6def704dc22d7a536849a9e4b51a08bf68778329
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Removal of includes not needed anymore

2013-05-17 Thread Rodolfo Ribeiro Gomes (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3952

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/52/3952/1

Removal of includes not needed anymore

Change-Id: I5dccad8629a59e92dc2864e2b3630732a631ee96
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
---
M vcl/inc/impimagetree.hxx
M vcl/source/gdi/impimagetree.cxx
2 files changed, 0 insertions(+), 15 deletions(-)



diff --git a/vcl/inc/impimagetree.hxx b/vcl/inc/impimagetree.hxx
index 0dc3110..d22e770 100644
--- a/vcl/inc/impimagetree.hxx
+++ b/vcl/inc/impimagetree.hxx
@@ -22,8 +22,6 @@
 
 #include sal/config.h
 
-#include list
-#include utility
 #include vector
 
 #include boost/unordered_map.hpp
diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index 86e2f22..fe252fa 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -20,11 +20,6 @@
 
 #include sal/config.h
 
-#include list
-#include memory
-#include utility
-#include vector
-#include boost/unordered_map.hpp
 #include boost/shared_ptr.hpp
 
 #include com/sun/star/container/XNameAccess.hpp
@@ -33,7 +28,6 @@
 #include com/sun/star/lang/XMultiServiceFactory.hpp
 #include com/sun/star/uno/Any.hxx
 #include com/sun/star/uno/Exception.hpp
-#include com/sun/star/uno/Reference.hxx
 #include com/sun/star/uno/RuntimeException.hpp
 #include com/sun/star/uno/Sequence.hxx
 
@@ -43,13 +37,6 @@
 #include osl/diagnose.h
 
 #include rtl/bootstrap.hxx
-#include rtl/string.h
-#include rtl/textenc.h
-#include rtl/ustrbuf.hxx
-#include rtl/ustring.h
-#include rtl/ustring.hxx
-
-#include sal/types.h
 
 #include tools/stream.hxx
 #include tools/urlobj.hxx

-- 
To view, visit https://gerrit.libreoffice.org/3952
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5dccad8629a59e92dc2864e2b3630732a631ee96
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Rodolfo Ribeiro Gomes l...@rodolfo.eng.br

___
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


[PATCH] Allow 'textual links' on icon theme packages

2013-05-17 Thread Rodolfo Ribeiro Gomes (via Code Review)
Hi,

I have submitted a patch for review:

https://gerrit.libreoffice.org/3953

To pull it, you can do:

git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/53/3953/1

Allow 'textual links' on icon theme packages

This would allow use of better icon names ( fdo#30425 ) and
minimizes space size needed by reuses of same icons bitmaps
with different filenames.

Icon packages shoud have a file named links.txt on root folder
in order to achieve that. It binds a name of a 'virtual' file to
an existing one, to avoid duplicates.

This file should contain one or more lines like this:
path/for/fakefile_that_reuse_a_bitmap.png path/for/existing_one.png

Implemented as discussed in:
http://lists.freedesktop.org/archives/libreoffice/2013-April/049650.html

Change-Id: Ia65a1ba18f93297fae47fa520104821f6f336694
Signed-off-by: Rodolfo Ribeiro Gomes rodolf...@gmail.com
---
M vcl/inc/impimagetree.hxx
M vcl/source/gdi/impimagetree.cxx
2 files changed, 92 insertions(+), 3 deletions(-)



diff --git a/vcl/inc/impimagetree.hxx b/vcl/inc/impimagetree.hxx
index d22e770..11c003e 100644
--- a/vcl/inc/impimagetree.hxx
+++ b/vcl/inc/impimagetree.hxx
@@ -72,12 +72,15 @@
 OUString, bool, OUStringHash  CheckStyleCache;
 typedef boost::unordered_map
 OUString, std::pair bool, BitmapEx , OUStringHash  IconCache;
+typedef boost::unordered_map
+OUString, OUString, OUStringHash  IconLinkCache;
 
 OUString m_style;
 Path m_path;
 CheckStyleCache m_checkStyleCache;
 IconCache m_iconCache;
 bool m_cacheIcons;
+IconLinkCache m_iconLinkCache;
 
 void setStyle(OUString const  style );
 
@@ -87,6 +90,10 @@
 bool iconCacheLookup( OUString const  name, bool localized, BitmapEx  
bitmap );
 
 bool find(std::vector OUString  const  paths, BitmapEx  bitmap );
+
+void loadImageLinks();
+void parseLinkFile(boost::shared_ptr SvStream  stream);
+OUString const  getRealImageName(OUString const  name);
 };
 
 typedef salhelper::SingletonRef ImplImageTree  ImplImageTreeSingletonRef;
diff --git a/vcl/source/gdi/impimagetree.cxx b/vcl/source/gdi/impimagetree.cxx
index fe252fa..b8426e7 100644
--- a/vcl/source/gdi/impimagetree.cxx
+++ b/vcl/source/gdi/impimagetree.cxx
@@ -193,7 +193,7 @@
 bitmap.SetEmpty();
 }
 std::vector OUString  paths;
-paths.push_back(name);
+paths.push_back(getRealImageName(name));
 if (localized) {
 sal_Int32 pos = name.lastIndexOf('/');
 if (pos != -1) {
@@ -202,7 +202,7 @@
 for (std::vector OUString ::const_reverse_iterator it( 
aFallbacks.rbegin());
 it != aFallbacks.rend(); ++it)
 {
-paths.push_back(createPath(name, pos, *it));
+paths.push_back( getRealImageName( createPath(name, pos, *it) 
) );
 }
 }
 }
@@ -225,6 +225,7 @@
 // for safety; empty m_style means not initialized
 m_iconCache.clear();
 m_checkStyleCache.clear();
+m_iconLinkCache.clear();
 }
 
 void ImplImageTree::setStyle(OUString const  style) {
@@ -233,6 +234,8 @@
 m_style = style;
 resetPaths();
 m_iconCache.clear();
+m_iconLinkCache.clear();
+loadImageLinks();
 }
 }
 
@@ -281,7 +284,8 @@
 std::vector OUString  const  paths, BitmapEx  bitmap)
 {
 if (!m_cacheIcons) {
-for (std::vector OUString ::const_reverse_iterator j(paths.rbegin());
+for (std::vector OUString ::const_reverse_iterator j(
+ paths.rbegin());
  j != paths.rend(); ++j)
 {
 osl::File file(m_path.first + / + *j);
@@ -324,4 +328,82 @@
 return false;
 }
 
+void ImplImageTree::loadImageLinks()
+{
+const OUString aLinkFilename(links.txt);
+
+if (!m_cacheIcons)
+{
+osl::File file(m_path.first + / + aLinkFilename);
+if (file.open(osl_File_OpenFlag_Read) == ::osl::FileBase::E_None)
+{
+parseLinkFile( wrapFile(file) );
+file.close();
+return;
+}
+}
+
+if ( !m_path.second.is() )
+{
+css::uno::Sequence css::uno::Any  args(1);
+args[0] = m_path.first + .zip;
+try
+{
+m_path.second.set(
+
comphelper::getProcessServiceFactory()-createInstanceWithArguments(
+OUString( com.sun.star.packages.zip.ZipFileAccess),
+args),
+css::uno::UNO_QUERY_THROW);
+}
+catch (css::uno::RuntimeException )
+{
+throw;
+}
+catch (const css::uno::Exception  e)
+{
+SAL_INFO(vcl, ImplImageTree::find exception 
+ e.Message   for   m_path.first);
+return;
+}
+}
+if ( m_path.second-hasByName(aLinkFilename) )
+{
+css::uno::Reference css::io::XInputStream  s;
+bool ok = m_path.second-getByName(aLinkFilename) = s;
+OSL_ASSERT(ok

  1   2   >