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.

6 files changed, 2 insertions, 47 deletions

diff --git a/cui/source/tabpages/border.cxx b/cui/source/tabpages/border.cxx
index 98d62e95eab8..d6c474f7c247 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 2d08dede00ce..7874eaaeb8f3 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 ce1af6feb970..65e41f65adc5 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 a34679c5191a..32093ab31d66 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 b321d4951517..744ab01797bc 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 75b99eb17c70..f7ce65494d0e 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;
 }