? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
17 Apr 2019

Pull Request for Issue joomla/backend-template#112 .

Summary of Changes

(Remove Tag Fieldset. In the login form, there is no need to group the input fields.)
Add label to fieldset.

Testing Instructions

Code Review

avatar chmst chmst - open - 17 Apr 2019
avatar chmst chmst - change - 17 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Apr 2019
Category Modules Administration
avatar brianteeman
brianteeman - comment - 17 Apr 2019

Why is there no need for this to be a fieldset?
A fieldset should be used when you need to show a relationship between multiple form inputs.
Surely there is a relationship between username and password ?

avatar franz-wohlkoenig franz-wohlkoenig - change - 17 Apr 2019
Title
[4.0][a11y] Remove superfluous fieldset
[4.0] [a11y] Remove superfluous fieldset
avatar franz-wohlkoenig franz-wohlkoenig - edited - 17 Apr 2019
avatar chmst
chmst - comment - 17 Apr 2019

It is a single form on a singele login screen. See also the linked issue and discussion

The frontend login screen has no fieldset. There it would make more sense as it can occur often and everywhere.

avatar zwiastunsw
zwiastunsw - comment - 17 Apr 2019

I'm against it. There is no point in saying that something doesn't make sense. Sometimes we don't see it, but it doesn't mean it doesn't make sense. The information about the group of fields is important for the users of assistive technologies.
The screen reader informs the user that it is dealing with a group of fields and reads its legend.

avatar brianteeman
brianteeman - comment - 17 Apr 2019

Just because its a single form doesnt mean that it is not a fieldset

avatar chmst chmst - change - 17 Apr 2019
Labels Added: ? ?
avatar chmst chmst - edited - 17 Apr 2019
avatar chmst chmst - change - 17 Apr 2019
The description was changed
Title
[4.0] [a11y] Remove superfluous fieldset
[4.0] [a11y] Label for fieldset in admin login
avatar chmst chmst - edited - 17 Apr 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 18 Apr 2019
Title
[4.0] [a11y] Label for fieldset in admin login
[4.0] Label for fieldset in admin login
avatar franz-wohlkoenig franz-wohlkoenig - edited - 18 Apr 2019
avatar wilsonge
wilsonge - comment - 4 May 2019

Closing as a11y team lead is against

avatar wilsonge wilsonge - close - 4 May 2019
avatar wilsonge wilsonge - change - 4 May 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-05-04 15:03:01
Closed_By wilsonge
Labels Added: ?
Removed: ?

Add a Comment

Login with GitHub to post a comment