X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/37a1f3f69b13506f10c99e1b6e7ee38f263257ea..adc27ddcb536d711f2940f0d0db258ae5e61eb6d:/src/html/m_image.cpp diff --git a/src/html/m_image.cpp b/src/html/m_image.cpp index 0f4027fb66..43904eac78 100644 --- a/src/html/m_image.cpp +++ b/src/html/m_image.cpp @@ -1,37 +1,38 @@ ///////////////////////////////////////////////////////////////////////////// -// Name: m_image.cpp +// Name: src/html/m_image.cpp // Purpose: wxHtml module for displaying images // Author: Vaclav Slavik // RCS-ID: $Id$ // Copyright: (c) 1999 Vaclav Slavik, Joel Lucsy -// Licence: wxWindows Licence +// Licence: wxWindows licence ///////////////////////////////////////////////////////////////////////////// -#ifdef __GNUG__ -#pragma implementation -#endif - #include "wx/wxprec.h" -#include "wx/defs.h" -#if wxUSE_HTML - -#ifdef __BORDLANDC__ -#pragma hdrstop +#ifdef __BORLANDC__ + #pragma hdrstop #endif -#ifndef WXPRECOMP -#include "wx/wx.h" +#if wxUSE_HTML && wxUSE_STREAMS + +#ifndef WX_PRECOMP + #include "wx/dynarray.h" + #include "wx/dc.h" + #include "wx/scrolwin.h" + #include "wx/timer.h" + #include "wx/dcmemory.h" + #include "wx/log.h" + #include "wx/math.h" + #include "wx/image.h" #endif #include "wx/html/forcelnk.h" #include "wx/html/m_templ.h" +#include "wx/html/htmlwin.h" +#include "wx/gifdecod.h" +#include "wx/artprov.h" -#include "wx/image.h" -#include "wx/dynarray.h" - -#include #include FORCE_LINK_ME(m_image) @@ -41,14 +42,14 @@ FORCE_LINK_ME(m_image) WX_DECLARE_OBJARRAY(int, CoordArray); #include "wx/arrimpl.cpp" // this is a magic incantation which must be done! -WX_DEFINE_OBJARRAY(CoordArray); +WX_DEFINE_OBJARRAY(CoordArray) -//-------------------------------------------------------------------------------- +// --------------------------------------------------------------------------- // wxHtmlImageMapAreaCell -// 0-width, 0-height cell that represents single area in imagemap -// (it's GetLink is called from wxHtmlImageCell's) -//-------------------------------------------------------------------------------- +// 0-width, 0-height cell that represents single area in +// imagemap (it's GetLink is called from wxHtmlImageCell's) +// --------------------------------------------------------------------------- class wxHtmlImageMapAreaCell : public wxHtmlCell { @@ -61,6 +62,13 @@ class wxHtmlImageMapAreaCell : public wxHtmlCell public: wxHtmlImageMapAreaCell( celltype t, wxString &coords, double pixel_scale = 1.0); virtual wxHtmlLinkInfo *GetLink( int x = 0, int y = 0 ) const; + void Draw(wxDC& WXUNUSED(dc), + int WXUNUSED(x), int WXUNUSED(y), + int WXUNUSED(view_y1), int WXUNUSED(view_y2), + wxHtmlRenderingInfo& WXUNUSED(info)) {} + + + wxDECLARE_NO_COPY_CLASS(wxHtmlImageMapAreaCell); }; @@ -73,7 +81,8 @@ wxHtmlImageMapAreaCell::wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::celltype wxString x = incoords, y; type = t; - while ((i = x.Find( ',' )) != -1) { + while ((i = x.Find( ',' )) != wxNOT_FOUND) + { coords.Add( (int)(pixel_scale * (double)wxAtoi( x.Left( i ).c_str())) ); x = x.Mid( i + 1 ); } @@ -82,7 +91,8 @@ wxHtmlImageMapAreaCell::wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::celltype wxHtmlLinkInfo *wxHtmlImageMapAreaCell::GetLink( int x, int y ) const { - switch (type) { + switch (type) + { case RECT: { int l, t, r, b; @@ -91,7 +101,8 @@ wxHtmlLinkInfo *wxHtmlImageMapAreaCell::GetLink( int x, int y ) const t = coords[ 1 ]; r = coords[ 2 ]; b = coords[ 3 ]; - if (x >= l && x <= r && y >= t && y <= b) { + if (x >= l && x <= r && y >= t && y <= b) + { return m_Link; } break; @@ -105,14 +116,16 @@ wxHtmlLinkInfo *wxHtmlImageMapAreaCell::GetLink( int x, int y ) const t = coords[ 1 ]; r = coords[ 2 ]; d = sqrt( (double) (((x - l) * (x - l)) + ((y - t) * (y - t))) ); - if (d < (double)r) { + if (d < (double)r) + { return m_Link; } } break; case POLY: { - if (coords.GetCount() >= 6) { + if (coords.GetCount() >= 6) + { int intersects = 0; int wherex = x; int wherey = y; @@ -123,46 +136,63 @@ wxHtmlLinkInfo *wxHtmlImageMapAreaCell::GetLink( int x, int y ) const int end = totalc; int pointer = 1; - if ((yval >= wherey) != (coords[pointer] >= wherey)) { - if ((xval >= wherex) == (coords[0] >= wherex)) { + if ((yval >= wherey) != (coords[pointer] >= wherey)) + { + if ((xval >= wherex) == (coords[0] >= wherex)) + { intersects += (xval >= wherex) ? 1 : 0; - } else { + } + else + { intersects += ((xval - (yval - wherey) * (coords[0] - xval) / (coords[pointer] - yval)) >= wherex) ? 1 : 0; } } - while (pointer < end) { + while (pointer < end) + { yval = coords[pointer]; pointer += 2; - if (yval >= wherey) { - while ((pointer < end) && (coords[pointer] >= wherey)) { + if (yval >= wherey) + { + while ((pointer < end) && (coords[pointer] >= wherey)) + { pointer += 2; } - if (pointer >= end) { + if (pointer >= end) + { break; } if ((coords[pointer - 3] >= wherex) == (coords[pointer - 1] >= wherex)) { intersects += (coords[pointer - 3] >= wherex) ? 1 : 0; - } else { + } + else + { intersects += ((coords[pointer - 3] - (coords[pointer - 2] - wherey) * (coords[pointer - 1] - coords[pointer - 3]) / (coords[pointer] - coords[pointer - 2])) >= wherex) ? 1 : 0; } - } else { - while ((pointer < end) && (coords[pointer] < wherey)) { + } + else + { + while ((pointer < end) && (coords[pointer] < wherey)) + { pointer += 2; } - if (pointer >= end) { + if (pointer >= end) + { break; } if ((coords[pointer - 3] >= wherex) == - (coords[pointer - 1] >= wherex)) { + (coords[pointer - 1] >= wherex)) + { intersects += (coords[pointer - 3] >= wherex) ? 1 : 0; - } else { + } + else + { intersects += ((coords[pointer - 3] - (coords[pointer - 2] - wherey) * (coords[pointer - 1] - coords[pointer - 3]) / @@ -170,14 +200,17 @@ wxHtmlLinkInfo *wxHtmlImageMapAreaCell::GetLink( int x, int y ) const } } } - if ((intersects & 1) != 0) { + if ((intersects & 1) != 0) + { return m_Link; } } } break; } - if (m_Next) { + + if (m_Next) + { wxHtmlImageMapAreaCell *a = (wxHtmlImageMapAreaCell*)m_Next; return a->GetLink( x, y ); } @@ -208,6 +241,12 @@ class wxHtmlImageMapCell : public wxHtmlCell public: virtual wxHtmlLinkInfo *GetLink( int x = 0, int y = 0 ) const; virtual const wxHtmlCell *Find( int cond, const void *param ) const; + void Draw(wxDC& WXUNUSED(dc), + int WXUNUSED(x), int WXUNUSED(y), + int WXUNUSED(view_y1), int WXUNUSED(view_y2), + wxHtmlRenderingInfo& WXUNUSED(info)) {} + + wxDECLARE_NO_COPY_CLASS(wxHtmlImageMapCell); }; @@ -226,7 +265,8 @@ wxHtmlLinkInfo *wxHtmlImageMapCell::GetLink( int x, int y ) const const wxHtmlCell *wxHtmlImageMapCell::Find( int cond, const void *param ) const { - if (cond == wxHTML_COND_ISIMAGEMAP) { + if (cond == wxHTML_COND_ISIMAGEMAP) + { if (m_Name == *((wxString*)(param))) return this; } @@ -244,50 +284,159 @@ const wxHtmlCell *wxHtmlImageMapCell::Find( int cond, const void *param ) const class wxHtmlImageCell : public wxHtmlCell { - public: - wxBitmap *m_Image; - wxHtmlImageMapCell *m_ImageMap; - wxString m_MapName; +public: + wxHtmlImageCell(wxHtmlWindowInterface *windowIface, + wxFSFile *input, int w = wxDefaultCoord, int h = wxDefaultCoord, + double scale = 1.0, int align = wxHTML_ALIGN_BOTTOM, + const wxString& mapname = wxEmptyString); + virtual ~wxHtmlImageCell(); + void Draw(wxDC& dc, int x, int y, int view_y1, int view_y2, + wxHtmlRenderingInfo& info); + virtual wxHtmlLinkInfo *GetLink(int x = 0, int y = 0) const; + + void SetImage(const wxImage& img); +#if wxUSE_GIF && wxUSE_TIMER + void AdvanceAnimation(wxTimer *timer); + virtual void Layout(int w); +#endif - wxHtmlImageCell(wxFSFile *input, int w = -1, int h = -1, double scale = 1.0, int align = wxHTML_ALIGN_BOTTOM, wxString mapname = wxEmptyString); - ~wxHtmlImageCell() {if (m_Image) delete m_Image; } - void Draw(wxDC& dc, int x, int y, int view_y1, int view_y2); - virtual wxHtmlLinkInfo *GetLink( int x = 0, int y = 0 ) const; +private: + wxBitmap *m_bitmap; + int m_bmpW, m_bmpH; + bool m_showFrame:1; + wxHtmlWindowInterface *m_windowIface; +#if wxUSE_GIF && wxUSE_TIMER + wxGIFDecoder *m_gifDecoder; + wxTimer *m_gifTimer; + int m_physX, m_physY; + size_t m_nCurrFrame; +#endif + double m_scale; + wxHtmlImageMapCell *m_imageMap; + wxString m_mapName; + + wxDECLARE_NO_COPY_CLASS(wxHtmlImageCell); }; +#if wxUSE_GIF && wxUSE_TIMER +class wxGIFTimer : public wxTimer +{ + public: + wxGIFTimer(wxHtmlImageCell *cell) : m_cell(cell) {} + virtual void Notify() + { + m_cell->AdvanceAnimation(this); + } + private: + wxHtmlImageCell *m_cell; + wxDECLARE_NO_COPY_CLASS(wxGIFTimer); +}; +#endif -//-------------------------------------------------------------------------------- + +//---------------------------------------------------------------------------- // wxHtmlImageCell -//-------------------------------------------------------------------------------- +//---------------------------------------------------------------------------- + -wxHtmlImageCell::wxHtmlImageCell(wxFSFile *input, int w, int h, double scale, int align, wxString mapname) : wxHtmlCell() +wxHtmlImageCell::wxHtmlImageCell(wxHtmlWindowInterface *windowIface, + wxFSFile *input, + int w, int h, double scale, int align, + const wxString& mapname) : wxHtmlCell() { - wxImage *img; - int ww, hh; - wxInputStream *s = input -> GetStream(); - - img = new wxImage(*s, wxBITMAP_TYPE_ANY); - - m_Image = NULL; - if (img && (img -> Ok())) { - ww = img -> GetWidth(); - hh = img -> GetHeight(); - if (w != -1) m_Width = w; else m_Width = ww; - if (h != -1) m_Height = h; else m_Height = hh; - - m_Width = (int)(scale * (double)w); - m_Height = (int)(scale * (double)h); - - if ((m_Width != ww) || (m_Height != hh)) { - wxImage img2 = img -> Scale(m_Width, m_Height); - m_Image = new wxBitmap(img2.ConvertToBitmap()); - } else - m_Image = new wxBitmap(img -> ConvertToBitmap()); - delete img; + m_windowIface = windowIface; + m_scale = scale; + m_showFrame = false; + m_bitmap = NULL; + m_bmpW = w; + m_bmpH = h; + m_imageMap = NULL; + m_mapName = mapname; + SetCanLiveOnPagebreak(false); +#if wxUSE_GIF && wxUSE_TIMER + m_gifDecoder = NULL; + m_gifTimer = NULL; + m_physX = m_physY = wxDefaultCoord; + m_nCurrFrame = 0; +#endif + + if ( m_bmpW && m_bmpH ) + { + if ( input ) + { + wxInputStream *s = input->GetStream(); + + if ( s ) + { +#if wxUSE_GIF && wxUSE_TIMER + bool readImg = true; + if ( m_windowIface && + (input->GetLocation().Matches(wxT("*.gif")) || + input->GetLocation().Matches(wxT("*.GIF"))) ) + { + m_gifDecoder = new wxGIFDecoder(); + if ( m_gifDecoder->LoadGIF(*s) == wxGIF_OK ) + { + wxImage img; + if ( m_gifDecoder->ConvertToImage(0, &img) ) + SetImage(img); + + readImg = false; + + if ( m_gifDecoder->IsAnimation() ) + { + m_gifTimer = new wxGIFTimer(this); + long delay = m_gifDecoder->GetDelay(0); + if ( delay == 0 ) + delay = 1; + m_gifTimer->Start(delay, true); + } + else + { + wxDELETE(m_gifDecoder); + } + } + else + { + wxDELETE(m_gifDecoder); + } + } + + if ( readImg ) +#endif // wxUSE_GIF && wxUSE_TIMER + { + wxImage image(*s, wxBITMAP_TYPE_ANY); + if ( image.Ok() ) + SetImage(image); + } + } + } + else // input==NULL, use "broken image" bitmap + { + if ( m_bmpW == wxDefaultCoord && m_bmpH == wxDefaultCoord ) + { + m_bmpW = 29; + m_bmpH = 31; + } + else + { + m_showFrame = true; + if ( m_bmpW == wxDefaultCoord ) m_bmpW = 31; + if ( m_bmpH == wxDefaultCoord ) m_bmpH = 33; + } + m_bitmap = + new wxBitmap(wxArtProvider::GetBitmap(wxART_MISSING_IMAGE)); + } } - switch (align) { + //else: ignore the 0-sized images used sometimes on the Web pages + + m_Width = (int)(scale * (double)m_bmpW); + m_Height = (int)(scale * (double)m_bmpH); + + switch (align) + { case wxHTML_ALIGN_TOP : m_Descent = m_Height; break; @@ -299,46 +448,167 @@ wxHtmlImageCell::wxHtmlImageCell(wxFSFile *input, int w, int h, double scale, in m_Descent = 0; break; } + } - m_ImageMap = NULL; - m_MapName = mapname; - SetCanLiveOnPagebreak(FALSE); +void wxHtmlImageCell::SetImage(const wxImage& img) +{ +#if !defined(__WXMSW__) || wxUSE_WXDIB + if ( img.Ok() ) + { + delete m_bitmap; + + int ww, hh; + ww = img.GetWidth(); + hh = img.GetHeight(); + + if ( m_bmpW == wxDefaultCoord ) + m_bmpW = ww; + if ( m_bmpH == wxDefaultCoord ) + m_bmpH = hh; + + // Only scale the bitmap at the rendering stage, + // so we don't lose quality twice +/* + if ((m_bmpW != ww) || (m_bmpH != hh)) + { + wxImage img2 = img.Scale(m_bmpW, m_bmpH); + m_bitmap = new wxBitmap(img2); + } + else +*/ + m_bitmap = new wxBitmap(img); + } +#endif } +#if wxUSE_GIF && wxUSE_TIMER +void wxHtmlImageCell::AdvanceAnimation(wxTimer *timer) +{ + wxImage img; + + // advance current frame + m_nCurrFrame++; + if (m_nCurrFrame == m_gifDecoder->GetFrameCount()) + m_nCurrFrame = 0; + + if ( m_physX == wxDefaultCoord ) + { + m_physX = m_physY = 0; + for (wxHtmlCell *cell = this; cell; cell = cell->GetParent()) + { + m_physX += cell->GetPosX(); + m_physY += cell->GetPosY(); + } + } + wxWindow *win = m_windowIface->GetHTMLWindow(); + wxPoint pos = + m_windowIface->HTMLCoordsToWindow(this, wxPoint(m_physX, m_physY)); + wxRect rect(pos, wxSize(m_Width, m_Height)); -void wxHtmlImageCell::Draw(wxDC& dc, int x, int y, int view_y1, int view_y2) + if ( win->GetClientRect().Intersects(rect) && + m_gifDecoder->ConvertToImage(m_nCurrFrame, &img) ) + { +#if !defined(__WXMSW__) || wxUSE_WXDIB + if ( m_gifDecoder->GetFrameSize(m_nCurrFrame) != wxSize(m_Width, m_Height) || + m_gifDecoder->GetFramePosition(m_nCurrFrame) != wxPoint(0, 0) ) + { + wxBitmap bmp(img); + wxMemoryDC dc; + dc.SelectObject(*m_bitmap); + dc.DrawBitmap(bmp, m_gifDecoder->GetFramePosition(m_nCurrFrame), + true /* use mask */); + } + else +#endif + SetImage(img); + win->Refresh(img.HasMask(), &rect); + } + + long delay = m_gifDecoder->GetDelay(m_nCurrFrame); + if ( delay == 0 ) + delay = 1; + timer->Start(delay, true); +} + +void wxHtmlImageCell::Layout(int w) +{ + wxHtmlCell::Layout(w); + m_physX = m_physY = wxDefaultCoord; +} + +#endif + +wxHtmlImageCell::~wxHtmlImageCell() { - if (m_Image) -// dc.DrawBitmap(*m_Image, x + m_PosX, y + m_PosY, (m_Image->GetMask() != (wxMask*) 0)); - dc.DrawBitmap(*m_Image, x + m_PosX, y + m_PosY, TRUE); - wxHtmlCell::Draw(dc, x, y, view_y1, view_y2); + delete m_bitmap; +#if wxUSE_GIF && wxUSE_TIMER + delete m_gifTimer; + delete m_gifDecoder; +#endif +} + + +void wxHtmlImageCell::Draw(wxDC& dc, int x, int y, + int WXUNUSED(view_y1), int WXUNUSED(view_y2), + wxHtmlRenderingInfo& WXUNUSED(info)) +{ + if ( m_showFrame ) + { + dc.SetBrush(*wxTRANSPARENT_BRUSH); + dc.SetPen(*wxBLACK_PEN); + dc.DrawRectangle(x + m_PosX, y + m_PosY, m_Width, m_Height); + x++, y++; + } + if ( m_bitmap ) + { + // We add in the scaling from the desired bitmap width + // and height, so we only do the scaling once. + double imageScaleX = 1.0; + double imageScaleY = 1.0; + if (m_bmpW != m_bitmap->GetWidth()) + imageScaleX = (double) m_bmpW / (double) m_bitmap->GetWidth(); + if (m_bmpH != m_bitmap->GetHeight()) + imageScaleY = (double) m_bmpH / (double) m_bitmap->GetHeight(); + + double us_x, us_y; + dc.GetUserScale(&us_x, &us_y); + dc.SetUserScale(us_x * m_scale * imageScaleX, us_y * m_scale * imageScaleY); + + dc.DrawBitmap(*m_bitmap, (int) ((x + m_PosX) / (m_scale*imageScaleX)), + (int) ((y + m_PosY) / (m_scale*imageScaleY)), true); + dc.SetUserScale(us_x, us_y); + } } wxHtmlLinkInfo *wxHtmlImageCell::GetLink( int x, int y ) const { - if (m_MapName.IsEmpty()) + if (m_mapName.empty()) return wxHtmlCell::GetLink( x, y ); - if (!m_ImageMap) { + if (!m_imageMap) + { wxHtmlContainerCell *p, *op; op = p = GetParent(); - while (p) { + while (p) + { op = p; p = p->GetParent(); } p = op; - wxHtmlCell *cell = (wxHtmlCell*)p->Find( wxHTML_COND_ISIMAGEMAP, (const void*)(&m_MapName)); - if (!cell) { - ((wxString&)m_MapName).Clear(); + wxHtmlCell *cell = (wxHtmlCell*)p->Find(wxHTML_COND_ISIMAGEMAP, + (const void*)(&m_mapName)); + if (!cell) + { + ((wxString&)m_mapName).Clear(); return wxHtmlCell::GetLink( x, y ); } - { // dirty hack, ask Joel why he fills m_ImageMap in this place + { // dirty hack, ask Joel why he fills m_ImageMap in this place // THE problem is that we're in const method and we can't modify m_ImageMap - wxHtmlImageMapCell **cx = (wxHtmlImageMapCell**)(&m_ImageMap); + wxHtmlImageMapCell **cx = (wxHtmlImageMapCell**)(&m_imageMap); *cx = (wxHtmlImageMapCell*)cell; } } - return m_ImageMap->GetLink( x, y ); + return m_imageMap->GetLink(x, y); } @@ -348,46 +618,62 @@ wxHtmlLinkInfo *wxHtmlImageCell::GetLink( int x, int y ) const //-------------------------------------------------------------------------------- TAG_HANDLER_BEGIN(IMG, "IMG,MAP,AREA") + TAG_HANDLER_CONSTR(IMG) { } TAG_HANDLER_PROC(tag) { - if (tag.GetName() == wxT("IMG")) { - if (tag.HasParam(wxT("SRC"))) { - int w = -1, h = -1; + if (tag.GetName() == wxT("IMG")) + { + if (tag.HasParam(wxT("SRC"))) + { + int w = wxDefaultCoord, h = wxDefaultCoord; int al; wxFSFile *str; wxString tmp = tag.GetParam(wxT("SRC")); wxString mn = wxEmptyString; - str = m_WParser -> GetFS() -> OpenFile(tmp); - if (tag.HasParam(wxT("WIDTH"))) tag.ScanParam(wxT("WIDTH"), wxT("%i"), &w); - if (tag.HasParam(wxT("HEIGHT"))) tag.ScanParam(wxT("HEIGHT"), wxT("%i"), &h); + str = m_WParser->OpenURL(wxHTML_URL_IMAGE, tmp); + + if (tag.HasParam(wxT("WIDTH"))) + tag.GetParamAsInt(wxT("WIDTH"), &w); + if (tag.HasParam(wxT("HEIGHT"))) + tag.GetParamAsInt(wxT("HEIGHT"), &h); al = wxHTML_ALIGN_BOTTOM; - if (tag.HasParam(wxT("ALIGN"))) { + if (tag.HasParam(wxT("ALIGN"))) + { wxString alstr = tag.GetParam(wxT("ALIGN")); alstr.MakeUpper(); // for the case alignment was in ".." - if (alstr == wxT("TEXTTOP")) al = wxHTML_ALIGN_TOP; - else if ((alstr == wxT("CENTER")) || (alstr == wxT("ABSCENTER"))) al = wxHTML_ALIGN_CENTER; + if (alstr == wxT("TEXTTOP")) + al = wxHTML_ALIGN_TOP; + else if ((alstr == wxT("CENTER")) || (alstr == wxT("ABSCENTER"))) + al = wxHTML_ALIGN_CENTER; } - if (tag.HasParam(wxT("USEMAP"))) { + if (tag.HasParam(wxT("USEMAP"))) + { mn = tag.GetParam( wxT("USEMAP") ); - if (mn[ (unsigned int) 0 ] == wxT('#')) { + if (mn.GetChar(0) == wxT('#')) + { mn = mn.Mid( 1 ); } } - wxHtmlImageCell *cel = NULL; - if (str) { - cel = new wxHtmlImageCell(str, w, h, m_WParser -> GetPixelScale(), al, mn); - cel -> SetLink(m_WParser -> GetLink()); - m_WParser -> GetContainer() -> InsertCell(cel); + wxHtmlImageCell *cel = new wxHtmlImageCell( + m_WParser->GetWindowInterface(), + str, w, h, + m_WParser->GetPixelScale(), + al, mn); + m_WParser->ApplyStateToCell(cel); + cel->SetId(tag.GetParam(wxT("id"))); // may be empty + m_WParser->GetContainer()->InsertCell(cel); + if (str) delete str; - } } } - if (tag.GetName() == wxT("MAP")) { + if (tag.GetName() == wxT("MAP")) + { m_WParser->CloseContainer(); m_WParser->OpenContainer(); - if (tag.HasParam(wxT("NAME"))) { + if (tag.HasParam(wxT("NAME"))) + { wxString tmp = tag.GetParam(wxT("NAME")); wxHtmlImageMapCell *cel = new wxHtmlImageMapCell( tmp ); m_WParser->GetContainer()->InsertCell( cel ); @@ -396,33 +682,43 @@ TAG_HANDLER_BEGIN(IMG, "IMG,MAP,AREA") m_WParser->CloseContainer(); m_WParser->OpenContainer(); } - if (tag.GetName() == wxT("AREA")) { - if (tag.HasParam(wxT("SHAPE"))) { + if (tag.GetName() == wxT("AREA")) + { + if (tag.HasParam(wxT("SHAPE"))) + { wxString tmp = tag.GetParam(wxT("SHAPE")); wxString coords = wxEmptyString; tmp.MakeUpper(); wxHtmlImageMapAreaCell *cel = NULL; - if (tag.HasParam(wxT("COORDS"))) { + if (tag.HasParam(wxT("COORDS"))) + { coords = tag.GetParam(wxT("COORDS")); } - if (tmp == wxT("POLY")) { - cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::POLY, coords, m_WParser -> GetPixelScale() ); - } else if (tmp == wxT("CIRCLE")) { - cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::CIRCLE, coords, m_WParser -> GetPixelScale() ); - } else if (tmp == wxT("RECT")) { - cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::RECT, coords, m_WParser -> GetPixelScale() ); + if (tmp == wxT("POLY")) + { + cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::POLY, coords, m_WParser->GetPixelScale() ); + } + else if (tmp == wxT("CIRCLE")) + { + cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::CIRCLE, coords, m_WParser->GetPixelScale() ); + } + else if (tmp == wxT("RECT")) + { + cel = new wxHtmlImageMapAreaCell( wxHtmlImageMapAreaCell::RECT, coords, m_WParser->GetPixelScale() ); } - if (cel != NULL && tag.HasParam(wxT("HREF"))) { - wxString tmp = tag.GetParam(wxT("HREF")); - wxString target = wxEmptyString; - if (tag.HasParam(wxT("TARGET"))) target = tag.GetParam(wxT("TARGET")); - cel->SetLink( wxHtmlLinkInfo(tmp, target)); + if (cel != NULL && tag.HasParam(wxT("HREF"))) + { + wxString target; + if (tag.HasParam(wxT("TARGET"))) + target = tag.GetParam(wxT("TARGET")); + cel->SetLink(wxHtmlLinkInfo(tag.GetParam(wxT("HREF")), target)); } - if (cel != NULL) m_WParser->GetContainer()->InsertCell( cel ); + if (cel != NULL) + m_WParser->GetContainer()->InsertCell( cel ); } } - return FALSE; + return false; } TAG_HANDLER_END(IMG)