Buttons not disabled when they should, if toolbar buttons are text-only

RESOLVED FIXED in Camino1.5

Status

Camino Graveyard
Toolbars & Menus
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: Håkan Waara, Assigned: Stuart Morgan)

Tracking

({fixed1.8.1.1})

Trunk
Camino1.5
PowerPC
Mac OS X
fixed1.8.1.1

Details

(Whiteboard: [good first bug])

Attachments

(1 attachment, 1 obsolete attachment)

1020 bytes, patch
Mike Pinkerton (not reading bugmail)
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

12 years ago
Camino 1.0

Steps to reproduce:

1. Open a new window
2. Ctrl-click the toolbar, and choose "Text only" for the buttons.

You'll notice that Back/Forward are clickable and enabled in the toolbar, even if you have not used this window before. This does not happen in the toolbar button as images-mode.
See also bug 316835, which tracks improperly disabled buttons as buttons.
Blocks: 316835
Target Milestone: --- → Camino1.1
Blocks: 341853
No longer blocks: 316835

Comment 2

11 years ago
I'll look into this.
Assignee: mikepinkerton → bugzilla
Whiteboard: [good first bug]
(Assignee)

Comment 3

11 years ago
Created attachment 242998 [details] [diff] [review]
fix

Why NSToolbarItem doesn't do this by default in the custom view case is beyond me.
Assignee: bugzilla → stuart.morgan
Status: NEW → ASSIGNED
Attachment #242998 - Flags: review?

Comment 4

11 years ago
Comment on attachment 242998 [details] [diff] [review]
fix

+// Make sure that them menu form, which is used for the text-only view,

r=me with the comment cleaned up
Attachment #242998 - Flags: review? → review+
(Assignee)

Comment 5

11 years ago
Created attachment 243000 [details] [diff] [review]
fixed fix

With gooder English
Attachment #242998 - Attachment is obsolete: true
Attachment #243000 - Flags: superreview?(mikepinkerton)
(Reporter)

Comment 6

11 years ago
(In reply to comment #3)
> Created an attachment (id=242998) [edit]
> fix
> 
> Why NSToolbarItem doesn't do this by default in the custom view case is beyond
> me.

File an apple bug?
(Assignee)

Comment 7

11 years ago
The docs mention that enabling of the menu item rep is separate for custom-view items, so they already know. I just don't know why they think it's desirable.
Comment on attachment 243000 [details] [diff] [review]
fixed fix

sr=pink
Attachment #243000 - Flags: superreview?(mikepinkerton) → superreview+
Whiteboard: [good first bug] → [good first bug][needs checkin]

Comment 9

11 years ago
Checked in on 1.8branch and trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Keywords: fixed1.8.1
Resolution: --- → FIXED
Whiteboard: [good first bug][needs checkin] → [good first bug]

Updated

11 years ago
Keywords: fixed1.8.1.1

Updated

11 years ago
Keywords: fixed1.8.1
You need to log in before you can comment on or make changes to this bug.