Skip to content

Conversation

@dgrammatiko
Copy link
Contributor

Executive summary

This PR converts the form validation on com_users to use plain jquery (no mootools call on every form ).
Also NO MORE INLINE SCRIPTS!

Testing

  1. Apply first PR Reduce unneeded calls for mootools #4888 (!important)
  2. Apply this PR
  3. In the admin area go to com_users and try to submit any form.

If no javascript errors are logged in your browser and the functionality remains the same your test is a pass in any other case please report the errors here

Please also check these:
administrator/index.php?option=com_checkin should demonstrate multiselect without mt
administrator/index.php?option=com_users&view=mail should demonstrate form sent and validate without mt
administrator/index.php?option=com_modules should demonstrate multiselect and combobox without mt
http://localhost/administrator/index.php?option=com_admin&view=sysinfo should demonstrate highlighter.js without mt
Logout and log in to demonstrate the use of noframes without mt.

@smanzi
Copy link

smanzi commented Nov 9, 2014

@test success

Tooltips in administrator/index.php?option=com_users&view=mail are moved a little bit too far right...

@dgrammatiko
Copy link
Contributor Author

@smanzi Tooltips render fine with 3.3.7dev version, so I guess it’s already fixed elsewhere 😃

screen shot 2014-11-11 at 5 33 19

@anibalsanchez
Copy link
Contributor

@test success

@roland-d
Copy link
Contributor

Moving to RTC as we have 2 successful tests.

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

@brianteeman brianteeman added the RTC This Pull Request is Ready To Commit label Nov 29, 2014
@Bakual Bakual added this to the Joomla! 3.4.0 milestone Dec 2, 2014
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants