Modify

Opened 6 years ago

Closed 4 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: hasienda
Priority: low Component: AccountManagerPlugin
Severity: minor Keywords: web_ui
Cc: naesten@… Trac Release: 0.10

Description

I've got a patch, though.

Attachments (1)

trac_login.patch (703 bytes) - added by naesten@… 6 years ago.

Download all attachments as: .zip

Change History (8)

Changed 6 years ago by naesten@…

comment:1 Changed 6 years ago by anonymous

  • Summary changed from No tabindex property (to get the user to the input fields quickly) is defined to patch: define tabindex attribute (to get the user to the input fields quickly)

comment:2 Changed 6 years ago by anonymous

  • Cc naesten@… added; anonymous removed

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

comment:3 Changed 6 years ago by anonymous

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

comment:4 Changed 4 years ago by hasienda

  • Keywords needinfo web_ui added; patch removed
  • Priority changed from normal to low
  • Severity changed from normal to minor
  • Summary changed from patch: define tabindex attribute to get the user to the username/password fields on the login page quickly to [patch] Add tabindex attribute to get user to username/password fields on login page quickly
  • Type changed from defect to enhancement

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 4 years ago by hasienda

  • Owner changed from mgood to hasienda

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

comment:6 Changed 4 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 4 years ago by hasienda

  • Keywords needinfo removed
  • Resolution set to wontfix
  • Status changed from new to closed

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.

Add Comment

Modify Ticket

Action
as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.