Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

fix(list): add rel to copied attributes for md-list-item buttons … #10352

Merged
merged 1 commit into from
May 9, 2017

Conversation

pr4v33n
Copy link
Contributor

@pr4v33n pr4v33n commented Feb 2, 2017

#10351

This is to add support for rel attribute to be copied to generated links.
See https://developers.google.com/web/tools/lighthouse/audits/noopener for more details.

…ngular#10351

This is to add support for rel attribute to be copied to generated links.
@googlebot googlebot added the cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ label Feb 2, 2017
@devversion
Copy link
Member

Thanks for the PR. I think we should add a test for this new change?

@ThomasBurleson
Copy link
Contributor

@pr4v33n - please add at least 2 unit tests for this change.

@ThomasBurleson ThomasBurleson added this to the 1.1.4 milestone Mar 7, 2017
@ThomasBurleson ThomasBurleson added the needs: unit tests This PR needs unit tests to cover the changes being proposed label Mar 7, 2017
@ThomasBurleson ThomasBurleson modified the milestones: 1.1.4, 1.1.5 May 7, 2017
@ThomasBurleson ThomasBurleson added needs: presubmit and removed needs: unit tests This PR needs unit tests to cover the changes being proposed labels May 8, 2017
@ThomasBurleson ThomasBurleson self-assigned this May 8, 2017
@kara kara added pr: merge ready This PR is ready for a caretaker to review and removed needs: presubmit labels May 9, 2017
@kara kara merged commit 241bbc4 into angular:master May 9, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes PR author has signed Google's CLA: https://opensource.google.com/docs/cla/ pr: merge ready This PR is ready for a caretaker to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants