Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1127)

Unified Diff: chrome/browser/ui/cocoa/confirm_bubble_view.h

Issue 8697001: Implement confirm bubble for Mac. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/confirm_bubble_view.h
===================================================================
--- chrome/browser/ui/cocoa/confirm_bubble_view.h (revision 0)
+++ chrome/browser/ui/cocoa/confirm_bubble_view.h (revision 0)
@@ -0,0 +1,53 @@
+// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
Robert Sesek 2011/11/28 15:55:39 #if guards
Hironori Bono 2011/12/09 11:14:32 Done.
+#import <Cocoa/Cocoa.h>
+
+#import "base/mac/cocoa_protocols.h"
+#include "base/memory/scoped_nsobject.h"
+
+@class ConfirmBubbleController;
+class ConfirmBubbleModel;
+
+// A view class that implements a bubble consisting of the following items:
+// * one icon ("icon")
+// * one title text ("title")
+// * one message text ("message")
+// * one optional link ("link")
+// * two optional buttons ("ok" and "cancel")
Mark Mentovai 2011/11/28 14:33:06 It has a close button too?
Hironori Bono 2011/12/09 11:14:32 As written below, I have removed the close button
+//
+// This bubble is convenient when we wish to ask transient, non-blocking
+// questions. Unlike a dialog, a bubble menu disappears when we click outside of
+// its window to avoid blocking user operations. A bubble is laid out as
+// follows:
+//
+// +------------------------+
+// | icon title x |
Robert Sesek 2011/11/28 15:55:39 It's a little odd to have the close button on the
Hironori Bono 2011/12/09 11:14:32 Thanks for noticing it. This change straightforwar
+// | message |
+// | link |
+// | [OK] [Cancel] |
Mark Mentovai 2011/11/28 14:33:06 Normal Mac button ordering in dialogs is to put th
Hironori Bono 2011/12/09 11:14:32 Done.
+// +------------------------+
+//
+@interface ConfirmBubbleView : NSView<NSTextViewDelegate> {
+ @private
+ NSView* parent_; // weak
+ ConfirmBubbleController* controller_; // weak
+
+ // Controls used in this bubble.
+ scoped_nsobject<NSImageView> icon_;
+ scoped_nsobject<NSTextView> titleLabel_;
+ scoped_nsobject<NSButton> closeButton_;
+ scoped_nsobject<NSTextView> messageLabel_;
+ scoped_nsobject<NSButton> okButton_;
+ scoped_nsobject<NSButton> cancelButton_;
+}
+
+// Initializes a bubble view and sets it to the first responder of |parent|.
+// Since this initializer programmatically creates a custom NSView (i.e. without
+// using a nib file), this function should be called from loadView: of the
+// controller object which owns this view.
+- (id)initWithParent:(NSView*)parent
+ controller:(ConfirmBubbleController*)controller;
+
+@end
Property changes on: chrome/browser/ui/cocoa/confirm_bubble_view.h
___________________________________________________________________
Added: svn:eol-style
+ LF

Powered by Google App Engine
This is Rietveld 408576698