feat: add pagespeed insights link

langswei commented 4 weeks ago

Add link to PageSpeed Insights in URL facet

Description

Added a link to PageSpeed Insights for each given URL in the URL facet.

Related Issue

Motivation and Context

Meant to give a clue to users how to diagnose issues and make it faster to do so even if they don't need the clue.

How Has This Been Tested?

https://pagespeed--helix-website--adobe.aem.live/tools/rum/explorer.html?domain=emigrationbrewing.com&filter=&view=month&domainkey=open&conversion.checkpoint=click

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

aem-code-sync[bot] commented 4 weeks ago

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed. In case there are problems, just click a checkbox below to rerun the respective action.

[ ] Re-run PSI checks
[ ] Re-sync branch

Commits

* [bc54c97](https://github.com/adobe/helix-website/commit/bc54c975d2e491bbceec9529917866163a7946e0) :white_check_mark: (latest) * [ba0e051](https://github.com/adobe/helix-website/commit/ba0e05104d402257ba60bcc8b8ab4a6132382967) :white_check_mark: * [a582e14](https://github.com/adobe/helix-website/commit/a582e1461f80c0308ca93b06af96600fa350fb9a) :white_check_mark:

aem-code-sync[bot] commented 4 weeks ago

Page	Scores	Audits	Google
/tools/rum/explorer.html?domain=emigrationbrewing.com&filter=&view=month&domaink ey=open&conversion.checkpoint=click

trieloff commented 4 weeks ago

The page speed icon should not be an img – we can add that in CSS, maybe with something like this https://codepen.io/trieloff-the-flexboxer/pen/qBGXjxx

langswei commented 3 weeks ago

The page speed icon should not be an img – we can add that in CSS, maybe with something like this https://codepen.io/trieloff-the-flexboxer/pen/qBGXjxx

I probably missed your point on this -- I had only read the initial comment which showed up in email, not the edited one here which is more pure CSS. Anyways have a look and advise if I should abandon that one.

https://github.com/adobe/helix-website/pull/555

adobe / helix-website