-
Notifications
You must be signed in to change notification settings - Fork 15
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
Make email link open new window/tab #55
Comments
Please don't do this. It would just open a new tab and it would confuse people. In fairness [a href="mailto:[email protected]" class="btn btn-lg btn-primary"][email protected][/a] should be good, but the Bootstrap CSS is overriding its selectability. I recommend we stick with [a href="mailto:[email protected]" ][email protected][/a] for now. It's not as flashy, but it's accessible. -Tome |
Tome, do you think you could unravel the CSS so we can close #54? |
I agree, |
I'll take a close look at it when I'm back into office tomorrow. Should be
a quick fix.
-Tome
On Thu, Apr 13, 2017 at 9:26 PM Dana Woodman ***@***.***> wrote:
I agree, mailto works even if it's less elegant. Once we have a contact
form we can remove that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#55 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AaDg8vHHOKLa3vLDvhpAVvy8QU5ewqdfks5rvstZgaJpZM4M81-Y>
.
--
*Tome Wilson*
Mill Race Theatrical Co.
www.millrace.co
|
Consensus seems to go toward closing this; will do so if there are no objections. |
Might want to set target="_blank" for the [email protected] email link.
The text was updated successfully, but these errors were encountered: