-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Privacy notice link opening in new tab changes #4229
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed to opens in new tab
changed to 'opens in new tab'
Review app deployed to https://get-into-teaching-app-review-4229.test.teacherservices.cloud |
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 LGTM (from tech perspective)
Trello card
https://trello.com/c/z1slwQj4
Context
On the ‘Get Into Teaching’ page, there is a link that opens in a new tab, but which neither visually nor programmatically forewarns users that this will be the case.
Changes proposed in this pull request
Added
(opens in a new tab)
to all privacy notice links - this is the only internal link that opens in a new tab across the site