This is a valid issue but the proposal is obviously not correct as the classes are not applied correctlty

This is a valid issue but the proposal is obviously not correct as the classes are not applied correctlty

are you talking about 'controls' class

This is a valid issue but the proposal is obviously not correct as the classes are not applied correctlty

are you talking about 'controls' class

Remove the checkbox from the div with class "contents-label" and create a specific div for it with class ''controls" and then wrap both the divs in one row. Should be alright.

Remove the checkbox from the div with class "contents-label" and create a specific div for it with class ''controls" and then wrap both the divs in one row. Should be alright.

can I use span instead of div

Remove the checkbox from the div with class "contents-label" and create a specific div for it with class ''controls" and then wrap both the divs in one row. Should be alright.

can I use span instead of div

It's better to keep the code as original as possible. Avoid using span

I tried to apply this patch but it said : The patch could not be applied because it conflicts with a previously applied patch: components/com_users/tmpl/login/default_login.php

can you please tell me more about it

Keep it simple with correct HTML structure. Probably you don't need a div nor a span element. The control classes are remains from Bootstrap 2 used in Joomla 3.

Keep it simple with correct HTML structure. Probably you don't need a div nor a span element. The control classes are remains from Bootstrap 2 used in Joomla 3.

Thanks

I think you should create an issue first regarding this. What say? @rjharishabh

I think you should create an issue first regarding this. What say? @rjharishabh

Yes sure, I will create an issue for this.

You don't need to create an issue, you can make a PR directly