Remove `noreferrer` from Site Link and Site Meta List blocks.

WordPress / wporg-showcase-2022

The official theme of the WordPress.org showcase.

https://wordpress.org/showcase/

20 stars 5 forks source link

Remove `noreferrer` from Site Link and Site Meta List blocks. #251

Closed ndiego closed 11 months ago

ndiego commented 11 months ago

Fixes #250

Removed noreferrer from the link in the Site Link and Site Meta List blocks per feedback in #234

renintw commented 11 months ago

I guess the original decision was based on a practice suggesting that using "noopener noreferrer" together is a safe approach. That's probably why it was implemented as such. It seems like "noreferrer" could be safely removed here in the showcase, as I haven't seen URLs containing sensitive user information.

ndiego commented 11 months ago

LGTM, seems like we'd want to remove it in the site meta list URL entry too?

Ah, good catch. Just updated the PR to include that.