User tests: Successful: Unsuccessful:
Load external script with async/defer (as invisble reCaptcha already does):
Status | New | ⇒ | Pending |
Category | ⇒ | Front End Plugins |
https://www.w3.org/TR/2011/WD-html5-20110525/scripting-1.html#attr-script-async
The defer attribute may be specified even if the async attribute is specified, to cause legacy Web browsers that only support defer (and not async) to fall back to the defer behavior instead of the synchronous blocking behavior that is the default.
Labels |
Added:
?
|
@acs-ferreira Last I heard, async
has a higher priority than defer
so it would always be used instead. Then defer
would be used if the browser didn't support async
Can you please rebase this PR on j4 branch.
When you want to validate an ajax form with a 3rd-party extension, recaptcha is not loaded at all.
Defers activation of the recaptcha widget to the behavior should resolve that. See:
https://www.drupal.org/project/recaptcha/issues/2493183
Can you please rebase this PR on j4 branch.
@HLeithner may I do a separate PR for J4?
Can you please rebase this PR on j4 branch.
@HLeithner may I do a separate PR for J4?
yes a new pr would work better for you
Pretty sure that the reason for asking if it could be a separate PR was so that this one could also be merged in j3
oh, if this is the case no, because commits to j3 get merged to j4 by george at the moment.
which is why I didnt understand the request to rebase
@brianteeman @HLeithner is a separate PR needed?
I missed that this is a bug fix and not a "feature". @hardik-codes forget the rebase or another pr we only need 2 successful tests and I can merge it.
Ok
Thanks for the info
@acs-ferreira can you please give Test Instructions in first Comment? Maybe we get then Tests.
@acs-ferreira can you please give Test Instructions in first Comment? Maybe we get then Tests.
closed due the lack of response.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-07-20 08:04:39 |
Closed_By | ⇒ | franz-wohlkoenig | |
Labels |
Added:
?
|
defer and async?
Are you sure you know what these attributes do?