? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
11 May 2021

Pull Request for Issue #33779

Summary of Changes

Joomla 4 has the concept of Extra Buttons for the login page.

Joomla 4 also forgot to add this concept to the offline.php page for the default template.

Testing Instructions

  1. Activate Webauthn (you need to be on https://)
  2. Set the website into offline mode
  3. Try to login with Webauthn on FRONTEND

Actual result BEFORE applying this Pull Request

No extra buttons

Expected result AFTER applying this Pull Request

Extra buttons that work

Screenshot 2021-05-11 at 19 35 28

Errors in the process also render correctly

Screenshot 2021-05-11 at 19 39 56

Documentation Changes Required

None.

avatar PhilETaylor PhilETaylor - open - 11 May 2021
avatar PhilETaylor PhilETaylor - change - 11 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 May 2021
Category Front End Templates (site)
avatar PhilETaylor PhilETaylor - change - 11 May 2021
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 11 May 2021
Category Front End Templates (site) Front End Templates (site) NPM Change
avatar PhilETaylor PhilETaylor - change - 11 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 11 May 2021
avatar PhilETaylor PhilETaylor - change - 11 May 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 11 May 2021
avatar richard67
richard67 - comment - 11 May 2021

Can issue #33779 be closed with this PR here? Or should it stay open because there is more to do?

avatar PhilETaylor
PhilETaylor - comment - 11 May 2021

I dont believe there is anymore to be done to satisfy the issue, which can now be closed.

This PR implements ALL the extra buttons, not just webauthn.

avatar HLeithner
HLeithner - comment - 12 May 2021

Offline.PHP is really a pain... Applied the patch and didn't saw any button until I found out that the page used a custom template ;)

Copy pasted the or into the other offline.php and it worked (style missing but that's a different story)

avatar richard67
richard67 - comment - 12 May 2021

Commandment 11:

Thou shalt not use custom templates.

avatar PhilETaylor
PhilETaylor - comment - 12 May 2021

So that is a successful Test then yes? haha 

We are only responsible for the core templates, obviously, if you have custom templates then you will need this additional code to support the extra buttons feature :-) doh

avatar dgrammatiko
dgrammatiko - comment - 12 May 2021

Commandment 11:

Thou shalt not use a custom template that is not supporting child templates in Joomla 4

avatar Quy Quy - test_item - 22 May 2021 - Tested successfully
avatar Quy
Quy - comment - 22 May 2021

I have tested this item successfully on defd2a7


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

avatar sandramay0905 sandramay0905 - test_item - 23 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 23 May 2021

I have tested this item successfully on defd2a7


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

avatar richard67 richard67 - change - 23 May 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 23 May 2021

RTC


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

avatar HLeithner HLeithner - change - 24 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-24 12:37:39
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 24 May 2021
avatar HLeithner HLeithner - merge - 24 May 2021
avatar HLeithner
HLeithner - comment - 24 May 2021

Thanks

Add a Comment

Login with GitHub to post a comment