Closed
Bug 286197
Opened 20 years ago
Closed 20 years ago
preferences should not close upon hitting enter on Mac OS X
Categories
(Toolkit :: Preferences, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla1.8beta4
People
(Reporter: jaas, Assigned: asaf)
References
Details
(Whiteboard: [no l10n impact][have patch])
Attachments
(1 file, 3 obsolete files)
738 bytes,
patch
|
mconnor
:
first-review+
mconnor
:
approval1.8b4+
|
Details | Diff | Splinter Review |
1) Open preferences
2) type in a new home page URL and hit enter
Preferences close upon hitting enter in step #2. This is not standard behavior
on Mac OS X and shouldn't happen.
Assignee | ||
Comment 1•20 years ago
|
||
(not like hidden, the disabled property/attribute doesn't affect hbox's
children.)
Assignee | ||
Updated•20 years ago
|
Priority: -- → P2
Target Milestone: --- → Firefox1.1
Assignee | ||
Comment 2•20 years ago
|
||
Comment on attachment 177736 [details] [diff] [review]
patch v1 - disable hidden buttons
hm, this breaks the window close button.
Attachment #177736 -
Attachment is obsolete: true
Attachment #177736 -
Flags: review?(mconnor)
Assignee | ||
Comment 3•20 years ago
|
||
Attachment #177841 -
Flags: review?(mconnor)
Assignee | ||
Updated•20 years ago
|
Attachment #177841 -
Attachment is obsolete: true
Attachment #177841 -
Flags: review?(mconnor)
Comment 4•20 years ago
|
||
Comment on attachment 177841 [details] [diff] [review]
v2
Let's do this by not specifying a default button on Mac (really, specifying
none). Depends on some other, necessary stuff.
Attachment #177841 -
Attachment is obsolete: false
Attachment #177841 -
Flags: review-
Assignee | ||
Comment 5•20 years ago
|
||
-> Toolkit/prefs
Flags: review-
Product: Firefox → Toolkit
Target Milestone: Firefox1.1 → ---
Version: Trunk → unspecified
Assignee | ||
Updated•20 years ago
|
Target Milestone: --- → mozilla1.8beta2
Assignee | ||
Comment 6•20 years ago
|
||
Of course, it needs the patch from the dialog.xml bug.
Attachment #177841 -
Attachment is obsolete: true
Attachment #178656 -
Flags: first-review?(mconnor)
Assignee | ||
Comment 7•20 years ago
|
||
Comment on attachment 178656 [details] [diff] [review]
the preferences part
The second default button patch well handles this case.
Attachment #178656 -
Attachment is obsolete: true
Attachment #178656 -
Flags: first-review?(mconnor)
Assignee | ||
Updated•20 years ago
|
Severity: normal → enhancement
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
OS: MacOS X → All
Priority: P2 → P1
Resolution: --- → FIXED
Assignee | ||
Comment 8•20 years ago
|
||
err, toolkit/prefs again.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Updated•20 years ago
|
Status: REOPENED → ASSIGNED
Assignee | ||
Updated•20 years ago
|
Priority: P1 → P2
Target Milestone: mozilla1.8beta2 → mozilla1.8beta3
Assignee | ||
Updated•20 years ago
|
Severity: enhancement → minor
OS: All → MacOS X
Target Milestone: mozilla1.8beta3 → mozilla1.8beta4
Assignee | ||
Comment 9•20 years ago
|
||
simply disable the accept button
Attachment #189372 -
Flags: first-review?(mconnor)
Assignee | ||
Updated•20 years ago
|
Flags: blocking1.8b4?
Whiteboard: [no l10n impact][have patch]
Updated•20 years ago
|
Flags: blocking1.8b4? → blocking1.8b4+
Updated•20 years ago
|
Attachment #189372 -
Flags: first-review?(mconnor)
Attachment #189372 -
Flags: first-review+
Attachment #189372 -
Flags: approval1.8b4+
Assignee | ||
Comment 10•20 years ago
|
||
Checking in preferences.xml;
/cvsroot/mozilla/toolkit/content/widgets/preferences.xml,v <-- preferences.xml
new revision: 1.24; previous revision: 1.23
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
Updated•18 years ago
|
Flags: in-testsuite?
Assignee | ||
Updated•18 years ago
|
QA Contact: mconnor → preferences
You need to log in
before you can comment on or make changes to this bug.
Description
•