Repository navigation
Remove Interactive EU from frontpage - #909
Conversation
|
I think the relevant CSS media queries should also be tweaked. |
Now that there's only one event and the logo remaining, there's no need for two logo's which is hidden/displayed responsively.
|
Thanks for the reminder @lpinca! Did a small CSS refactoring and simplified HTML a bit. Still looks the same as the after screenshot, just w/o the redundant code. |
| alt="Node.js Interactive" /> | ||
| </span> | ||
| <a class="interactive-item" href="http://events.linuxfoundation.org/events/node-interactive"> | ||
| <img src="/static/images/interactive/nodejs-interactive-logo-center.png" /> |
There was a problem hiding this comment.
The wrapper span is still required or the image will not be scaled well. This is how it looks in Chrome, responsive mode (Tablet - 768px):
|
Do we care about IE11? If so can anyone test this on IE11? I remember I spent some time to make this banner work there. |
Scaling of logo and responsiveness looks to be working as expected on my win7/IE11 VM. |
|
Now that this is merged, I realized that the background is Amsterdam and it should probably be Austin. heh I'll reach out to them and see if they can provide a new image for Austin. |
|
@williamkapke we can cut the background. The right part should be correct, it is huge.
|
Since the conference has ended, it's about time to remove the Interactive EU banner from the frontpage?
BEFORE
AFTER