? ? Pending

User tests: Successful: Unsuccessful:

avatar hans2103
hans2103
21 Nov 2020

Pull Request for Issue #29722 .

Summary of Changes

This PR will add inline styling for the Google invisible reCaptcha v2 and prevents it from hiding behind other elements by adding a huge z-index.

Testing Instructions

  • follow the instructions as mentioned on #29722
  • apply this patch
  • no NPM needed
  • repeat the instructions

Actual result BEFORE applying this Pull Request

Google invisible reCaptcha v2 badge is hiding behind other blocks

Expected result AFTER applying this Pull Request

Google invisible reCaptcha v2 badge is in front of other blocks

Documentation Changes Required

avatar hans2103 hans2103 - open - 21 Nov 2020
avatar hans2103 hans2103 - change - 21 Nov 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Nov 2020
Category Front End Plugins
avatar brianteeman
brianteeman - comment - 21 Nov 2020

can it please be done without using an inline style

avatar hans2103
hans2103 - comment - 21 Nov 2020

@brianteeman I was not able to find the exact location of the inline styles on the element. And loading an extra css file when this plugin is enabled does not seem to be a better solution then using the proposed style element.

avatar ceford ceford - test_item - 22 Nov 2020 - Tested successfully
avatar ceford
ceford - comment - 22 Nov 2020

I have tested this item successfully on 5bf6e64

It works as described, although getting a V2 Key is a pain. It seems to me the obvious place to put the style is in template.css. Will other template developers want to use a different z-index?


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

avatar brianteeman
brianteeman - comment - 22 Nov 2020

@hans2103 the aim is to try and remove inline styles - not easy - but we certainly should not be adding any. This is to allow stricter CSP policies for security.

cc @zero-24 @wilsonge

avatar gostn gostn - test_item - 25 Nov 2020 - Tested successfully
avatar gostn
gostn - comment - 25 Nov 2020

I have tested this item successfully on 5bf6e64


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

avatar Quy
Quy - comment - 25 Nov 2020

@hans2103 You can see how it is done in PR #31098.

avatar hans2103
hans2103 - comment - 26 Nov 2020

@Quy I know how it is done. The reason I added is using inline styling was to prevent an extra httpRequest.

avatar hans2103 hans2103 - change - 26 Nov 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 26 Nov 2020
Category Front End Plugins Repository NPM Change Front End Plugins
avatar richard67
richard67 - comment - 26 Nov 2020

@ceford @gostn This PR has received changes. Could you test again? Thanks in advance.

avatar ceford ceford - test_item - 26 Nov 2020 - Tested successfully
avatar ceford
ceford - comment - 26 Nov 2020

I have tested this item successfully on 39fa0ea

It still works. Do we have any information on the extra time needed to fetch a style sheet containing one line?


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

avatar Fedik
Fedik - comment - 26 Nov 2020

The reason I added is using inline styling was to prevent an extra httpRequest.

You can add it to existing template css, no need inline css or separated css file.
It is template styling issue, not reCaptcha issue

avatar gostn gostn - test_item - 26 Nov 2020 - Tested successfully
avatar gostn
gostn - comment - 26 Nov 2020

I have tested this item successfully on 39fa0ea


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

avatar richard67 richard67 - change - 26 Nov 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 26 Nov 2020

RTC


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

avatar laoneo laoneo - change - 30 Nov 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-11-30 15:11:12
Closed_By laoneo
Labels Added: ?
avatar laoneo laoneo - close - 30 Nov 2020
avatar laoneo laoneo - merge - 30 Nov 2020
avatar laoneo
laoneo - comment - 30 Nov 2020

Thanks!

Add a Comment

Login with GitHub to post a comment