User tests: Successful: Unsuccessful:
Adds missing type
attribute to backend login button.
Code review or test that you can still login to backend.
No.
Status | New | ⇒ | Pending |
Category | ⇒ | Modules Administration |
I have tested this item
I had the problems as reported in #31165 but not always, and cannot say when it occurred. So I can confirm that this is the right attribute and the problem does not occur with this change.
This should not be required as any button inside a form has the default value of "type=submit"
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button
This should not be required as any button inside a form has the default value of "type=submit"
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/button
@brianteeman But does it do harm to specify the type attribute?
No - but my point is that if everyone is saying this is an "intermittent" problem then I doubt very much that adding the default value for the button is actually resolving the problem
No - but my point is that if everyone is saying this is an "intermittent" problem then I doubt very much that adding the default value for the button is actually resolving the problem
Agree.
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-10-21 06:37:02 |
Closed_By | ⇒ | infograf768 | |
Labels |
Added:
?
?
|
tks
I have tested this item✅ successfully on 3b79193
Code review: The PR adds the right type attribute to the button. See https://www.w3schools.com/tags/att_button_type.asp:
The "submit" type is the right one.
Real test: As I did not have problems with login like they were reported in issue #31165 , I could only check that login still works as well as before and that the type attribute really has been added.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31169.