X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/8fd2b35cbe5d66bf0c32b91e5313918a518ee5fe..73ef393701bd6c2c689a05d5637cc701e2f5cc2f:/src/html/helpfrm.cpp
diff --git a/src/html/helpfrm.cpp b/src/html/helpfrm.cpp
index cf03e1065c..a71bab2521 100644
--- a/src/html/helpfrm.cpp
+++ b/src/html/helpfrm.cpp
@@ -21,15 +21,24 @@
#pragma hdrstop
#endif
-#include "wx/defs.h"
+#if wxUSE_WXHTML_HELP
-#if wxUSE_HTML && wxUSE_STREAMS
#ifndef WXPRECOMP
-#include "wx/wx.h"
-#endif
+ #include "wx/intl.h"
+ #include "wx/log.h"
+
+ #include "wx/object.h"
+ #include "wx/layout.h"
+ #include "wx/sizer.h"
+
+ #include "wx/bmpbuttn.h"
+ #include "wx/statbox.h"
+ #include "wx/radiobox.h"
+#endif // WXPRECOMP
#include "wx/html/helpfrm.h"
#include "wx/html/helpctrl.h"
+#include "wx/textctrl.h"
#include "wx/notebook.h"
#include "wx/imaglist.h"
#include "wx/treectrl.h"
@@ -46,28 +55,36 @@
// Bitmaps:
#ifndef __WXMSW__
-#include "bitmaps/wpanel.xpm"
-#include "bitmaps/wback.xpm"
-#include "bitmaps/wforward.xpm"
-#include "bitmaps/wbook.xpm"
-#include "bitmaps/woptions.xpm"
-#include "bitmaps/wfolder.xpm"
-#include "bitmaps/wpage.xpm"
-#include "bitmaps/whelp.xpm"
-#include "bitmaps/whlproot.xpm"
-#include "bitmaps/wbkadd.xpm"
-#include "bitmaps/wbkdel.xpm"
-#include "bitmaps/wup.xpm"
-#include "bitmaps/wupnode.xpm"
-#include "bitmaps/wdown.xpm"
-#include "bitmaps/wopen.xpm"
-#include "bitmaps/wprint.xpm"
-#endif
-
+ // XPM hack: make the arrays const
+ #define static static const
+
+ #include "bitmaps/wpanel.xpm"
+ #include "bitmaps/wback.xpm"
+ #include "bitmaps/wforward.xpm"
+ #include "bitmaps/wbook.xpm"
+ #include "bitmaps/woptions.xpm"
+ #include "bitmaps/wfolder.xpm"
+ #include "bitmaps/wpage.xpm"
+ #include "bitmaps/whelp.xpm"
+ #include "bitmaps/whlproot.xpm"
+ #include "bitmaps/wbkadd.xpm"
+ #include "bitmaps/wbkdel.xpm"
+ #include "bitmaps/wup.xpm"
+ #include "bitmaps/wupnode.xpm"
+ #include "bitmaps/wdown.xpm"
+ #include "bitmaps/wopen.xpm"
+ #include "bitmaps/wprint.xpm"
+
+ #undef static
+#endif // __WXMSW__
// what is considered "small index"?
#define INDEX_IS_SMALL 100
+/* Motif defines this as a macro */
+#ifdef Below
+#undef Below
+#endif
//--------------------------------------------------------------------------
// wxHtmlHelpTreeItemData (private)
@@ -76,6 +93,11 @@
class wxHtmlHelpTreeItemData : public wxTreeItemData
{
public:
+#if defined(__VISAGECPP__)
+// VA needs a default ctor for some reason....
+ wxHtmlHelpTreeItemData() : wxTreeItemData()
+ { m_Id = 0; }
+#endif
wxHtmlHelpTreeItemData(int id) : wxTreeItemData()
{ m_Id = id;}
@@ -111,7 +133,7 @@ class wxHtmlHelpHtmlWindow : public wxHtmlWindow
virtual void OnLinkClicked(const wxHtmlLinkInfo& link)
{
wxHtmlWindow::OnLinkClicked(link);
- m_Frame -> NotifyPageChanged();
+ m_Frame->NotifyPageChanged();
}
private:
@@ -124,6 +146,38 @@ class wxHtmlHelpHtmlWindow : public wxHtmlWindow
// wxHtmlHelpFrame
//---------------------------------------------------------------------------
+// Command IDs :
+enum
+{
+ //wxID_HTML_HELPFRAME = wxID_HIGHEST + 1,
+ wxID_HTML_PANEL = wxID_HIGHEST + 2,
+ wxID_HTML_BACK,
+ wxID_HTML_FORWARD,
+ wxID_HTML_UPNODE,
+ wxID_HTML_UP,
+ wxID_HTML_DOWN,
+ wxID_HTML_PRINT,
+ wxID_HTML_OPENFILE,
+ wxID_HTML_OPTIONS,
+ wxID_HTML_BOOKMARKSLIST,
+ wxID_HTML_BOOKMARKSADD,
+ wxID_HTML_BOOKMARKSREMOVE,
+ wxID_HTML_TREECTRL,
+ wxID_HTML_INDEXPAGE,
+ wxID_HTML_INDEXLIST,
+ wxID_HTML_INDEXTEXT,
+ wxID_HTML_INDEXBUTTON,
+ wxID_HTML_INDEXBUTTONALL,
+ wxID_HTML_NOTEBOOK,
+ wxID_HTML_SEARCHPAGE,
+ wxID_HTML_SEARCHTEXT,
+ wxID_HTML_SEARCHLIST,
+ wxID_HTML_SEARCHBUTTON,
+ wxID_HTML_SEARCHCHOICE,
+ wxID_HTML_COUNTINFO
+};
+
+
IMPLEMENT_DYNAMIC_CLASS(wxHtmlHelpFrame, wxFrame)
wxHtmlHelpFrame::wxHtmlHelpFrame(wxWindow* parent, wxWindowID id, const wxString& title,
@@ -135,10 +189,12 @@ wxHtmlHelpFrame::wxHtmlHelpFrame(wxWindow* parent, wxWindowID id, const wxString
void wxHtmlHelpFrame::Init(wxHtmlHelpData* data)
{
- if (data) {
+ if (data)
+ {
m_Data = data;
m_DataCreated = FALSE;
- } else {
+ } else
+ {
m_Data = new wxHtmlHelpData();
m_DataCreated = TRUE;
}
@@ -191,9 +247,11 @@ void wxHtmlHelpFrame::Init(wxHtmlHelpData* data)
// moreover, if no contents, index or searchpage is needed, m_Splitter and
// m_NavigPan will be NULL too (with m_HtmlWin directly connected to the frame)
-bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id, const wxString& title,
- int style)
+bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id,
+ const wxString& WXUNUSED(title), int style)
{
+ m_hfStyle = style;
+
wxImageList *ContentsImageList = new wxImageList(16, 16);
ContentsImageList->Add(wxICON(wbook));
ContentsImageList->Add(wxICON(wfolder));
@@ -216,16 +274,18 @@ bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id, const wxString& ti
CreateStatusBar();
// toolbar?
- if (style & (wxHF_TOOLBAR | wxHF_FLATTOOLBAR)) {
+ if (style & (wxHF_TOOLBAR | wxHF_FLAT_TOOLBAR))
+ {
wxToolBar *toolBar = CreateToolBar(wxNO_BORDER | wxTB_HORIZONTAL |
wxTB_DOCKABLE |
- (style & wxHF_FLATTOOLBAR ? wxTB_FLAT : 0));
+ (style & wxHF_FLAT_TOOLBAR ? wxTB_FLAT : 0));
toolBar->SetMargins( 2, 2 );
AddToolbarButtons(toolBar, style);
- toolBar -> Realize();
+ toolBar->Realize();
}
- if (style & (wxHF_CONTENTS | wxHF_INDEX | wxHF_SEARCH)) {
+ if (style & (wxHF_CONTENTS | wxHF_INDEX | wxHF_SEARCH))
+ {
// traditional help controller; splitter window with html page on the
// right and a notebook containing various pages on the left
m_Splitter = new wxSplitterWindow(this);
@@ -233,227 +293,243 @@ bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id, const wxString& ti
m_HtmlWin = new wxHtmlHelpHtmlWindow(this, m_Splitter);
m_NavigPan = new wxNotebook(m_Splitter, wxID_HTML_NOTEBOOK,
wxDefaultPosition, wxDefaultSize);
- } else { // only html window, no notebook with index,contents etc
+ }
+ else
+ { // only html window, no notebook with index,contents etc
m_HtmlWin = new wxHtmlWindow(this);
}
- m_HtmlWin -> SetRelatedFrame(this, m_TitleFormat);
- m_HtmlWin -> SetRelatedStatusBar(0);
+ m_HtmlWin->SetRelatedFrame(this, m_TitleFormat);
+ m_HtmlWin->SetRelatedStatusBar(0);
if (m_Config)
- m_HtmlWin -> ReadCustomization(m_Config, m_ConfigRoot);
+ m_HtmlWin->ReadCustomization(m_Config, m_ConfigRoot);
// contents tree panel?
- if (style & wxHF_CONTENTS) {
+ if (style & wxHF_CONTENTS)
+ {
wxWindow *dummy = new wxPanel(m_NavigPan, wxID_HTML_INDEXPAGE);
+
+ long treeStyle = wxSUNKEN_BORDER | wxTR_HAS_BUTTONS;
+ #ifndef __WXMSW__ // FIXME - temporary, till MSW supports wxTR_HIDE_ROOT
+ treeStyle |= wxTR_HIDE_ROOT;
+ #endif
- if (style & wxHF_BOOKMARKS) {
+ if (style & wxHF_BOOKMARKS)
+ {
wxLayoutConstraints *b1 = new wxLayoutConstraints;
wxBitmapButton *bmpbt = new wxBitmapButton(dummy, wxID_HTML_BOOKMARKSREMOVE, wxBITMAP(wbkdel), wxDefaultPosition, wxSize(20,20));
- b1 -> top.SameAs (dummy, wxTop, 10);
- b1 -> right.SameAs (dummy, wxRight, 10);
- b1 -> height.AsIs();
- b1 -> width.AsIs();
- bmpbt -> SetConstraints(b1);
+ b1->top.SameAs (dummy, wxTop, 10);
+ b1->right.SameAs (dummy, wxRight, 10);
+ b1->height.AsIs();
+ b1->width.AsIs();
+ bmpbt->SetConstraints(b1);
wxLayoutConstraints *b2 = new wxLayoutConstraints;
wxBitmapButton *bmpbt2 = new wxBitmapButton(dummy, wxID_HTML_BOOKMARKSADD, wxBITMAP(wbkadd), wxDefaultPosition, wxSize(20,20));
- b2 -> top.SameAs (dummy, wxTop, 10);
- b2 -> right.LeftOf (bmpbt, 2);
- b2 -> height.AsIs();
- b2 -> width.AsIs();
- bmpbt2 -> SetConstraints(b2);
+ b2->top.SameAs (dummy, wxTop, 10);
+ b2->right.LeftOf (bmpbt, 2);
+ b2->height.AsIs();
+ b2->width.AsIs();
+ bmpbt2->SetConstraints(b2);
#if wxUSE_TOOLTIPS
- bmpbt -> SetToolTip(_("Remove current page from bookmarks"));
- bmpbt2 -> SetToolTip(_("Add current page to bookmarks"));
+ bmpbt->SetToolTip(_("Remove current page from bookmarks"));
+ bmpbt2->SetToolTip(_("Add current page to bookmarks"));
#endif //wxUSE_TOOLTIPS
wxLayoutConstraints *b3 = new wxLayoutConstraints;
m_Bookmarks = new wxComboBox(dummy, wxID_HTML_BOOKMARKSLIST, wxEmptyString,
wxDefaultPosition, wxDefaultSize, 0, NULL, wxCB_READONLY | wxCB_SORT);
- m_Bookmarks -> Append(_("(bookmarks)"));
+ m_Bookmarks->Append(_("(bookmarks)"));
for (unsigned i = 0; i < m_BookmarksNames.GetCount(); i++)
- m_Bookmarks -> Append(m_BookmarksNames[i]);
- m_Bookmarks -> SetSelection(0);
+ m_Bookmarks->Append(m_BookmarksNames[i]);
+ m_Bookmarks->SetSelection(0);
- b3 -> centreY.SameAs (bmpbt2, wxCentreY);
- b3 -> left.SameAs (dummy, wxLeft, 10);
- b3 -> right.LeftOf (bmpbt2, 5);
- b3 -> height.AsIs();
- m_Bookmarks -> SetConstraints(b3);
+ b3->centreY.SameAs (bmpbt2, wxCentreY);
+ b3->left.SameAs (dummy, wxLeft, 10);
+ b3->right.LeftOf (bmpbt2, 5);
+ b3->height.AsIs();
+ m_Bookmarks->SetConstraints(b3);
wxLayoutConstraints *b4 = new wxLayoutConstraints;
m_ContentsBox = new wxTreeCtrl(dummy, wxID_HTML_TREECTRL,
wxDefaultPosition, wxDefaultSize,
- wxTR_HAS_BUTTONS | wxSUNKEN_BORDER);
- m_ContentsBox -> AssignImageList(ContentsImageList);
+ treeStyle);
+ m_ContentsBox->AssignImageList(ContentsImageList);
- b4 -> top.Below (m_Bookmarks, 10);
- b4 -> left.SameAs (dummy, wxLeft, 0);
- b4 -> right.SameAs (dummy, wxRight, 0);
- b4 -> bottom.SameAs (dummy, wxBottom, 0);
- m_ContentsBox -> SetConstraints(b4);
+ b4->top.Below (m_Bookmarks, 10);
+ b4->left.SameAs (dummy, wxLeft, 0);
+ b4->right.SameAs (dummy, wxRight, 0);
+ b4->bottom.SameAs (dummy, wxBottom, 0);
+ m_ContentsBox->SetConstraints(b4);
- dummy -> SetAutoLayout(TRUE);
- dummy -> Layout();
+ dummy->SetAutoLayout(TRUE);
+ dummy->Layout();
- m_NavigPan -> AddPage(dummy, _("Contents"));
+ m_NavigPan->AddPage(dummy, _("Contents"));
}
else
{
m_ContentsBox = new wxTreeCtrl(m_NavigPan, wxID_HTML_TREECTRL,
wxDefaultPosition, wxDefaultSize,
- wxTR_HAS_BUTTONS | wxSUNKEN_BORDER);
- m_ContentsBox -> AssignImageList(ContentsImageList);
- m_NavigPan -> AddPage(m_ContentsBox, _("Contents"));
+ treeStyle);
+ m_ContentsBox->AssignImageList(ContentsImageList);
+ m_NavigPan->AddPage(m_ContentsBox, _("Contents"));
}
m_ContentsPage = notebook_page++;
}
// index listbox panel?
- if (style & wxHF_INDEX) {
+ if (style & wxHF_INDEX)
+ {
wxWindow *dummy = new wxPanel(m_NavigPan, wxID_HTML_INDEXPAGE);
wxLayoutConstraints *b1 = new wxLayoutConstraints;
m_IndexText = new wxTextCtrl(dummy, wxID_HTML_INDEXTEXT, wxEmptyString, wxDefaultPosition, wxDefaultSize, wxTE_PROCESS_ENTER);
- b1 -> top.SameAs (dummy, wxTop, 10);
- b1 -> left.SameAs (dummy, wxLeft, 10);
- b1 -> right.SameAs (dummy, wxRight, 10);
- b1 -> height.AsIs();
- m_IndexText -> SetConstraints(b1);
+ b1->top.SameAs (dummy, wxTop, 10);
+ b1->left.SameAs (dummy, wxLeft, 10);
+ b1->right.SameAs (dummy, wxRight, 10);
+ b1->height.AsIs();
+ m_IndexText->SetConstraints(b1);
wxLayoutConstraints *b4 = new wxLayoutConstraints;
m_IndexButtonAll = new wxButton(dummy, wxID_HTML_INDEXBUTTONALL, _("Show all"));
- b4 -> top.Below (m_IndexText, 10);
- b4 -> right.SameAs (dummy, wxRight, 10);
- b4 -> width.AsIs();
- b4 -> height.AsIs();
- m_IndexButtonAll -> SetConstraints(b4);
+ b4->top.Below (m_IndexText, 10);
+ b4->right.SameAs (dummy, wxRight, 10);
+ b4->width.AsIs();
+ b4->height.AsIs();
+ m_IndexButtonAll->SetConstraints(b4);
wxLayoutConstraints *b2 = new wxLayoutConstraints;
m_IndexButton = new wxButton(dummy, wxID_HTML_INDEXBUTTON, _("Find"));
- b2 -> top.Below (m_IndexText, 10);
- b2 -> right.LeftOf (m_IndexButtonAll, 10);
- b2 -> width.AsIs();
- b2 -> height.AsIs();
- m_IndexButton -> SetConstraints(b2);
+ b2->top.Below (m_IndexText, 10);
+ b2->right.LeftOf (m_IndexButtonAll, 10);
+ b2->width.AsIs();
+ b2->height.AsIs();
+ m_IndexButton->SetConstraints(b2);
wxLayoutConstraints *b5 = new wxLayoutConstraints;
m_IndexCountInfo = new wxStaticText(dummy, wxID_HTML_COUNTINFO, wxEmptyString, wxDefaultPosition, wxDefaultSize, wxALIGN_RIGHT | wxST_NO_AUTORESIZE);
- b5 -> top.Below (m_IndexButton, 5);
- b5 -> right.SameAs (dummy, wxRight, 10);
- b5 -> left.SameAs (dummy, wxLeft, 10);
- b5 -> height.AsIs();
- m_IndexCountInfo -> SetConstraints(b5);
+ b5->top.Below (m_IndexButton, 5);
+ b5->right.SameAs (dummy, wxRight, 10);
+ b5->left.SameAs (dummy, wxLeft, 10);
+ b5->height.AsIs();
+ m_IndexCountInfo->SetConstraints(b5);
wxLayoutConstraints *b3 = new wxLayoutConstraints;
m_IndexList = new wxListBox(dummy, wxID_HTML_INDEXLIST, wxDefaultPosition, wxDefaultSize, 0, NULL, wxLB_SINGLE);
- b3 -> top.Below (m_IndexCountInfo, 5);
- b3 -> left.SameAs (dummy, wxLeft, 0);
- b3 -> right.SameAs (dummy, wxRight, 0);
- b3 -> bottom.SameAs (dummy, wxBottom, 0);
- m_IndexList -> SetConstraints(b3);
+ b3->top.Below (m_IndexCountInfo, 5);
+ b3->left.SameAs (dummy, wxLeft, 0);
+ b3->right.SameAs (dummy, wxRight, 0);
+ b3->bottom.SameAs (dummy, wxBottom, 0);
+ m_IndexList->SetConstraints(b3);
#if wxUSE_TOOLTIPS
- m_IndexButtonAll -> SetToolTip(_("Show all items in index"));
- m_IndexButton -> SetToolTip(_("Display all index items that contain given substring. Search is case insensitive."));
+ m_IndexButtonAll->SetToolTip(_("Show all items in index"));
+ m_IndexButton->SetToolTip(_("Display all index items that contain given substring. Search is case insensitive."));
#endif //wxUSE_TOOLTIPS
- dummy -> SetAutoLayout(TRUE);
- dummy -> Layout();
+ dummy->SetAutoLayout(TRUE);
+ dummy->Layout();
- m_NavigPan -> AddPage(dummy, _("Index"));
+ m_NavigPan->AddPage(dummy, _("Index"));
m_IndexPage = notebook_page++;
}
// search list panel?
- if (style & wxHF_SEARCH) {
+ if (style & wxHF_SEARCH)
+ {
wxWindow *dummy = new wxPanel(m_NavigPan, wxID_HTML_SEARCHPAGE);
wxLayoutConstraints *b1 = new wxLayoutConstraints;
m_SearchText = new wxTextCtrl(dummy, wxID_HTML_SEARCHTEXT, wxEmptyString, wxDefaultPosition, wxDefaultSize, wxTE_PROCESS_ENTER);
- b1 -> top.SameAs (dummy, wxTop, 10);
- b1 -> left.SameAs (dummy, wxLeft, 10);
- b1 -> right.SameAs (dummy, wxRight, 10);
- b1 -> height.AsIs();
- m_SearchText -> SetConstraints(b1);
+ b1->top.SameAs (dummy, wxTop, 10);
+ b1->left.SameAs (dummy, wxLeft, 10);
+ b1->right.SameAs (dummy, wxRight, 10);
+ b1->height.AsIs();
+ m_SearchText->SetConstraints(b1);
wxLayoutConstraints *b4 = new wxLayoutConstraints;
m_SearchChoice = new wxChoice(dummy, wxID_HTML_SEARCHCHOICE, wxDefaultPosition,
wxDefaultSize);
- b4 -> top.Below (m_SearchText, 10);
- b4 -> left.SameAs (dummy, wxLeft, 10);
- b4 -> right.SameAs (dummy, wxRight, 10);
- b4 -> height.AsIs();
- m_SearchChoice -> SetConstraints(b4);
+ b4->top.Below (m_SearchText, 10);
+ b4->left.SameAs (dummy, wxLeft, 10);
+ b4->right.SameAs (dummy, wxRight, 10);
+ b4->height.AsIs();
+ m_SearchChoice->SetConstraints(b4);
wxLayoutConstraints *b5 = new wxLayoutConstraints;
m_SearchCaseSensitive = new wxCheckBox(dummy, -1, _("Case sensitive"));
- b5 -> top.Below (m_SearchChoice, 10);
- b5 -> left.SameAs (dummy, wxLeft, 10);
- b5 -> width.AsIs();
- b5 -> height.AsIs ();
- m_SearchCaseSensitive -> SetConstraints(b5);
+ b5->top.Below (m_SearchChoice, 10);
+ b5->left.SameAs (dummy, wxLeft, 10);
+ b5->width.AsIs();
+ b5->height.AsIs ();
+ m_SearchCaseSensitive->SetConstraints(b5);
wxLayoutConstraints *b6 = new wxLayoutConstraints;
m_SearchWholeWords = new wxCheckBox(dummy, -1, _("Whole words only"));
- b6 -> top.Below (m_SearchCaseSensitive, 0);
- b6 -> left.SameAs (dummy, wxLeft, 10);
- b6 -> width.AsIs();
- b6 -> height.AsIs ();
- m_SearchWholeWords -> SetConstraints(b6);
+ b6->top.Below (m_SearchCaseSensitive, 0);
+ b6->left.SameAs (dummy, wxLeft, 10);
+ b6->width.AsIs();
+ b6->height.AsIs ();
+ m_SearchWholeWords->SetConstraints(b6);
wxLayoutConstraints *b2 = new wxLayoutConstraints;
m_SearchButton = new wxButton(dummy, wxID_HTML_SEARCHBUTTON, _("Search"));
#if wxUSE_TOOLTIPS
- m_SearchButton -> SetToolTip(_("Search contents of help book(s) for all occurences of the text you typed above"));
+ m_SearchButton->SetToolTip(_("Search contents of help book(s) for all occurences of the text you typed above"));
#endif //wxUSE_TOOLTIPS
- b2 -> top.Below (m_SearchWholeWords, 0);
- b2 -> right.SameAs (dummy, wxRight, 10);
- b2 -> width.AsIs();
- b2 -> height.AsIs();
- m_SearchButton -> SetConstraints(b2);
+ b2->top.Below (m_SearchWholeWords, 0);
+ b2->right.SameAs (dummy, wxRight, 10);
+ b2->width.AsIs();
+ b2->height.AsIs();
+ m_SearchButton->SetConstraints(b2);
wxLayoutConstraints *b3 = new wxLayoutConstraints;
m_SearchList = new wxListBox(dummy, wxID_HTML_SEARCHLIST, wxDefaultPosition, wxDefaultSize, 0, NULL, wxLB_SINGLE);
- b3 -> top.Below (m_SearchButton, 10);
- b3 -> left.SameAs (dummy, wxLeft, 0);
- b3 -> right.SameAs (dummy, wxRight, 0);
- b3 -> bottom.SameAs (dummy, wxBottom, 0);
- m_SearchList -> SetConstraints(b3);
-
- dummy -> SetAutoLayout(TRUE);
- dummy -> Layout();
- m_NavigPan -> AddPage(dummy, _("Search"));
+ b3->top.Below (m_SearchButton, 10);
+ b3->left.SameAs (dummy, wxLeft, 0);
+ b3->right.SameAs (dummy, wxRight, 0);
+ b3->bottom.SameAs (dummy, wxBottom, 0);
+ m_SearchList->SetConstraints(b3);
+
+ dummy->SetAutoLayout(TRUE);
+ dummy->Layout();
+ m_NavigPan->AddPage(dummy, _("Search"));
m_SearchPage = notebook_page++;
}
- m_HtmlWin -> Show(TRUE);
+ m_HtmlWin->Show(TRUE);
RefreshLists();
// showtime
- if (m_NavigPan && m_Splitter) {
- m_Splitter -> SetMinimumPaneSize(20);
+ if (m_NavigPan && m_Splitter)
+ {
+ m_Splitter->SetMinimumPaneSize(20);
if (m_Cfg.navig_on)
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
- else {
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
- m_Splitter -> Unsplit();
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ else
+ {
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ m_Splitter->Unsplit();
}
- if (m_Cfg.navig_on) {
- m_NavigPan -> Show(TRUE);
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+
+ if (m_Cfg.navig_on)
+ {
+ m_NavigPan->Show(TRUE);
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
}
- else {
- m_NavigPan -> Show(FALSE);
- m_Splitter -> Initialize(m_HtmlWin);
+ else
+ {
+ m_NavigPan->Show(FALSE);
+ m_Splitter->Initialize(m_HtmlWin);
}
}
return TRUE;
@@ -490,46 +566,46 @@ void wxHtmlHelpFrame::AddToolbarButtons(wxToolBar *toolBar, int style)
wxT("One or more HTML help frame toolbar bitmap could not be loaded.")) ;
- toolBar -> AddTool(wxID_HTML_PANEL, wpanelBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_PANEL, wpanelBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Show/hide navigation panel"));
- toolBar -> AddSeparator();
- toolBar -> AddTool(wxID_HTML_BACK, wbackBitmap, wxNullBitmap,
+ toolBar->AddSeparator();
+ toolBar->AddTool(wxID_HTML_BACK, wbackBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Go back"));
- toolBar -> AddTool(wxID_HTML_FORWARD, wforwardBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_FORWARD, wforwardBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Go forward"));
- toolBar -> AddSeparator();
+ toolBar->AddSeparator();
- toolBar -> AddTool(wxID_HTML_UPNODE, wupnodeBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_UPNODE, wupnodeBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Go one level up in document hierarchy"));
- toolBar -> AddTool(wxID_HTML_UP, wupBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_UP, wupBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Previous page"));
- toolBar -> AddTool(wxID_HTML_DOWN, wdownBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_DOWN, wdownBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Next page"));
- if ((style & wxHF_PRINT) || (style & wxHF_OPENFILES))
- toolBar -> AddSeparator();
+ if ((style & wxHF_PRINT) || (style & wxHF_OPEN_FILES))
+ toolBar->AddSeparator();
- if (style & wxHF_OPENFILES)
- toolBar -> AddTool(wxID_HTML_OPENFILE, wopenBitmap, wxNullBitmap,
+ if (style & wxHF_OPEN_FILES)
+ toolBar->AddTool(wxID_HTML_OPENFILE, wopenBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Open HTML document"));
#if wxUSE_PRINTING_ARCHITECTURE
if (style & wxHF_PRINT)
- toolBar -> AddTool(wxID_HTML_PRINT, wprintBitmap, wxNullBitmap,
+ toolBar->AddTool(wxID_HTML_PRINT, wprintBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Print this page"));
#endif
- toolBar -> AddSeparator();
- toolBar -> AddTool(wxID_HTML_OPTIONS, woptionsBitmap, wxNullBitmap,
+ toolBar->AddSeparator();
+ toolBar->AddTool(wxID_HTML_OPTIONS, woptionsBitmap, wxNullBitmap,
FALSE, -1, -1, (wxObject *) NULL,
_("Display options dialog"));
}
@@ -546,7 +622,8 @@ void wxHtmlHelpFrame::SetTitleFormat(const wxString& format)
bool wxHtmlHelpFrame::Display(const wxString& x)
{
wxString url = m_Data->FindPageByName(x);
- if (! url.IsEmpty()) {
+ if (!url.IsEmpty())
+ {
m_HtmlWin->LoadPage(url);
NotifyPageChanged();
return TRUE;
@@ -557,7 +634,8 @@ bool wxHtmlHelpFrame::Display(const wxString& x)
bool wxHtmlHelpFrame::Display(const int id)
{
wxString url = m_Data->FindPageById(id);
- if (! url.IsEmpty()) {
+ if (!url.IsEmpty())
+ {
m_HtmlWin->LoadPage(url);
NotifyPageChanged();
return TRUE;
@@ -571,13 +649,20 @@ bool wxHtmlHelpFrame::DisplayContents()
{
if (! m_ContentsBox)
return FALSE;
- if (!m_Splitter -> IsSplit()) {
- m_NavigPan -> Show(TRUE);
- m_HtmlWin -> Show(TRUE);
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ if (!m_Splitter->IsSplit())
+ {
+ m_NavigPan->Show(TRUE);
+ m_HtmlWin->Show(TRUE);
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
m_Cfg.navig_on = TRUE;
}
- m_NavigPan -> SetSelection(0);
+ m_NavigPan->SetSelection(0);
+ if (m_Data->GetBookRecArray().GetCount() > 0)
+ {
+ wxHtmlBookRecord& book = m_Data->GetBookRecArray()[0];
+ if (!book.GetStart().IsEmpty())
+ m_HtmlWin->LoadPage(book.GetFullPath(book.GetStart()));
+ }
return TRUE;
}
@@ -587,12 +672,19 @@ bool wxHtmlHelpFrame::DisplayIndex()
{
if (! m_IndexList)
return FALSE;
- if (!m_Splitter -> IsSplit()) {
- m_NavigPan -> Show(TRUE);
- m_HtmlWin -> Show(TRUE);
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ if (!m_Splitter->IsSplit())
+ {
+ m_NavigPan->Show(TRUE);
+ m_HtmlWin->Show(TRUE);
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ }
+ m_NavigPan->SetSelection(1);
+ if (m_Data->GetBookRecArray().GetCount() > 0)
+ {
+ wxHtmlBookRecord& book = m_Data->GetBookRecArray()[0];
+ if (!book.GetStart().IsEmpty())
+ m_HtmlWin->LoadPage(book.GetFullPath(book.GetStart()));
}
- m_NavigPan -> SetSelection(1);
return TRUE;
}
@@ -607,54 +699,56 @@ bool wxHtmlHelpFrame::KeywordSearch(const wxString& keyword)
wxString foundstr;
wxString book = wxEmptyString;
- if (!m_Splitter -> IsSplit()) {
- m_NavigPan -> Show(TRUE);
- m_HtmlWin -> Show(TRUE);
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ if (!m_Splitter->IsSplit())
+ {
+ m_NavigPan->Show(TRUE);
+ m_HtmlWin->Show(TRUE);
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
}
- m_NavigPan -> SetSelection(m_SearchPage);
- m_SearchList -> Clear();
- m_SearchText -> SetValue(keyword);
- m_SearchButton -> Enable(FALSE);
+ m_NavigPan->SetSelection(m_SearchPage);
+ m_SearchList->Clear();
+ m_SearchText->SetValue(keyword);
+ m_SearchButton->Enable(FALSE);
if (m_SearchChoice->GetSelection() != 0)
book = m_SearchChoice->GetStringSelection();
wxHtmlSearchStatus status(m_Data, keyword,
- m_SearchCaseSensitive -> GetValue(), m_SearchWholeWords -> GetValue(),
+ m_SearchCaseSensitive->GetValue(), m_SearchWholeWords->GetValue(),
book);
wxProgressDialog progress(_("Searching..."), _("No matching page found yet"),
status.GetMaxIndex(), this,
wxPD_APP_MODAL | wxPD_CAN_ABORT | wxPD_AUTO_HIDE);
- while (status.IsActive()) {
+ while (status.IsActive())
+ {
curi = status.GetCurIndex();
if (curi % 32 == 0 && progress.Update(curi) == FALSE)
break;
- if (status.Search()) {
+ if (status.Search())
+ {
foundstr.Printf(_("Found %i matches"), ++foundcnt);
progress.Update(status.GetCurIndex(), foundstr);
- m_SearchList -> Append(status.GetName(), status.GetContentsItem());
+ m_SearchList->Append(status.GetName(), status.GetContentsItem());
}
}
- m_SearchButton -> Enable(TRUE);
- m_SearchText -> SetSelection(0, keyword.Length());
- m_SearchText -> SetFocus();
- if (foundcnt) {
- wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_SearchList -> GetClientData(0);
+ m_SearchButton->Enable(TRUE);
+ m_SearchText->SetSelection(0, keyword.Length());
+ m_SearchText->SetFocus();
+ if (foundcnt)
+ {
+ wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_SearchList->GetClientData(0);
if (it)
{
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
+ m_HtmlWin->LoadPage(it->GetFullPath());
NotifyPageChanged();
}
}
return (foundcnt > 0);
}
-#define MAX_ROOTS 64
-
void wxHtmlHelpFrame::CreateContents()
{
if (! m_ContentsBox)
@@ -663,44 +757,101 @@ void wxHtmlHelpFrame::CreateContents()
m_ContentsBox->Clear();
if (m_PagesHash) delete m_PagesHash;
- m_PagesHash = new wxHashTable(wxKEY_STRING, 2 * m_Data -> GetContentsCnt());
- m_PagesHash -> DeleteContents(TRUE);
+ m_PagesHash = new wxHashTable(wxKEY_STRING, 2 * m_Data->GetContentsCnt());
+ m_PagesHash->DeleteContents(TRUE);
int cnt = m_Data->GetContentsCnt();
int i;
+ size_t booksCnt = m_Data->GetBookRecArray().GetCount();
wxHtmlContentsItem *it;
+ const int MAX_ROOTS = 64;
wxTreeItemId roots[MAX_ROOTS];
+ // VS: this array holds information about whether we've set item icon at
+ // given level. This is neccessary because m_Data has flat structure
+ // and there's no way of recognizing if some item has subitems or not.
+ // We set the icon later: when we find an item with level=n, we know
+ // that the last item with level=n-1 was folder with subitems, so we
+ // set its icon accordingly
bool imaged[MAX_ROOTS];
+ m_ContentsBox->DeleteAllItems();
+
+ // FIXME - will go away when wxMSW's wxTreeCtrl supports wxTR_HIDE_ROOT!
+ bool hasSuperRoot = (booksCnt > 1) ||
+ (m_ContentsBox->GetWindowStyle() & wxTR_HIDE_ROOT);
+
+ // Don't show (Help) root if there's only one boook
+ if (hasSuperRoot)
+ {
+ roots[0] = m_ContentsBox->AddRoot(_("(Help)"));
+ m_ContentsBox->SetItemImage(roots[0], IMG_RootFolder);
+ m_ContentsBox->SetItemSelectedImage(roots[0], IMG_RootFolder);
+ imaged[0] = TRUE;
+ }
+
+ for (it = m_Data->GetContents(), i = 0; i < cnt; i++, it++)
+ {
+ // Handle books:
+ if (it->m_Level == 0)
+ {
+ // special case, only one book, make it tree's root:
+ if (!hasSuperRoot)
+ {
+ roots[0] = roots[1] = m_ContentsBox->AddRoot(
+ it->m_Name, IMG_Book, -1,
+ new wxHtmlHelpTreeItemData(i));
+ imaged[0] = imaged[1] = TRUE;
+ m_ContentsBox->SetItemBold(roots[1], TRUE);
+ }
+ // multiple books:
+ else
+ {
+ if (m_hfStyle & wxHF_MERGE_BOOKS)
+ // VS: we don't want book nodes, books' content should
+ // appear under tree's root. This line will create "fake"
+ // record about book node so that the rest of this look
+ // will believe there really _is_ book node and will
+ // behave correctly.
+ roots[1] = roots[0];
+ else
+ {
+ roots[1] = m_ContentsBox->AppendItem(roots[0],
+ it->m_Name, IMG_Book, -1,
+ new wxHtmlHelpTreeItemData(i));
+ m_ContentsBox->SetItemBold(roots[1], TRUE);
+ }
+ imaged[1] = TRUE;
+ }
+ }
+ // ...and their contents:
+ else
+ {
+ roots[it->m_Level + 1] = m_ContentsBox->AppendItem(
+ roots[it->m_Level], it->m_Name, IMG_Page,
+ -1, new wxHtmlHelpTreeItemData(i));
+ imaged[it->m_Level + 1] = FALSE;
+ }
- m_ContentsBox -> DeleteAllItems();
- roots[0] = m_ContentsBox -> AddRoot(_("(Help)"));
- m_ContentsBox -> SetItemImage(roots[0], IMG_RootFolder);
- m_ContentsBox -> SetItemSelectedImage(roots[0], IMG_RootFolder);
- imaged[0] = TRUE;
-
- for (it = m_Data->GetContents(), i = 0; i < cnt; i++, it++) {
- roots[it -> m_Level + 1] = m_ContentsBox -> AppendItem(
- roots[it -> m_Level], it -> m_Name, IMG_Page, -1,
- new wxHtmlHelpTreeItemData(i));
- m_PagesHash -> Put(it -> m_Book -> GetBasePath() + it -> m_Page,
- new wxHtmlHelpHashData(i, roots[it -> m_Level + 1]));
-
- if (it -> m_Level == 0) {
- m_ContentsBox -> SetItemBold(roots[1], TRUE);
- m_ContentsBox -> SetItemImage(roots[1], IMG_Book);
- m_ContentsBox -> SetItemSelectedImage(roots[1], IMG_Book);
- imaged[1] = TRUE;
- } else imaged[it -> m_Level + 1] = FALSE;
-
- if (!imaged[it -> m_Level]) {
- m_ContentsBox -> SetItemImage(roots[it -> m_Level], IMG_Folder);
- m_ContentsBox -> SetItemSelectedImage(roots[it -> m_Level], IMG_Folder);
- imaged[it -> m_Level] = TRUE;
+ m_PagesHash->Put(it->GetFullPath(),
+ new wxHtmlHelpHashData(i, roots[it->m_Level + 1]));
+
+ // Set the icon for the node one level up in the hiearachy,
+ // unless already done (see comment above imaged[] declaration)
+ if (!imaged[it->m_Level])
+ {
+ int image = IMG_Folder;
+ if (m_hfStyle & wxHF_ICONS_BOOK)
+ image = IMG_Book;
+ else if (m_hfStyle & wxHF_ICONS_BOOK_CHAPTER)
+ image = (it->m_Level == 1) ? IMG_Book : IMG_Folder;
+ m_ContentsBox->SetItemImage(roots[it->m_Level], image);
+ m_ContentsBox->SetItemSelectedImage(roots[it->m_Level], image);
+ imaged[it->m_Level] = TRUE;
}
}
- m_ContentsBox -> Expand(roots[0]);
+
+ m_ContentsBox->Expand(roots[0]);
}
@@ -716,22 +867,22 @@ void wxHtmlHelpFrame::CreateIndex()
wxString cnttext;
if (cnt > INDEX_IS_SMALL) cnttext.Printf(_("%i of %i"), 0, cnt);
else cnttext.Printf(_("%i of %i"), cnt, cnt);
- m_IndexCountInfo -> SetLabel(cnttext);
+ m_IndexCountInfo->SetLabel(cnttext);
if (cnt > INDEX_IS_SMALL) return;
wxHtmlContentsItem* index = m_Data->GetIndex();
for (int i = 0; i < cnt; i++)
- m_IndexList -> Append(index[i].m_Name, (char*)(index + i));
+ m_IndexList->Append(index[i].m_Name, (char*)(index + i));
}
void wxHtmlHelpFrame::CreateSearch()
{
if (! (m_SearchList && m_SearchChoice))
return ;
- m_SearchList -> Clear();
- m_SearchChoice -> Clear();
- m_SearchChoice -> Append(_("Search in all books"));
+ m_SearchList->Clear();
+ m_SearchChoice->Clear();
+ m_SearchChoice->Append(_("Search in all books"));
const wxHtmlBookRecArray& bookrec = m_Data->GetBookRecArray();
int i, cnt = bookrec.GetCount();
for (i = 0; i < cnt; i++)
@@ -752,43 +903,47 @@ void wxHtmlHelpFrame::ReadCustomization(wxConfigBase *cfg, const wxString& path)
wxString oldpath;
wxString tmp;
- if (path != wxEmptyString) {
- oldpath = cfg -> GetPath();
- cfg -> SetPath(_T("/") + path);
+ if (path != wxEmptyString)
+ {
+ oldpath = cfg->GetPath();
+ cfg->SetPath(_T("/") + path);
}
- m_Cfg.navig_on = cfg -> Read(wxT("hcNavigPanel"), m_Cfg.navig_on) != 0;
- m_Cfg.sashpos = cfg -> Read(wxT("hcSashPos"), m_Cfg.sashpos);
- m_Cfg.x = cfg -> Read(wxT("hcX"), m_Cfg.x);
- m_Cfg.y = cfg -> Read(wxT("hcY"), m_Cfg.y);
- m_Cfg.w = cfg -> Read(wxT("hcW"), m_Cfg.w);
- m_Cfg.h = cfg -> Read(wxT("hcH"), m_Cfg.h);
+ m_Cfg.navig_on = cfg->Read(wxT("hcNavigPanel"), m_Cfg.navig_on) != 0;
+ m_Cfg.sashpos = cfg->Read(wxT("hcSashPos"), m_Cfg.sashpos);
+ m_Cfg.x = cfg->Read(wxT("hcX"), m_Cfg.x);
+ m_Cfg.y = cfg->Read(wxT("hcY"), m_Cfg.y);
+ m_Cfg.w = cfg->Read(wxT("hcW"), m_Cfg.w);
+ m_Cfg.h = cfg->Read(wxT("hcH"), m_Cfg.h);
- m_FixedFace = cfg -> Read(wxT("hcFixedFace"), m_FixedFace);
- m_NormalFace = cfg -> Read(wxT("hcNormalFace"), m_NormalFace);
- m_FontSize = cfg -> Read(wxT("hcFontSize"), m_FontSize);
+ m_FixedFace = cfg->Read(wxT("hcFixedFace"), m_FixedFace);
+ m_NormalFace = cfg->Read(wxT("hcNormalFace"), m_NormalFace);
+ m_FontSize = cfg->Read(wxT("hcFontSize"), m_FontSize);
{
int i;
int cnt;
wxString val, s;
- cnt = cfg -> Read(wxT("hcBookmarksCnt"), 0L);
- if (cnt != 0) {
+ cnt = cfg->Read(wxT("hcBookmarksCnt"), 0L);
+ if (cnt != 0)
+ {
m_BookmarksNames.Clear();
m_BookmarksPages.Clear();
- if (m_Bookmarks) {
- m_Bookmarks -> Clear();
- m_Bookmarks -> Append(_("(bookmarks)"));
+ if (m_Bookmarks)
+ {
+ m_Bookmarks->Clear();
+ m_Bookmarks->Append(_("(bookmarks)"));
}
- for (i = 0; i < cnt; i++) {
+ for (i = 0; i < cnt; i++)
+ {
val.Printf(wxT("hcBookmark_%i"), i);
- s = cfg -> Read(val);
+ s = cfg->Read(val);
m_BookmarksNames.Add(s);
- if (m_Bookmarks) m_Bookmarks -> Append(s);
+ if (m_Bookmarks) m_Bookmarks->Append(s);
val.Printf(wxT("hcBookmark_%i_url"), i);
- s = cfg -> Read(val);
+ s = cfg->Read(val);
m_BookmarksPages.Add(s);
}
}
@@ -798,7 +953,7 @@ void wxHtmlHelpFrame::ReadCustomization(wxConfigBase *cfg, const wxString& path)
m_HtmlWin->ReadCustomization(cfg);
if (path != wxEmptyString)
- cfg -> SetPath(oldpath);
+ cfg->SetPath(oldpath);
}
void wxHtmlHelpFrame::WriteCustomization(wxConfigBase *cfg, const wxString& path)
@@ -806,32 +961,35 @@ void wxHtmlHelpFrame::WriteCustomization(wxConfigBase *cfg, const wxString& path
wxString oldpath;
wxString tmp;
- if (path != wxEmptyString) {
- oldpath = cfg -> GetPath();
- cfg -> SetPath(_T("/") + path);
+ if (path != wxEmptyString)
+ {
+ oldpath = cfg->GetPath();
+ cfg->SetPath(_T("/") + path);
}
- cfg -> Write(wxT("hcNavigPanel"), m_Cfg.navig_on);
- cfg -> Write(wxT("hcSashPos"), (long)m_Cfg.sashpos);
- cfg -> Write(wxT("hcX"), (long)m_Cfg.x);
- cfg -> Write(wxT("hcY"), (long)m_Cfg.y);
- cfg -> Write(wxT("hcW"), (long)m_Cfg.w);
- cfg -> Write(wxT("hcH"), (long)m_Cfg.h);
- cfg -> Write(wxT("hcFixedFace"), m_FixedFace);
- cfg -> Write(wxT("hcNormalFace"), m_NormalFace);
- cfg -> Write(wxT("hcFontSize"), (long)m_FontSize);
-
- if (m_Bookmarks) {
+ cfg->Write(wxT("hcNavigPanel"), m_Cfg.navig_on);
+ cfg->Write(wxT("hcSashPos"), (long)m_Cfg.sashpos);
+ cfg->Write(wxT("hcX"), (long)m_Cfg.x);
+ cfg->Write(wxT("hcY"), (long)m_Cfg.y);
+ cfg->Write(wxT("hcW"), (long)m_Cfg.w);
+ cfg->Write(wxT("hcH"), (long)m_Cfg.h);
+ cfg->Write(wxT("hcFixedFace"), m_FixedFace);
+ cfg->Write(wxT("hcNormalFace"), m_NormalFace);
+ cfg->Write(wxT("hcFontSize"), (long)m_FontSize);
+
+ if (m_Bookmarks)
+ {
int i;
int cnt = m_BookmarksNames.GetCount();
wxString val;
- cfg -> Write(wxT("hcBookmarksCnt"), (long)cnt);
- for (i = 0; i < cnt; i++) {
+ cfg->Write(wxT("hcBookmarksCnt"), (long)cnt);
+ for (i = 0; i < cnt; i++)
+ {
val.Printf(wxT("hcBookmark_%i"), i);
- cfg -> Write(val, m_BookmarksNames[i]);
+ cfg->Write(val, m_BookmarksNames[i]);
val.Printf(wxT("hcBookmark_%i_url"), i);
- cfg -> Write(val, m_BookmarksPages[i]);
+ cfg->Write(val, m_BookmarksPages[i]);
}
}
@@ -839,7 +997,7 @@ void wxHtmlHelpFrame::WriteCustomization(wxConfigBase *cfg, const wxString& path
m_HtmlWin->WriteCustomization(cfg);
if (path != wxEmptyString)
- cfg -> SetPath(oldpath);
+ cfg->SetPath(oldpath);
}
@@ -857,7 +1015,7 @@ static void SetFontsToHtmlWin(wxHtmlWindow *win, wxString scalf, wxString fixf,
{16, 20, 24, 32, 38, 45, 50}
};
- win -> SetFonts(scalf, fixf, f_sizes[size]);
+ win->SetFonts(scalf, fixf, f_sizes[size]);
}
@@ -878,40 +1036,41 @@ class wxHtmlHelpFrameOptionsDialog : public wxDialog
sizer = new wxBoxSizer(wxHORIZONTAL);
sizer2 = new wxStaticBoxSizer( new wxStaticBox(this, -1, _("Normal font:")), wxVERTICAL);
- sizer2 -> Add(NormalFont = new wxComboBox(this, -1, wxEmptyString, wxDefaultPosition,
+ sizer2->Add(NormalFont = new wxComboBox(this, -1, wxEmptyString, wxDefaultPosition,
wxSize(200, 200),
0, NULL, wxCB_DROPDOWN | wxCB_READONLY),
1, wxEXPAND | wxLEFT | wxRIGHT, 10);
- sizer -> Add(sizer2, 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10);
+ sizer->Add(sizer2, 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10);
sizer2 = new wxStaticBoxSizer( new wxStaticBox(this, -1, _("Fixed font:")), wxVERTICAL);
- sizer2 -> Add(FixedFont = new wxComboBox(this, -1, wxEmptyString, wxDefaultPosition,
+ sizer2->Add(FixedFont = new wxComboBox(this, -1, wxEmptyString, wxDefaultPosition,
wxSize(200, 200),
0, NULL, wxCB_DROPDOWN | wxCB_READONLY),
1, wxEXPAND | wxLEFT | wxRIGHT, 10);
- sizer -> Add(sizer2, 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10);
+ sizer->Add(sizer2, 0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10);
- topsizer -> Add(sizer);
+ topsizer->Add(sizer);
- topsizer -> Add(RadioBox = new wxRadioBox(this, -1, _("Font size:"),
+ topsizer->Add(RadioBox = new wxRadioBox(this, -1, _("Font size:"),
wxDefaultPosition, wxDefaultSize, 5, choices, 5),
0, wxEXPAND | wxLEFT|wxRIGHT|wxTOP, 10);
- topsizer -> Add(new wxStaticText(this, -1, _("Preview:")),
+ topsizer->Add(new wxStaticText(this, -1, _("Preview:")),
0, wxLEFT | wxTOP, 10);
- topsizer -> Add(TestWin = new wxHtmlWindow(this, -1, wxDefaultPosition, wxSize(-1, 150)),
+ topsizer->Add(TestWin = new wxHtmlWindow(this, -1, wxDefaultPosition, wxSize(-1, 150),
+ wxHW_SCROLLBAR_AUTO | wxSUNKEN_BORDER),
1, wxEXPAND | wxLEFT|wxTOP|wxRIGHT, 10);
sizer = new wxBoxSizer(wxHORIZONTAL);
- sizer -> Add(new wxButton(this, wxID_OK, _("OK")), 0, wxALL, 10);
- sizer -> Add(new wxButton(this, wxID_CANCEL, _("Cancel")), 0, wxALL, 10);
- topsizer -> Add(sizer, 0, wxALIGN_RIGHT);
+ sizer->Add(new wxButton(this, wxID_OK, _("OK")), 0, wxALL, 10);
+ sizer->Add(new wxButton(this, wxID_CANCEL, _("Cancel")), 0, wxALL, 10);
+ topsizer->Add(sizer, 0, wxALIGN_RIGHT);
SetAutoLayout(TRUE);
SetSizer(topsizer);
- topsizer -> Fit(this);
+ topsizer->Fit(this);
Centre(wxBOTH);
}
@@ -920,10 +1079,10 @@ class wxHtmlHelpFrameOptionsDialog : public wxDialog
{
wxBusyCursor bcur;
SetFontsToHtmlWin(TestWin,
- NormalFont -> GetStringSelection(),
- FixedFont -> GetStringSelection(),
- RadioBox -> GetSelection());
- TestWin -> SetPage(_(
+ NormalFont->GetStringSelection(),
+ FixedFont->GetStringSelection(),
+ RadioBox->GetSelection());
+ TestWin->SetPage(_(
"\
Normal face
(and underlined. Italic face. \
Bold face. Bold italic face.
\
@@ -948,7 +1107,7 @@ Normal face
(and underlined. Italic face. \
));
}
- void OnUpdate(wxCommandEvent& event)
+ void OnUpdate(wxCommandEvent& WXUNUSED(event))
{
UpdateTestWin();
}
@@ -967,36 +1126,39 @@ void wxHtmlHelpFrame::OptionsDialog()
wxHtmlHelpFrameOptionsDialog dlg(this);
unsigned i;
- if (m_NormalFonts == NULL) {
+ if (m_NormalFonts == NULL)
+ {
wxFontEnumerator enu;
enu.EnumerateFacenames();
m_NormalFonts = new wxArrayString;
*m_NormalFonts = *enu.GetFacenames();
- m_NormalFonts -> Sort();
+ m_NormalFonts->Sort();
}
- if (m_FixedFonts == NULL) {
+ if (m_FixedFonts == NULL)
+ {
wxFontEnumerator enu;
enu.EnumerateFacenames(wxFONTENCODING_SYSTEM, TRUE);
m_FixedFonts = new wxArrayString;
*m_FixedFonts = *enu.GetFacenames();
- m_FixedFonts -> Sort();
+ m_FixedFonts->Sort();
}
- for (i = 0; i < m_NormalFonts -> GetCount(); i++)
- dlg.NormalFont -> Append((*m_NormalFonts)[i]);
- for (i = 0; i < m_FixedFonts -> GetCount(); i++)
- dlg.FixedFont -> Append((*m_FixedFonts)[i]);
- if (!m_NormalFace.IsEmpty()) dlg.NormalFont -> SetStringSelection(m_NormalFace);
- else dlg.NormalFont -> SetSelection(0);
- if (!m_FixedFace.IsEmpty()) dlg.FixedFont -> SetStringSelection(m_FixedFace);
- else dlg.FixedFont -> SetSelection(0);
- dlg.RadioBox -> SetSelection(m_FontSize);
+ for (i = 0; i < m_NormalFonts->GetCount(); i++)
+ dlg.NormalFont->Append((*m_NormalFonts)[i]);
+ for (i = 0; i < m_FixedFonts->GetCount(); i++)
+ dlg.FixedFont->Append((*m_FixedFonts)[i]);
+ if (!m_NormalFace.IsEmpty()) dlg.NormalFont->SetStringSelection(m_NormalFace);
+ else dlg.NormalFont->SetSelection(0);
+ if (!m_FixedFace.IsEmpty()) dlg.FixedFont->SetStringSelection(m_FixedFace);
+ else dlg.FixedFont->SetSelection(0);
+ dlg.RadioBox->SetSelection(m_FontSize);
dlg.UpdateTestWin();
- if (dlg.ShowModal() == wxID_OK) {
- m_NormalFace = dlg.NormalFont -> GetStringSelection();
- m_FixedFace = dlg.FixedFont -> GetStringSelection();
- m_FontSize = dlg.RadioBox -> GetSelection();
+ if (dlg.ShowModal() == wxID_OK)
+ {
+ m_NormalFace = dlg.NormalFont->GetStringSelection();
+ m_FixedFace = dlg.FixedFont->GetStringSelection();
+ m_FontSize = dlg.RadioBox->GetSelection();
SetFontsToHtmlWin(m_HtmlWin, m_NormalFace, m_FixedFace, m_FontSize);
}
}
@@ -1007,18 +1169,18 @@ void wxHtmlHelpFrame::NotifyPageChanged()
{
if (m_UpdateContents && m_PagesHash)
{
- wxString an = m_HtmlWin -> GetOpenedAnchor();
+ wxString an = m_HtmlWin->GetOpenedAnchor();
wxHtmlHelpHashData *ha;
if (an.IsEmpty())
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage());
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage());
else
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage() + wxT("#") + an);
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage() + wxT("#") + an);
if (ha)
{
bool olduc = m_UpdateContents;
m_UpdateContents = FALSE;
- m_ContentsBox -> SelectItem(ha -> m_Id);
- m_ContentsBox -> EnsureVisible(ha -> m_Id);
+ m_ContentsBox->SelectItem(ha->m_Id);
+ m_ContentsBox->EnsureVisible(ha->m_Id);
m_UpdateContents = olduc;
}
}
@@ -1033,32 +1195,35 @@ EVENT HANDLING :
void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
{
- switch (event.GetId()) {
-
+ switch (event.GetId())
+ {
case wxID_HTML_BACK :
- m_HtmlWin -> HistoryBack();
+ m_HtmlWin->HistoryBack();
NotifyPageChanged();
break;
case wxID_HTML_FORWARD :
- m_HtmlWin -> HistoryForward();
+ m_HtmlWin->HistoryForward();
NotifyPageChanged();
break;
case wxID_HTML_UP :
if (m_PagesHash)
{
- wxString an = m_HtmlWin -> GetOpenedAnchor();
+ wxString an = m_HtmlWin->GetOpenedAnchor();
wxHtmlHelpHashData *ha;
if (an.IsEmpty())
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage());
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage());
else
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage() + wxT("#") + an);
- if (ha && ha -> m_Index > 0)
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage() + wxT("#") + an);
+ if (ha && ha->m_Index > 0)
{
- wxHtmlContentsItem *it = m_Data -> GetContents() + (ha -> m_Index - 1);
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
- NotifyPageChanged();
+ wxHtmlContentsItem *it = m_Data->GetContents() + (ha->m_Index - 1);
+ if (it->m_Page[0] != 0)
+ {
+ m_HtmlWin->LoadPage(it->GetFullPath());
+ NotifyPageChanged();
+ }
}
}
break;
@@ -1066,24 +1231,27 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
case wxID_HTML_UPNODE :
if (m_PagesHash)
{
- wxString an = m_HtmlWin -> GetOpenedAnchor();
+ wxString an = m_HtmlWin->GetOpenedAnchor();
wxHtmlHelpHashData *ha;
if (an.IsEmpty())
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage());
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage());
else
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(m_HtmlWin -> GetOpenedPage() + wxT("#") + an);
- if (ha && ha -> m_Index > 0)
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(m_HtmlWin->GetOpenedPage() + wxT("#") + an);
+ if (ha && ha->m_Index > 0)
{
- int level = m_Data -> GetContents()[ha -> m_Index].m_Level - 1;
+ int level = m_Data->GetContents()[ha->m_Index].m_Level - 1;
wxHtmlContentsItem *it;
- int ind = ha -> m_Index - 1;
+ int ind = ha->m_Index - 1;
- it = m_Data -> GetContents() + ind;
- while (ind >= 0 && it -> m_Level != level) ind--, it--;
+ it = m_Data->GetContents() + ind;
+ while (ind >= 0 && it->m_Level != level) ind--, it--;
if (ind >= 0)
{
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
- NotifyPageChanged();
+ if (it->m_Page[0] != 0)
+ {
+ m_HtmlWin->LoadPage(it->GetFullPath());
+ NotifyPageChanged();
+ }
}
}
}
@@ -1092,22 +1260,26 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
case wxID_HTML_DOWN :
if (m_PagesHash)
{
- wxString an = m_HtmlWin -> GetOpenedAnchor();
+ wxString an = m_HtmlWin->GetOpenedAnchor();
wxString adr;
wxHtmlHelpHashData *ha;
- if (an.IsEmpty()) adr = m_HtmlWin -> GetOpenedPage();
- else adr = m_HtmlWin -> GetOpenedPage() + wxT("#") + an;
+ if (an.IsEmpty()) adr = m_HtmlWin->GetOpenedPage();
+ else adr = m_HtmlWin->GetOpenedPage() + wxT("#") + an;
- ha = (wxHtmlHelpHashData*) m_PagesHash -> Get(adr);
+ ha = (wxHtmlHelpHashData*) m_PagesHash->Get(adr);
- if (ha && ha -> m_Index < m_Data -> GetContentsCnt() - 1)
+ if (ha && ha->m_Index < m_Data->GetContentsCnt() - 1)
{
- wxHtmlContentsItem *it = m_Data -> GetContents() + (ha -> m_Index + 1);
+ wxHtmlContentsItem *it = m_Data->GetContents() + (ha->m_Index + 1);
- while (it -> m_Book -> GetBasePath() + it -> m_Page == adr) it++;
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
- NotifyPageChanged();
+ while (it->GetFullPath() == adr) it++;
+
+ if (it->m_Page[0] != 0)
+ {
+ m_HtmlWin->LoadPage(it->GetFullPath());
+ NotifyPageChanged();
+ }
}
}
break;
@@ -1116,14 +1288,17 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
{
if (! (m_Splitter && m_NavigPan))
return ;
- if (m_Splitter -> IsSplit()) {
- m_Cfg.sashpos = m_Splitter -> GetSashPosition();
- m_Splitter -> Unsplit(m_NavigPan);
+ if (m_Splitter->IsSplit())
+ {
+ m_Cfg.sashpos = m_Splitter->GetSashPosition();
+ m_Splitter->Unsplit(m_NavigPan);
m_Cfg.navig_on = FALSE;
- } else {
- m_NavigPan -> Show(TRUE);
- m_HtmlWin -> Show(TRUE);
- m_Splitter -> SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
+ }
+ else
+ {
+ m_NavigPan->Show(TRUE);
+ m_HtmlWin->Show(TRUE);
+ m_Splitter->SplitVertically(m_NavigPan, m_HtmlWin, m_Cfg.sashpos);
m_Cfg.navig_on = TRUE;
}
}
@@ -1138,11 +1313,13 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
wxString item;
wxString url;
- item = m_HtmlWin -> GetOpenedPageTitle();
- url = m_HtmlWin -> GetOpenedPage();
- if (item == wxEmptyString) item = url.AfterLast(wxT('/'));
- if (m_BookmarksPages.Index(url) == wxNOT_FOUND) {
- m_Bookmarks -> Append(item);
+ item = m_HtmlWin->GetOpenedPageTitle();
+ url = m_HtmlWin->GetOpenedPage();
+ if (item == wxEmptyString)
+ item = url.AfterLast(wxT('/'));
+ if (m_BookmarksPages.Index(url) == wxNOT_FOUND)
+ {
+ m_Bookmarks->Append(item);
m_BookmarksNames.Add(item);
m_BookmarksPages.Add(url);
}
@@ -1154,12 +1331,13 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
wxString item;
int pos;
- item = m_Bookmarks -> GetStringSelection();
+ item = m_Bookmarks->GetStringSelection();
pos = m_BookmarksNames.Index(item);
- if (pos != wxNOT_FOUND) {
+ if (pos != wxNOT_FOUND)
+ {
m_BookmarksNames.Remove(pos);
m_BookmarksPages.Remove(pos);
- m_Bookmarks -> Delete(m_Bookmarks -> GetSelection());
+ m_Bookmarks->Delete(m_Bookmarks->GetSelection());
}
}
break;
@@ -1169,10 +1347,10 @@ void wxHtmlHelpFrame::OnToolbar(wxCommandEvent& event)
{
if (m_Printer == NULL)
m_Printer = new wxHtmlEasyPrinting(_("Help Printing"), this);
- if (!m_HtmlWin -> GetOpenedPage())
+ if (!m_HtmlWin->GetOpenedPage())
wxLogWarning(_("Cannot print empty page."));
else
- m_Printer -> PrintFile(m_HtmlWin -> GetOpenedPage());
+ m_Printer->PrintFile(m_HtmlWin->GetOpenedPage());
}
break;
#endif
@@ -1197,11 +1375,11 @@ All files (*.*)|*"
if (ext == _T(".zip") || ext == _T(".htb") || ext == _T(".hhp"))
{
wxBusyCursor bcur;
- m_Data -> AddBook(s);
+ m_Data->AddBook(s);
RefreshLists();
}
else
- m_HtmlWin -> LoadPage(s);
+ m_HtmlWin->LoadPage(s);
}
}
break;
@@ -1215,13 +1393,14 @@ void wxHtmlHelpFrame::OnContentsSel(wxTreeEvent& event)
wxHtmlHelpTreeItemData *pg;
wxHtmlContentsItem *it;
- pg = (wxHtmlHelpTreeItemData*) m_ContentsBox -> GetItemData(event.GetItem());
+ pg = (wxHtmlHelpTreeItemData*) m_ContentsBox->GetItemData(event.GetItem());
if (pg && m_UpdateContents)
{
- it = m_Data -> GetContents() + (pg -> m_Id);
+ it = m_Data->GetContents() + (pg->m_Id);
m_UpdateContents = FALSE;
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
+ if (it->m_Page[0] != 0)
+ m_HtmlWin->LoadPage(it->GetFullPath());
m_UpdateContents = TRUE;
}
}
@@ -1230,23 +1409,27 @@ void wxHtmlHelpFrame::OnContentsSel(wxTreeEvent& event)
void wxHtmlHelpFrame::OnIndexSel(wxCommandEvent& WXUNUSED(event))
{
- wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_IndexList -> GetClientData(m_IndexList -> GetSelection());
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
+ wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_IndexList->GetClientData(m_IndexList->GetSelection());
+ if (it->m_Page[0] != 0)
+ m_HtmlWin->LoadPage(it->GetFullPath());
NotifyPageChanged();
}
void wxHtmlHelpFrame::OnIndexFind(wxCommandEvent& event)
{
- wxString sr = m_IndexText -> GetLineText(0);
+ wxString sr = m_IndexText->GetLineText(0);
sr.MakeLower();
if (sr == wxEmptyString)
+ {
OnIndexAll(event);
-
- else {
+ }
+ else
+ {
wxBusyCursor bcur;
const wxChar *cstr = sr.c_str();
- wxChar mybuff[512], *ptr;
+ wxChar mybuff[512];
+ wxChar *ptr;
bool first = TRUE;
m_IndexList->Clear();
@@ -1261,11 +1444,14 @@ void wxHtmlHelpFrame::OnIndexFind(wxCommandEvent& event)
for (ptr = mybuff; *ptr != 0; ptr++)
if (*ptr >= _T('A') && *ptr <= _T('Z'))
*ptr -= (wxChar)(_T('A') - _T('a'));
- if (wxStrstr(mybuff, cstr) != NULL) {
- m_IndexList -> Append(index[i].m_Name, (char*)(index + i));
+ if (wxStrstr(mybuff, cstr) != NULL)
+ {
+ m_IndexList->Append(index[i].m_Name, (char*)(index + i));
displ++;
- if (first) {
- m_HtmlWin -> LoadPage(index[i].m_Book -> GetBasePath() + index[i].m_Page);
+ if (first)
+ {
+ if (index[i].m_Page[0] != 0)
+ m_HtmlWin->LoadPage(index[i].GetFullPath());
NotifyPageChanged();
first = FALSE;
}
@@ -1274,10 +1460,10 @@ void wxHtmlHelpFrame::OnIndexFind(wxCommandEvent& event)
wxString cnttext;
cnttext.Printf(_("%i of %i"), displ, cnt);
- m_IndexCountInfo -> SetLabel(cnttext);
+ m_IndexCountInfo->SetLabel(cnttext);
- m_IndexText -> SetSelection(0, sr.Length());
- m_IndexText -> SetFocus();
+ m_IndexText->SetSelection(0, sr.Length());
+ m_IndexText->SetFocus();
}
}
@@ -1290,10 +1476,13 @@ void wxHtmlHelpFrame::OnIndexAll(wxCommandEvent& WXUNUSED(event))
bool first = TRUE;
wxHtmlContentsItem* index = m_Data->GetIndex();
- for (int i = 0; i < cnt; i++) {
- m_IndexList -> Append(index[i].m_Name, (char*)(index + i));
- if (first) {
- m_HtmlWin -> LoadPage(index[i].m_Book -> GetBasePath() + index[i].m_Page);
+ for (int i = 0; i < cnt; i++)
+ {
+ m_IndexList->Append(index[i].m_Name, (char*)(index + i));
+ if (first)
+ {
+ if (index[i].m_Page[0] != 0)
+ m_HtmlWin->LoadPage(index[i].GetFullPath());
NotifyPageChanged();
first = FALSE;
}
@@ -1301,34 +1490,35 @@ void wxHtmlHelpFrame::OnIndexAll(wxCommandEvent& WXUNUSED(event))
wxString cnttext;
cnttext.Printf(_("%i of %i"), cnt, cnt);
- m_IndexCountInfo -> SetLabel(cnttext);
+ m_IndexCountInfo->SetLabel(cnttext);
}
void wxHtmlHelpFrame::OnSearchSel(wxCommandEvent& WXUNUSED(event))
{
- wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_SearchList -> GetClientData(m_SearchList -> GetSelection());
+ wxHtmlContentsItem *it = (wxHtmlContentsItem*) m_SearchList->GetClientData(m_SearchList->GetSelection());
if (it)
{
- m_HtmlWin -> LoadPage(it -> m_Book -> GetBasePath() + it -> m_Page);
+ if (it->m_Page[0] != 0)
+ m_HtmlWin->LoadPage(it->GetFullPath());
NotifyPageChanged();
}
}
void wxHtmlHelpFrame::OnSearch(wxCommandEvent& WXUNUSED(event))
{
- wxString sr = m_SearchText -> GetLineText(0);
+ wxString sr = m_SearchText->GetLineText(0);
if (sr != wxEmptyString) KeywordSearch(sr);
}
void wxHtmlHelpFrame::OnBookmarksSel(wxCommandEvent& WXUNUSED(event))
{
- wxString sr = m_Bookmarks -> GetStringSelection();
+ wxString sr = m_Bookmarks->GetStringSelection();
if (sr != wxEmptyString && sr != _("(bookmarks)"))
{
- m_HtmlWin -> LoadPage(m_BookmarksPages[m_BookmarksNames.Index(sr)]);
+ m_HtmlWin->LoadPage(m_BookmarksPages[m_BookmarksNames.Index(sr)]);
NotifyPageChanged();
}
}
@@ -1338,7 +1528,7 @@ void wxHtmlHelpFrame::OnCloseWindow(wxCloseEvent& evt)
GetSize(&m_Cfg.w, &m_Cfg.h);
GetPosition(&m_Cfg.x, &m_Cfg.y);
- if (m_Splitter && m_Cfg.navig_on) m_Cfg.sashpos = m_Splitter -> GetSashPosition();
+ if (m_Splitter && m_Cfg.navig_on) m_Cfg.sashpos = m_Splitter->GetSashPosition();
if (m_Config)
WriteCustomization(m_Config, m_ConfigRoot);
@@ -1367,4 +1557,5 @@ BEGIN_EVENT_TABLE(wxHtmlHelpFrame, wxFrame)
EVT_CLOSE(wxHtmlHelpFrame::OnCloseWindow)
END_EVENT_TABLE()
-#endif
+#endif // wxUSE_WXHTML_HELP
+