Modify

Opened 15 years ago

Closed 13 years ago

#4664 closed enhancement (wontfix)

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

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

Description

I've got a patch, though.

Attachments (1)

trac_login.patch (703 bytes) - added by SamB 15 years ago.

Download all attachments as: .zip

Change History (8)

Changed 15 years ago by SamB

Attachment: trac_login.patch added

comment:1 Changed 15 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 15 years ago by anonymous

Cc: SamB added; anonymous removed

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

comment:3 Changed 15 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 13 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 13 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 13 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 13 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

Change Properties
Set your email in Preferences
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.