Closed Bug 286197 Opened 19 years ago Closed 19 years ago

preferences should not close upon hitting enter on Mac OS X

Categories

(Toolkit :: Preferences, defect, P2)

PowerPC
macOS
defect

Tracking

()

RESOLVED FIXED
mozilla1.8beta4

People

(Reporter: jaas, Assigned: asaf)

References

Details

(Whiteboard: [no l10n impact][have patch])

Attachments

(1 file, 3 obsolete files)

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.
(not like hidden, the disabled property/attribute doesn't affect hbox's
children.)
Assignee: joshmoz → bugs.mano
Status: NEW → ASSIGNED
Attachment #177736 - Flags: review?(mconnor)
Priority: -- → P2
Target Milestone: --- → Firefox1.1
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)
Attached patch v2 (obsolete) — Splinter Review
Attachment #177841 - Flags: review?(mconnor)
Depends on: 274712
Depends on: 284776
Attachment #177841 - Attachment is obsolete: true
Attachment #177841 - Flags: review?(mconnor)
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-
-> Toolkit/prefs
Flags: review-
Product: Firefox → Toolkit
Target Milestone: Firefox1.1 → ---
Version: Trunk → unspecified
Target Milestone: --- → mozilla1.8beta2
Attached patch the preferences part (obsolete) — Splinter Review
Of course, it needs the patch from the dialog.xml bug.
Attachment #177841 - Attachment is obsolete: true
Attachment #178656 - Flags: first-review?(mconnor)
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)
Severity: normal → enhancement
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
OS: MacOS X → All
Priority: P2 → P1
Resolution: --- → FIXED
err, toolkit/prefs again.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Status: REOPENED → ASSIGNED
Priority: P1 → P2
Target Milestone: mozilla1.8beta2 → mozilla1.8beta3
Severity: enhancement → minor
OS: All → MacOS X
Target Milestone: mozilla1.8beta3 → mozilla1.8beta4
Attached patch back to plan aSplinter Review
simply disable the accept button
Attachment #189372 - Flags: first-review?(mconnor)
Flags: blocking1.8b4?
Whiteboard: [no l10n impact][have patch]
Flags: blocking1.8b4? → blocking1.8b4+
Attachment #189372 - Flags: first-review?(mconnor)
Attachment #189372 - Flags: first-review+
Attachment #189372 - Flags: approval1.8b4+
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: 19 years ago19 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
QA Contact: mconnor → preferences
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: