hrev45311 adds 1 changeset to branch 'master' old head: 52266c1de7bf47c813ed30eaa10e22ba367cfca4 new head: fff103f567777567fafabf5b7c3f6c252642f80d overview: http://cgit.haiku-os.org/haiku/log/?qt=range&q=fff103f+%5E52266c1 ---------------------------------------------------------------------------- fff103f: WebPositive close x fixes. * Remove the Close tab and Close find bar tooltips. * Offset the x again. * Add some comments. [ John Scipione <jscipione@xxxxxxxxx> ] ---------------------------------------------------------------------------- Revision: hrev45311 Commit: fff103f567777567fafabf5b7c3f6c252642f80d URL: http://cgit.haiku-os.org/haiku/commit/?id=fff103f Author: John Scipione <jscipione@xxxxxxxxx> Date: Thu Feb 21 02:46:04 2013 UTC ---------------------------------------------------------------------------- 2 files changed, 8 insertions(+), 9 deletions(-) src/apps/webpositive/BrowserWindow.cpp | 3 +-- src/apps/webpositive/tabview/TabManager.cpp | 14 +++++++------- ---------------------------------------------------------------------------- diff --git a/src/apps/webpositive/BrowserWindow.cpp b/src/apps/webpositive/BrowserWindow.cpp index e266768..ab923e7 100644 --- a/src/apps/webpositive/BrowserWindow.cpp +++ b/src/apps/webpositive/BrowserWindow.cpp @@ -271,6 +271,7 @@ public: | BControlLook::B_BLEND_FRAME); be_control_look->DrawButtonBackground(this, frame, updateRect, base, BControlLook::B_ACTIVATED); + closeRect.OffsetBy(1, 1); tint *= 1.2; } else { SetHighColor(base); @@ -502,8 +503,6 @@ BrowserWindow::BrowserWindow(BRect frame, SettingsMessage* appSettings, // Find group fFindCloseButton = new CloseButton(new BMessage(EDIT_HIDE_FIND_GROUP)); - fFindCloseButton->SetToolTip(B_TRANSLATE_COMMENT("Close find bar", - "find bar close button tooltip")); fFindTextControl = new BTextControl("find", B_TRANSLATE("Find:"), "", new BMessage(EDIT_FIND_NEXT)); fFindTextControl->SetModificationMessage(new BMessage(FIND_TEXT_CHANGED)); diff --git a/src/apps/webpositive/tabview/TabManager.cpp b/src/apps/webpositive/tabview/TabManager.cpp index b94f0b5..29368f5 100644 --- a/src/apps/webpositive/tabview/TabManager.cpp +++ b/src/apps/webpositive/tabview/TabManager.cpp @@ -584,13 +584,10 @@ WebTabView::MouseMoved(BPoint where, uint32 transit, fOverCloseRect = overCloseRect; ContainerView()->Invalidate(closeRect); } - // Set the Tooltip - if (overCloseRect) - fController->SetToolTip(B_TRANSLATE("Close tab")); - else - fController->SetToolTip(Label()); - } else - fController->SetToolTip(Label()); + } + + // Set the tool tip + fController->SetToolTip(Label()); TabView::MouseMoved(where, transit, dragMessage); } @@ -640,15 +637,18 @@ void WebTabView::_DrawCloseButton(BView* owner, BRect& frame, tint *= 1.2; if (fClicked && fOverCloseRect) { + // Draw the button frame BRect buttonRect(closeRect.InsetByCopy(-4, -4)); be_control_look->DrawButtonFrame(owner, buttonRect, updateRect, base, base, BControlLook::B_ACTIVATED | BControlLook::B_BLEND_FRAME); be_control_look->DrawButtonBackground(owner, buttonRect, updateRect, base, BControlLook::B_ACTIVATED); + closeRect.OffsetBy(1, 1); tint *= 1.2; } + // Draw the × base = tint_color(base, tint); owner->SetHighColor(base); owner->SetPenSize(2);