Add Exception dialog: make Get Certificate the default button, make buttons accessible

VERIFIED DUPLICATE of bug 399048

Status

Core Graveyard
Security: UI
VERIFIED DUPLICATE of bug 399048
11 years ago
2 years ago

People

(Reporter: Steffen Wilberg, Assigned: Steffen Wilberg)

Tracking

({access})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 284222 [details] [diff] [review]
patch

In the Certificate Manager - Servers - Add Exception dialog:
1. The Get Certificate button should be the default.
2. Pressing enter in the text field should get the cert.
3. Tabbing to the Get Certificate or the View button and pressing enter should work.

This patch does that by
1. adding |dlgtype="extra2"| to the Get Certificate button and adding |defaultButton="extra2"| to the dialog.
2. Unfortunately, pressing enter in the textbox does still not work with that, so I added a onkeypress handler.
3. To make the buttons keyboard accessible, I changed the onclick handlers to oncommand handlers.
Attachment #284222 - Flags: review?(kengert)
(Assignee)

Comment 1

11 years ago
Requesting blocking because the buttons are inaccessible.
Flags: blocking1.9?
Keywords: access

Comment 2

11 years ago
Steffen, thanks a lot for your patch. You probably had not seen we tried to work on this in bug 399048, too. There might be conflicts between the patches in both bugs, we should try to merge them.

I see you are proposing a solution to the "default button" problem, which we have not yet solved in the other bug.

I see you are proposing the extra2 button approach. Will this change the position of the button?
(In reply to comment #2)
> I see you are proposing the extra2 button approach. Will this change the
> position of the button?

It won't. I actually wasn't aware of the code in dialog.xml's _configureButtons that allows you to specify "extra" buttons other than those included in the dialog binding's anonymous content. This patch looks good to me.
(Assignee)

Comment 4

11 years ago
I've seen that bug and it looked like you wanted to concentrate on the accesskeys. The merge conflicts are probably trivial.

The button location won't change, it just has that double-border initially for being the default.

Comment 5

11 years ago
I just merged your changes over to the other patch and tried it, it works great, thanks!!!

I think we should have a single bug, in order to minimize conflicts.
I'll resolve this one as a duplicate and add your changes to the other one.

Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 399048

Comment 6

11 years ago
Comment on attachment 284222 [details] [diff] [review]
patch

removing review request.

I think Johnathan should review it in bug 399048
Attachment #284222 - Flags: review?(kengert)
(Assignee)

Updated

11 years ago
Flags: blocking1.9?
V.Duplicate.
Status: RESOLVED → VERIFIED
No longer depends on: 387480
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.