Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
Already on GitHub? Sign in to your account
Add warning icon to alert pages #935
Conversation
|
LGTM. I think you should merge now, and I'll rebase my upcoming PR on top of it. Thanks! |
saivann
merged commit fa4d5fd
into
bitcoin-dot-org:master
Jul 4, 2015
1 check was pending
continuous-integration/travis-ci/pr
The Travis CI build is in progress
Details
saivann
deleted the
saivann:alerticon branch
Jul 4, 2015
|
Awh, in case you were waiting for bitcoin.org to update like me... I forgot to sign my commit :) Please feel free to go ahead with any additional changes. |
|
Ok. About to open a PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
saivann commentedJul 4, 2015
Alert pages look like typical blog posts, not a PSA. Just adding one icon to the layout seems enough to fix that. Please feel free to merge at any moment or ignore if you feel it's unimportant.
Edit: Travis fail due to broken HTML in spv-mining not related to this PR.