Skip to content
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

add target="_blank" to match all other links on page #3154

Merged
merged 9 commits into from
May 3, 2019

Conversation

SeanPM5
Copy link
Contributor

@SeanPM5 SeanPM5 commented May 3, 2019

This ensures that link opens in a new window rather than taking over the current one. All other links on this page have this attribute. See #3153 for more info.

This is my first ever PR here and I am inexperienced at Github so apologies in advance if I didn't do this correctly.

@thomasloven
Copy link
Contributor

The last three are hidden away in src/panels/config/js/automation.js.

@SeanPM5
Copy link
Contributor Author

SeanPM5 commented May 3, 2019

Thanks @thomasloven was having trouble finding those last three.

@thomasloven
Copy link
Contributor

Next hint: Travis fails due to formating errors. If you click "Details" to the right of the red cross below 👇 , you'll get its output. The important stuff starts at line 830.

Otherwise, this looks good!

@balloob balloob merged commit adb92e1 into home-assistant:dev May 3, 2019
@balloob
Copy link
Member

balloob commented May 3, 2019

Fighting prettier manually is tough. I just copy pasted the files locally, ran through prettier and pasted them back.

Thanks Sean!

@SeanPM5
Copy link
Contributor Author

SeanPM5 commented May 3, 2019

Thanks @thomasloven & @robbiet480 for the helpful pointers and @balloob for taking it across the finish line after I got totally stumped on the prettier stuff. Very kind of you guys to guide me through my first PR here.

@balloob balloob mentioned this pull request May 8, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Jul 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants