Modify

Opened 8 years ago

Closed 6 years ago

#4664 closed enhancement (wontfix)

[patch] Add tabindex attribute to get user to username/password fields on login page quickly

Reported by: Samuel Bronson Owned by: Steffen Hoffmann
Priority: low Component: AccountManagerPlugin
Severity: minor Keywords: web_ui
Cc: Samuel Bronson Trac Release: 0.10

Description

I've got a patch, though.

Attachments (1)

trac_login.patch (703 bytes) - added by Samuel Bronson 8 years ago.

Download all attachments as: .zip

Change History (8)

Changed 8 years ago by Samuel Bronson

Attachment: trac_login.patch added

comment:1 Changed 8 years ago by anonymous

Summary: No tabindex property (to get the user to the input fields quickly) is definedpatch: define tabindex attribute (to get the user to the input fields quickly)

comment:2 Changed 8 years ago by anonymous

Cc: Samuel Bronson added; anonymous removed

Sigh. And now the description field looks really silly :-(.

comment:3 Changed 8 years ago by anonymous

Summary: patch: define tabindex attribute (to get the user to the input fields quickly)patch: define tabindex attribute to get the user to the username/password fields on the login page quickly

comment:4 Changed 6 years ago by Steffen Hoffmann

Keywords: needinfo web_ui added; patch removed
Priority: normallow
Severity: normalminor
Summary: patch: define tabindex attribute to get the user to the username/password fields on the login page quickly[patch] Add tabindex attribute to get user to username/password fields on login page quickly
Type: defectenhancement

Anyone here bother to test this with Trac 0.10 anymore?

As I develop for Trac 0.12/0.13 now, I wonder, if this is still needed. Step up, to make me commit this please. I just require someone to test as a pre-requisite. If there is no more interest, I'll close this enhancement request.

comment:5 Changed 6 years ago by Steffen Hoffmann

Owner: changed from Matt Good to Steffen Hoffmann

I'm tracking this ticket among others at a new wiki page dedicated to collaboration for AccountManagerPlugin.

comment:6 Changed 6 years ago by anonymous

It looks unnecessary to me. On my Trac instance/browser at least the cursor starts in username, and tabs immediately to password. I've always preferred changing the layout to adding tab indexes, when possible. Tab indexes (if necessary) then make the rest of tabbing on the page unpredictable.

comment:7 Changed 6 years ago by Steffen Hoffmann

Keywords: needinfo removed
Resolution: wontfix
Status: newclosed

Thank you for the follow-up.

I'll shut this ticket down since it's obviously lacking interest these days to satisfy the effort of integration and testing the included patch. Samuel Bronson's work will still be available, if this is of concern for old Trac installations.

Modify Ticket

Action
as closed The owner will remain Steffen Hoffmann.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.