From 00655497da3542fb363c136c929b31fb9c7a67d9 Mon Sep 17 00:00:00 2001 From: Robert Roebling Date: Tue, 7 Dec 1999 14:54:32 +0000 Subject: [PATCH] Minor optical changes to toolbar and wxHelpFrame. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@4856 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- src/gtk/tbargtk.cpp | 33 ++++------------- src/gtk1/tbargtk.cpp | 33 ++++------------- src/html/bitmaps/wbkadd.xpm | 22 ++++++------ src/html/bitmaps/wfolder.xpm | 69 ++++++++++++------------------------ src/html/helpfrm.cpp | 5 ++- 5 files changed, 49 insertions(+), 113 deletions(-) diff --git a/src/gtk/tbargtk.cpp b/src/gtk/tbargtk.cpp index 1e0f1d9f5f..7df75acd3a 100644 --- a/src/gtk/tbargtk.cpp +++ b/src/gtk/tbargtk.cpp @@ -178,8 +178,7 @@ bool wxToolBar::Create( wxWindow *parent, wxWindowID id, m_toolbar = GTK_TOOLBAR( gtk_toolbar_new( GTK_ORIENTATION_HORIZONTAL, GTK_TOOLBAR_ICONS ) ); -// gtk_toolbar_set_space_style( m_toolbar, GTK_TOOLBAR_SPACE_LINE ); - m_separation = 5; + m_separation = 7; gtk_toolbar_set_space_size( m_toolbar, m_separation ); m_hasToolAlready = FALSE; @@ -330,8 +329,8 @@ wxToolBarTool *wxToolBar::AddTool( int toolIndex, const wxBitmap& bitmap, GtkRequisition req; (* GTK_WIDGET_CLASS( GTK_OBJECT(m_widget)->klass )->size_request ) (m_widget, &req ); - m_width = req.width; - m_height = req.height; + m_width = req.width + m_xMargin; + m_height = req.height + 2*m_yMargin + 4; gtk_signal_connect( GTK_OBJECT(tool->m_item), "enter_notify_event", @@ -359,8 +358,8 @@ bool wxToolBar::AddControl(wxControl *control) GtkRequisition req; (* GTK_WIDGET_CLASS( GTK_OBJECT(m_widget)->klass )->size_request ) (m_widget, &req ); - m_width = req.width; - m_height = req.height; + m_width = req.width + m_xMargin; + m_height = req.height + 2*m_yMargin + 4; m_tools.Append( tool ); @@ -401,26 +400,6 @@ void wxToolBar::ClearTools() bool wxToolBar::Realize() { - m_x = 0; - m_y = 0; - m_width = 100; - m_height = 0; - - wxNode *node = m_tools.First(); - while (node) - { - wxToolBarTool *tool = (wxToolBarTool*)node->Data(); - if (tool->m_bitmap1.Ok()) - { - int tool_height = tool->m_bitmap1.GetHeight(); - if (tool_height > m_height) m_height = tool_height; - } - - node = node->Next(); - } - - m_height += 5 + 2*m_yMargin; - return TRUE; } @@ -536,7 +515,7 @@ void wxToolBar::SetMargins( int x, int y ) { wxCHECK_RET( !m_hasToolAlready, wxT("wxToolBar::SetMargins must be called before adding tool.") ); - if (x > 2) gtk_toolbar_append_space( m_toolbar ); // oh well + if (x > 1) gtk_toolbar_append_space( m_toolbar ); // oh well m_xMargin = x; m_yMargin = y; diff --git a/src/gtk1/tbargtk.cpp b/src/gtk1/tbargtk.cpp index 1e0f1d9f5f..7df75acd3a 100644 --- a/src/gtk1/tbargtk.cpp +++ b/src/gtk1/tbargtk.cpp @@ -178,8 +178,7 @@ bool wxToolBar::Create( wxWindow *parent, wxWindowID id, m_toolbar = GTK_TOOLBAR( gtk_toolbar_new( GTK_ORIENTATION_HORIZONTAL, GTK_TOOLBAR_ICONS ) ); -// gtk_toolbar_set_space_style( m_toolbar, GTK_TOOLBAR_SPACE_LINE ); - m_separation = 5; + m_separation = 7; gtk_toolbar_set_space_size( m_toolbar, m_separation ); m_hasToolAlready = FALSE; @@ -330,8 +329,8 @@ wxToolBarTool *wxToolBar::AddTool( int toolIndex, const wxBitmap& bitmap, GtkRequisition req; (* GTK_WIDGET_CLASS( GTK_OBJECT(m_widget)->klass )->size_request ) (m_widget, &req ); - m_width = req.width; - m_height = req.height; + m_width = req.width + m_xMargin; + m_height = req.height + 2*m_yMargin + 4; gtk_signal_connect( GTK_OBJECT(tool->m_item), "enter_notify_event", @@ -359,8 +358,8 @@ bool wxToolBar::AddControl(wxControl *control) GtkRequisition req; (* GTK_WIDGET_CLASS( GTK_OBJECT(m_widget)->klass )->size_request ) (m_widget, &req ); - m_width = req.width; - m_height = req.height; + m_width = req.width + m_xMargin; + m_height = req.height + 2*m_yMargin + 4; m_tools.Append( tool ); @@ -401,26 +400,6 @@ void wxToolBar::ClearTools() bool wxToolBar::Realize() { - m_x = 0; - m_y = 0; - m_width = 100; - m_height = 0; - - wxNode *node = m_tools.First(); - while (node) - { - wxToolBarTool *tool = (wxToolBarTool*)node->Data(); - if (tool->m_bitmap1.Ok()) - { - int tool_height = tool->m_bitmap1.GetHeight(); - if (tool_height > m_height) m_height = tool_height; - } - - node = node->Next(); - } - - m_height += 5 + 2*m_yMargin; - return TRUE; } @@ -536,7 +515,7 @@ void wxToolBar::SetMargins( int x, int y ) { wxCHECK_RET( !m_hasToolAlready, wxT("wxToolBar::SetMargins must be called before adding tool.") ); - if (x > 2) gtk_toolbar_append_space( m_toolbar ); // oh well + if (x > 1) gtk_toolbar_append_space( m_toolbar ); // oh well m_xMargin = x; m_yMargin = y; diff --git a/src/html/bitmaps/wbkadd.xpm b/src/html/bitmaps/wbkadd.xpm index 2590275e57..6a5c7d07c7 100644 --- a/src/html/bitmaps/wbkadd.xpm +++ b/src/html/bitmaps/wbkadd.xpm @@ -7,16 +7,16 @@ static char * wbkadd_xpm[] = { " ", " ... ", " ... ", -" ...++ ", -" ...++ ", -" ...++ ", +" ...+ ", +" ...+ ", +" ...+ ", " ............. ", " ............. ", -" .............++", -" +++...+++++++", -" +++...+++++++", -" ...++ ", -" ...++ ", -" ...++ ", -" +++ ", -" +++ "}; +" .............+ ", +" +++...++++++ ", +" ...+ ", +" ...+ ", +" ...+ ", +" ...+ ", +" ++ ", +" "}; diff --git a/src/html/bitmaps/wfolder.xpm b/src/html/bitmaps/wfolder.xpm index 228b48a420..b9283ebfb7 100644 --- a/src/html/bitmaps/wfolder.xpm +++ b/src/html/bitmaps/wfolder.xpm @@ -1,50 +1,25 @@ /* XPM */ static char * wfolder_xpm[] = { -"16 16 31 1", -" c None", +/* width height ncolors chars_per_pixel */ +"16 16 3 1", +/* colors */ +" s None c None", ". c #000000", -"+ c #7F6E54", -"@ c #555555", -"# c #7F6140", -"$ c #FFCF94", -"% c #FFFFFF", -"& c #D5D5D5", -"* c #4B4336", -"= c #FFDCA8", -"- c #BFA57E", -"; c #EFEFEF", -"> c #DFDFDF", -", c #B8B8B9", -"' c #6E6E6F", -") c #BF7E42", -"! c #FFA858", -"~ c #FFC280", -"{ c #CFCFCF", -"] c #55402C", -"^ c #3C2C2C", -"/ c #7F542C", -"( c #C0C0C0", -"_ c #B0B0B2", -": c #969698", -"< c #A8A8AB", -"[ c #A0A0A4", -"} c #2C2C2C", -"| c #7C7C7E", -"1 c #161616", -"2 c #3F2A16", -" .+. ", -".@#$+. ", -".%&@#$+.+* ", -".%%%&@#$==-. ", -".%%;>,')!~$+ ", -".%;>{{,']^/~. ", -".;>{{((,,_:]/ ", -".>{{((,,_<[}/ ", -".{{((,,_<[[^/ ", -"._((,,_<[[[}/ ", -" }|_,_<[[[[}/ ", -" .}|<[[[[[}/ ", -" .}|[[[[}/ ", -" .}|[[}/.. ", -" .}|}/.. ", -" .12. "}; +"+ c #ffffff", +/* pixels */ +" ", +" ........... ", +" .+++++++++. ", +" .++..++..+. ", +" ........++++. ", +" .++++++..+.+. ", +" .+.+.++.+.++. ", +" .++++++....+. ", +" .+.+.+++++.+. ", +" .+++++++++.+. ", +" .+.+.+.+.+.+. ", +" .+++++++++.+. ", +" .+.+.+.+.+... ", +" .+++++++++. ", +" ........... ", +" "}; diff --git a/src/html/helpfrm.cpp b/src/html/helpfrm.cpp index 52cc30e279..08bf9f3de7 100644 --- a/src/html/helpfrm.cpp +++ b/src/html/helpfrm.cpp @@ -166,7 +166,7 @@ bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id, const wxString& ti // toolbar? if (style & wxHF_TOOLBAR) { wxToolBar *toolBar = CreateToolBar(wxNO_BORDER | wxTB_HORIZONTAL | wxTB_DOCKABLE); - toolBar -> SetMargins(2, 2); + toolBar->SetMargins( 2, 2 ); toolBar -> AddTool(wxID_HTML_PANEL, wxBITMAP(wpanel), wxNullBitmap, FALSE, -1, -1, (wxObject *) NULL, @@ -188,6 +188,9 @@ bool wxHtmlHelpFrame::Create(wxWindow* parent, wxWindowID id, const wxString& ti m_Bookmarks -> Append(m_BookmarksNames[i]); m_Bookmarks -> SetSelection(0); toolBar -> AddControl(m_Bookmarks); +#ifdef __WXGTK__ + toolBar -> AddSeparator(); +#endif toolBar -> AddTool(wxID_HTML_BOOKMARKSADD, wxBITMAP(wbkadd), wxNullBitmap, FALSE, -1, -1, (wxObject *) NULL, _("Add current page to bookmarks"));