-
-
Notifications
You must be signed in to change notification settings - Fork 6
change: tweak signup form and link to blog subscribe page #627
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
Conversation
Test-bot: skip
User Test ResultsTest specification and instructions User tests are not required |
| #mc_embed_signup_page #num-subscribers span {padding:.5em; border:1px solid #ccc; margin-right:.5em; font-weight:bold;} | ||
|
|
||
| #mc_embed_signup_page #mc-embedded-subscribe-form div.mce_inline_error {display:inline-block; margin:2px 0 1em 0; padding:5px 10px; background-color:rgba(255,255,255,0.85); -webkit-border-radius: 3px; -moz-border-radius: 3px; border-radius: 3px; font-size:14px; font-weight:normal; z-index:1; color:#e85c41;} | ||
| #mc_embed_signup_page #mc-embedded-subscribe-form input.mce_inline_error {border:2px solid #e85c41;} |
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.
Was this all previously from classic-10_7.css?
Would it be cleaner to put into a separate mailchimp.css file?
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.
We could move to a separate .css, but it's only used in this page, so it's not a big win.
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.
And yeah, it was previously in that linked .css; I have tweaked it to make it look more keyman.com-ish (e.g. font, color)
darcywong00
left a comment
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
|
I'm merging this to get it in the end-of-sprint tasks |
Test-bot: skip