? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
20 Oct 2020

Summary of Changes

Adds missing type attribute to backend login button.

Testing Instructions

Code review or test that you can still login to backend.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 20 Oct 2020
avatar SharkyKZ SharkyKZ - change - 20 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 20 Oct 2020
Category Modules Administration
avatar richard67 richard67 - test_item - 20 Oct 2020 - Tested successfully
avatar richard67
richard67 - comment - 20 Oct 2020

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:

Tip: Always specify the type attribute for the element. Different browsers may use different default types for the element.

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.
avatar chmst chmst - test_item - 20 Oct 2020 - Tested successfully
avatar chmst
chmst - comment - 20 Oct 2020

I have tested this item successfully on 3b79193


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31169.

avatar chmst
chmst - comment - 20 Oct 2020

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 comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31169.

avatar brianteeman
brianteeman - comment - 20 Oct 2020

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

avatar richard67
richard67 - comment - 20 Oct 2020

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?

avatar brianteeman
brianteeman - comment - 20 Oct 2020

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

avatar richard67
richard67 - comment - 20 Oct 2020

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.

avatar Quy Quy - change - 20 Oct 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 20 Oct 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31169.

avatar infograf768 infograf768 - change - 21 Oct 2020
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: ? ?
avatar infograf768 infograf768 - close - 21 Oct 2020
avatar infograf768 infograf768 - merge - 21 Oct 2020
avatar infograf768
infograf768 - comment - 21 Oct 2020

tks

Add a Comment

Login with GitHub to post a comment