replace use of deprecated GtkPixmap, gtk_container_border_width

[wxWidgets.git] / src / html / m_list.cpp

diff --git a/src/html/m_list.cpp b/src/html/m_list.cpp
index f1b90e44910d00f78cafd4e8c5afc06b169ae4a5..2328b748b874e65ff267236cab42f57e6d7ba032 100644 (file)
--- a/src/html/m_list.cpp
+++ b/src/html/m_list.cpp
@@ -1,25 +1,20 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        m_list.cpp
+// Name:        src/html/m_list.cpp

 // Purpose:     wxHtml module for lists
 // Author:      Vaclav Slavik
 // RCS-ID:      $Id$
 // Copyright:   (c) 1999 Vaclav Slavik
 // Purpose:     wxHtml module for lists
 // Author:      Vaclav Slavik
 // RCS-ID:      $Id$
 // Copyright:   (c) 1999 Vaclav Slavik

-// Licence:     wxWidgets Licence
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
-#pragma implementation
-#endif

 
 #include "wx/wxprec.h"
 
 
 #include "wx/wxprec.h"
 

-
-#include "wx/defs.h"
-#if wxUSE_HTML && wxUSE_STREAMS
-

 #ifdef __BORLANDC__
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #endif
 

+#if wxUSE_HTML && wxUSE_STREAMS
+

 #ifndef WXPRECOMP
     #include "wx/brush.h"
     #include "wx/dc.h"
 #ifndef WXPRECOMP
     #include "wx/brush.h"
     #include "wx/dc.h"


@@ -53,7 +48,8 @@ wxHtmlListmarkCell::wxHtmlListmarkCell(wxDC* dc, const wxColour& clr) : wxHtmlCe
 {
     m_Width =  dc->GetCharHeight();
     m_Height = dc->GetCharHeight();
 {
     m_Width =  dc->GetCharHeight();
     m_Height = dc->GetCharHeight();

-    m_Descent = 0;
+    // bottom of list mark is lined with bottom of letters in next cell
+    m_Descent = m_Height / 3;

 }
 
 
 }
 
 


@@ -63,7 +59,7 @@ void wxHtmlListmarkCell::Draw(wxDC& dc, int x, int y,
                               wxHtmlRenderingInfo& WXUNUSED(info))
 {
     dc.SetBrush(m_Brush);
                               wxHtmlRenderingInfo& WXUNUSED(info))
 {
     dc.SetBrush(m_Brush);

-    dc.DrawEllipse(x + m_PosX + m_Width / 3, y + m_PosY + m_Height / 3, 
+    dc.DrawEllipse(x + m_PosX + m_Width / 3, y + m_PosY + m_Height / 3,

                    (m_Width / 3), (m_Width / 3));
 }
 
                    (m_Width / 3), (m_Width / 3));
 }
 


@@ -88,6 +84,7 @@ class wxHtmlListCell : public wxHtmlContainerCell
         wxHtmlListItemStruct *m_RowInfo;
         void ReallocRows(int rows);
         void ComputeMinMaxWidths();
         wxHtmlListItemStruct *m_RowInfo;
         void ReallocRows(int rows);
         void ComputeMinMaxWidths();

+        int ComputeMaxBase(wxHtmlCell *cell);

         int m_ListmarkWidth;
 
     public:
         int m_ListmarkWidth;
 
     public:


@@ -95,7 +92,7 @@ class wxHtmlListCell : public wxHtmlContainerCell
         virtual ~wxHtmlListCell();
         void AddRow(wxHtmlContainerCell *mark, wxHtmlContainerCell *cont);
         virtual void Layout(int w);
         virtual ~wxHtmlListCell();
         void AddRow(wxHtmlContainerCell *mark, wxHtmlContainerCell *cont);
         virtual void Layout(int w);

-        
+

     DECLARE_NO_COPY_CLASS(wxHtmlListCell)
 };
 
     DECLARE_NO_COPY_CLASS(wxHtmlListCell)
 };
 


@@ -111,6 +108,23 @@ wxHtmlListCell::~wxHtmlListCell()
     if (m_RowInfo) free(m_RowInfo);
 }
 
     if (m_RowInfo) free(m_RowInfo);
 }
 

+int wxHtmlListCell::ComputeMaxBase(wxHtmlCell *cell)
+{
+    if(!cell)
+        return 0;
+
+    wxHtmlCell *child = cell->GetFirstChild();
+
+    while(child)
+    {
+        int base = ComputeMaxBase( child );
+        if ( base > 0 ) return base + child->GetPosY();
+        child = child->GetNext();
+    }
+
+    return cell->GetHeight() - cell->GetDescent();
+}
+

 void wxHtmlListCell::Layout(int w)
 {
     wxHtmlCell::Layout(w);
 void wxHtmlListCell::Layout(int w)
 {
     wxHtmlCell::Layout(w);


@@ -123,11 +137,20 @@ void wxHtmlListCell::Layout(int w)
     int vpos = 0;
     for (int r = 0; r < m_NumRows; r++)
     {
     int vpos = 0;
     for (int r = 0; r < m_NumRows; r++)
     {

+        // do layout first time to layout contents and adjust pos

         m_RowInfo[r].mark->Layout(m_ListmarkWidth);
         m_RowInfo[r].mark->Layout(m_ListmarkWidth);

-        m_RowInfo[r].mark->SetPos(m_IndentLeft, vpos);

         m_RowInfo[r].cont->Layout(s_width - m_ListmarkWidth);
         m_RowInfo[r].cont->Layout(s_width - m_ListmarkWidth);

-        m_RowInfo[r].cont->SetPos(m_IndentLeft + m_ListmarkWidth, vpos);
-        vpos += wxMax(m_RowInfo[r].cont->GetHeight(), m_RowInfo[r].mark->GetHeight());
+
+        const int base_mark = ComputeMaxBase( m_RowInfo[r].mark );
+        const int base_cont = ComputeMaxBase( m_RowInfo[r].cont );
+        const int adjust_mark = vpos + wxMax(base_cont-base_mark,0);
+        const int adjust_cont = vpos + wxMax(base_mark-base_cont,0);
+
+        m_RowInfo[r].mark->SetPos(m_IndentLeft, adjust_mark);
+        m_RowInfo[r].cont->SetPos(m_IndentLeft + m_ListmarkWidth, adjust_cont);
+
+        vpos = wxMax(adjust_mark + m_RowInfo[r].mark->GetHeight(),
+                     adjust_cont + m_RowInfo[r].cont->GetHeight());

     }
     m_Height = vpos;
 }
     }
     m_Height = vpos;
 }


@@ -142,8 +165,8 @@ void wxHtmlListCell::AddRow(wxHtmlContainerCell *mark, wxHtmlContainerCell *cont
 void wxHtmlListCell::ReallocRows(int rows)
 {
     m_RowInfo = (wxHtmlListItemStruct*) realloc(m_RowInfo, sizeof(wxHtmlListItemStruct) * rows);
 void wxHtmlListCell::ReallocRows(int rows)
 {
     m_RowInfo = (wxHtmlListItemStruct*) realloc(m_RowInfo, sizeof(wxHtmlListItemStruct) * rows);

-    m_RowInfo[rows - 1].mark = 0;
-    m_RowInfo[rows - 1].cont = 0;
+    m_RowInfo[rows - 1].mark = NULL;
+    m_RowInfo[rows - 1].cont = NULL;

     m_RowInfo[rows - 1].minWidth = 0;
     m_RowInfo[rows - 1].maxWidth = 0;
 
     m_RowInfo[rows - 1].minWidth = 0;
     m_RowInfo[rows - 1].maxWidth = 0;
 


@@ -153,7 +176,7 @@ void wxHtmlListCell::ReallocRows(int rows)
 void wxHtmlListCell::ComputeMinMaxWidths()
 {
     if (m_NumRows == 0) return;
 void wxHtmlListCell::ComputeMinMaxWidths()
 {
     if (m_NumRows == 0) return;

-    
+

     m_MaxTotalWidth = 0;
     m_Width = 0;
 
     m_MaxTotalWidth = 0;
     m_Width = 0;
 


@@ -183,7 +206,7 @@ class wxHtmlListcontentCell : public wxHtmlContainerCell
 {
 public:
     wxHtmlListcontentCell(wxHtmlContainerCell *p) : wxHtmlContainerCell(p) {}
 {
 public:
     wxHtmlListcontentCell(wxHtmlContainerCell *p) : wxHtmlContainerCell(p) {}

-    virtual void Layout(int w) { 
+    virtual void Layout(int w) {

         // Reset top indentation, fixes <li><p>
         SetIndent(0, wxHTML_INDENT_TOP);
         wxHtmlContainerCell::Layout(w);
         // Reset top indentation, fixes <li><p>
         SetIndent(0, wxHTML_INDENT_TOP);
         wxHtmlContainerCell::Layout(w);


@@ -204,6 +227,7 @@ TAG_HANDLER_BEGIN(OLULLI, "OL,UL,LI")
 
     TAG_HANDLER_CONSTR(OLULLI)
     {
 
     TAG_HANDLER_CONSTR(OLULLI)
     {

+        m_List = NULL;

         m_Numbering = 0;
     }
 
         m_Numbering = 0;
     }
 


@@ -228,9 +252,9 @@ TAG_HANDLER_BEGIN(OLULLI, "OL,UL,LI")
             else
             {
                 c->SetAlignHor(wxHTML_ALIGN_RIGHT);
             else
             {
                 c->SetAlignHor(wxHTML_ALIGN_RIGHT);

-                wxString mark;
-                mark.Printf(wxT("%i."), m_Numbering);
-                c->InsertCell(new wxHtmlWordCell(mark, *(m_WParser->GetDC())));
+                wxString markStr;
+                markStr.Printf(wxT("%i. "), m_Numbering);
+                c->InsertCell(new wxHtmlWordCell(markStr, *(m_WParser->GetDC())));

             }
             m_WParser->CloseContainer();
 
             }
             m_WParser->CloseContainer();
 


@@ -239,7 +263,7 @@ TAG_HANDLER_BEGIN(OLULLI, "OL,UL,LI")
             m_List->AddRow(mark, c);
             c = m_WParser->OpenContainer();
             m_WParser->SetContainer(new wxHtmlListcontentCell(c));
             m_List->AddRow(mark, c);
             c = m_WParser->OpenContainer();
             m_WParser->SetContainer(new wxHtmlListcontentCell(c));

-            
+

             if (m_Numbering != 0) m_Numbering++;
         }
 
             if (m_Numbering != 0) m_Numbering++;
         }
 


@@ -257,7 +281,7 @@ TAG_HANDLER_BEGIN(OLULLI, "OL,UL,LI")
             wxHtmlListCell *oldList = m_List;
             m_List = new wxHtmlListCell(c);
             m_List->SetIndent(2 * m_WParser->GetCharWidth(), wxHTML_INDENT_LEFT);
             wxHtmlListCell *oldList = m_List;
             m_List = new wxHtmlListCell(c);
             m_List->SetIndent(2 * m_WParser->GetCharWidth(), wxHTML_INDENT_LEFT);

-            
+

             ParseInner(tag);
 
             m_WParser->SetContainer(oldcont);
             ParseInner(tag);
 
             m_WParser->SetContainer(oldcont);


@@ -265,7 +289,7 @@ TAG_HANDLER_BEGIN(OLULLI, "OL,UL,LI")
 
             m_Numbering = oldnum;
             m_List = oldList;
 
             m_Numbering = oldnum;
             m_List = oldList;

-            return TRUE;
+            return true;

         }
         return false;
 
         }
         return false;