WordPress.org

Make WordPress Core


Ignore:
Timestamp:
10/19/2020 11:37:53 PM (12 months ago)
Author:
SergeyBiryukov
Message:

General: Remove noreferrer from wp_targeted_link_rel() and other uses.

When noopener noreferrer was originally added in #37941 and related tickets, the noreferrer bit was specifically included due to Firefox not supporting noopener at the time.

Since noopener has been supported by all major browsers for a while, it should now be safe to remove the noreferrer attribute from core.

Props Mista-Flo, audrasjb, joostdevalk, jonoaldersonwp, peterwilsoncc, elgameel.
Fixes #49558.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/menu/walker-nav-menu.php

    r46586 r49215  
    3838
    3939    /**
    40      * Tests when an item's target is _blank, that rel="noopener noreferrer" is added.
     40     * Tests when an item's target is _blank, that rel="noopener" is added.
    4141     *
    4242     * @ticket 43290
     
    6565        $this->walker->start_el( $expected, (object) $item, 0, (object) $args );
    6666
    67         $this->assertSame( "<li id=\"menu-item-{$post_id}\" class=\"menu-item-{$post_id}\"><a target=\"_blank\" rel=\"noopener noreferrer\">{$post_title}</a>", $expected );
     67        $this->assertSame( "<li id=\"menu-item-{$post_id}\" class=\"menu-item-{$post_id}\"><a target=\"_blank\" rel=\"noopener\">{$post_title}</a>", $expected );
    6868    }
    6969
Note: See TracChangeset for help on using the changeset viewer.