-
Notifications
You must be signed in to change notification settings - Fork 3
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
Temporarily hide emails #214
Conversation
@@ -11,18 +11,16 @@ | |||
<div class="col-xs-12"> | |||
<form class="form" method="post" role="form" | |||
enctype="multipart/form-data"> | |||
<p>Alumni/BSD/Medicine/Medical School affiliates: if you are unable |
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.
Nooo. Why is this back at the top again?
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.
Oh! I thought that was Git being wonky. Will revert.
e8400f8
to
6b2fc5a
Compare
Reverted! |
Has merge conflicts? |
listing might not be posted immediately. We reserve the right to | ||
remove listings at any time. | ||
</div> | ||
<p>Alumni/BSD/Medicine/Medical School affiliates: if you are unable |
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.
Why is it a
tag instead of the
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.
Oh, good point. I think I must've been using an old master
for this one...
a62aa5c
to
bfe63dc
Compare
listing might not be posted immediately. We reserve the right to | ||
remove listings at any time. | ||
</div> | ||
<p class="alert alert-warning" role="alert"> |
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.
Have not reverted to the correct master?
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.
I think it is. The <p>
is more semantic than a <div>
, but it should show up in the same manner.
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.
Oh oops. I missed the CSS class there. Sure.
bfe63dc
to
8542076
Compare
I realized on Wednesday that we never really talked about showing people's emails, so I un-showed them, pending further policy discussion.