[haiku-commits] haiku: hrev45311 - in src/apps/webpositive/tabview: . src/apps/webpositive

  • From: jscipione@xxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Thu, 21 Feb 2013 03:48:17 +0100 (CET)

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);


Other related posts:

  • » [haiku-commits] haiku: hrev45311 - in src/apps/webpositive/tabview: . src/apps/webpositive - jscipione