[Libreoffice-commits] .: cui/source sfx2/inc sw/source

Caolán McNamara caolan at kemper.freedesktop.org
Wed Nov 9 08:46:38 PST 2011


 cui/source/tabpages/border.cxx           |   11 -----------
 cui/source/tabpages/swpossizetabpage.cxx |    5 -----
 sfx2/inc/sfx2/htmlmode.hxx               |    1 -
 sw/source/ui/config/viewopt.cxx          |    4 ++--
 sw/source/ui/frmdlg/frmpage.cxx          |    7 -------
 sw/source/ui/frmdlg/wrap.cxx             |   21 ---------------------
 6 files changed, 2 insertions(+), 47 deletions(-)

New commits:
commit 090454c4c99498a79d87c55e1903132bd068d810
Author: Harri Pitkänen <hatapitk at iki.fi>
Date:   Tue Nov 8 19:36:21 2011 +0200

    Enable features behind HTMLMODE_FULL_ABS_POS for all HTML export modes
    
    HTMLMODE_FULL_ABS_POS appears to have been used for enabling accurate
    margin settings for absolutely positioned graphics objects. If it was
    disabled one could not set (in HTML editor mode) separate margins for
    left and right (or bottom/top) sides of the image. The setting was
    disabled for Netscape and enabled for other browsers.
    
    This patch removes the setting and unconditionally enables the feature
    in HTML editor mode. All modern browsers support
    margin-{left,right,bottom,top} CSS properties.

diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx
index 98d62e9..d6c474f 100644
--- a/cui/source/tabpages/border.cxx
+++ b/cui/source/tabpages/border.cxx
@@ -559,17 +559,6 @@ void SvxBorderTabPage::Reset( const SfxItemSet& rSet )
             aLbShadowColor.Disable();
             aFlShadow     .Disable();
 
-            sal_uInt16 nLBCount = aLbLineStyle.GetEntryCount();
-            // ist es ein Absatzdialog, dann alle Linien fuer
-            // Sw-Export, sonst ist die Page nicht da
-            if(!(mbHorEnabled || mbVerEnabled)
-                 && 0 == (nHtmlMode & HTMLMODE_FULL_ABS_POS) &&
-                SFX_ITEM_AVAILABLE > rSet.GetItemState(GetWhich( SID_ATTR_PARA_LINESPACE )))
-            {
-                for( sal_uInt16 i = nLBCount - 1; i > LINESTYLE_HTML_MAX; --i)
-                    aLbLineStyle.RemoveEntry(i);
-            }
-
             if( !(nSWMode & SW_BORDER_MODE_TABLE) )
             {
                 aUserDefFT.Disable();
diff --git a/cui/source/tabpages/swpossizetabpage.cxx b/cui/source/tabpages/swpossizetabpage.cxx
index 2d08ded..7874eaa 100644
--- a/cui/source/tabpages/swpossizetabpage.cxx
+++ b/cui/source/tabpages/swpossizetabpage.cxx
@@ -895,11 +895,6 @@ void SvxSwPosSizeTabPage::Reset( const SfxItemSet& rSet)
 
     if(m_bHtmlMode)
     {
-        if( 0 == (m_nHtmlMode & HTMLMODE_FULL_ABS_POS))
-        {
-            m_aHeightFT .Enable( sal_False );
-            m_aHeightMF .Enable( sal_False );
-        }
         if( 0 == (m_nHtmlMode & HTMLMODE_SOME_ABS_POS))
         {
             if(GetAnchorType() == TextContentAnchorType_AT_PAGE)
diff --git a/sfx2/inc/sfx2/htmlmode.hxx b/sfx2/inc/sfx2/htmlmode.hxx
index ce1af6f..65e41f6 100644
--- a/sfx2/inc/sfx2/htmlmode.hxx
+++ b/sfx2/inc/sfx2/htmlmode.hxx
@@ -37,7 +37,6 @@
 #define HTMLMODE_PARA_BLOCK         0x0100
 #define HTMLMODE_DROPCAPS           0x0200
 #define HTMLMODE_FIRSTLINE          0x0400 /* First-line intent with Spacer == NS 3.0 */
-#define HTMLMODE_FULL_ABS_POS       0x1000
 #define HTMLMODE_SOME_ABS_POS       0x2000
 #define HTMLMODE_RESERVED1          0x4000
 #define HTMLMODE_RESERVED0          0x8000
diff --git a/sw/source/ui/config/viewopt.cxx b/sw/source/ui/config/viewopt.cxx
index a34679c..32093ab 100644
--- a/sw/source/ui/config/viewopt.cxx
+++ b/sw/source/ui/config/viewopt.cxx
@@ -340,7 +340,7 @@ sal_uInt16      GetHtmlMode(const SwDocShell* pShell)
         {
             case HTML_CFG_MSIE:
                 nRet |= HTMLMODE_FULL_STYLES|
-                        HTMLMODE_FULL_ABS_POS|HTMLMODE_SOME_ABS_POS;
+                        HTMLMODE_SOME_ABS_POS;
             break;
             case HTML_CFG_NS40:
                 nRet |= HTMLMODE_FRM_COLUMNS|
@@ -349,7 +349,7 @@ sal_uInt16      GetHtmlMode(const SwDocShell* pShell)
             case HTML_CFG_WRITER:
                 nRet |= HTMLMODE_FRM_COLUMNS|HTMLMODE_FULL_STYLES|
                         HTMLMODE_DROPCAPS|
-                        HTMLMODE_FULL_ABS_POS|HTMLMODE_SOME_ABS_POS;
+                        HTMLMODE_SOME_ABS_POS;
             break;
         }
     }
diff --git a/sw/source/ui/frmdlg/frmpage.cxx b/sw/source/ui/frmdlg/frmpage.cxx
index b321d49..744ab01 100644
--- a/sw/source/ui/frmdlg/frmpage.cxx
+++ b/sw/source/ui/frmdlg/frmpage.cxx
@@ -908,13 +908,6 @@ void SwFrmPage::Reset( const SfxItemSet &rSet )
 
     if(bHtmlMode)
     {
-        if(nDlgType == DLG_FRM_STD &&
-                0 == (nHtmlMode & HTMLMODE_FULL_ABS_POS))
-        {
-            aHeightFT   .Enable( sal_False );
-            aHeightED   .Enable( sal_False );
-            aRelHeightCB.Enable( sal_False );
-        }
         if( 0 == (nHtmlMode & HTMLMODE_SOME_ABS_POS))
         {
             if (GetAnchor() == FLY_AT_PAGE)
diff --git a/sw/source/ui/frmdlg/wrap.cxx b/sw/source/ui/frmdlg/wrap.cxx
index 75b99eb..f7ce654 100644
--- a/sw/source/ui/frmdlg/wrap.cxx
+++ b/sw/source/ui/frmdlg/wrap.cxx
@@ -480,12 +480,6 @@ void SwWrapTabPage::ActivatePage(const SfxItemSet& rSet)
         nTop    = aVal.nVPos - aVal.nMinVPos;
         nBottom = aVal.nMaxHeight - aVal.nHeight;
 
-        if (bHtmlMode && 0 == (nHtmlMode & HTMLMODE_FULL_ABS_POS))
-        {
-            nLeft = nRight = ( nLeft + nRight ) / 2;
-            nTop = nBottom = ( nTop + nBottom ) / 2;
-        }
-        else
         {
             if (aVal.nAnchorType == FLY_AS_CHAR)
             {
@@ -620,20 +614,6 @@ int SwWrapTabPage::DeactivatePage(SfxItemSet* _pSet)
  --------------------------------------------------------------------*/
 IMPL_LINK( SwWrapTabPage, RangeModifyHdl, MetricField *, pEdit )
 {
-    if(bHtmlMode && 0 == (nHtmlMode & HTMLMODE_FULL_ABS_POS))
-    {
-        sal_Int64 nValue = pEdit->GetValue();
-        if (pEdit == &aLeftMarginED)
-            aRightMarginED.SetUserValue(nValue, FUNIT_NONE);
-        else if (pEdit == &aRightMarginED)
-            aLeftMarginED.SetUserValue(nValue, FUNIT_NONE);
-        else if (pEdit == &aTopMarginED)
-            aBottomMarginED.SetUserValue(nValue, FUNIT_NONE);
-        else if (pEdit == &aBottomMarginED)
-            aTopMarginED.SetUserValue(nValue, FUNIT_NONE);
-    }
-    else
-    {
         sal_Int64 nValue = pEdit->GetValue();
         MetricField *pOpposite = 0;
 
@@ -655,7 +635,6 @@ IMPL_LINK( SwWrapTabPage, RangeModifyHdl, MetricField *, pEdit )
             if (nValue + nOpposite > Max(pEdit->GetMax(), pOpposite->GetMax()))
                 pOpposite->SetValue(pOpposite->GetMax() - nValue);
         }
-    }
 
     return 0;
 }


More information about the Libreoffice-commits mailing list