-
-
Notifications
You must be signed in to change notification settings - Fork 233
Add basic UI template for API #1466
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
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.
Looks good to me.
body {background: none;} | ||
.navbar {background: #363636; border-top: 3px solid #3298dc;} | ||
.navbar-inverse .navbar-brand {color: white;} | ||
.nexb-blue |
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.
This line seems problematic.
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.
Yeah, it does, good eye, looks like somebody started to define the CSS for .next-blue
and got distracted, never finished. @pombredanne , where did this code come from?
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.
@tdruez @johnmhoran .nexb-blue is now the name... it is defined in https://github.com/nexB/vulnerablecode/pull/1466/files#diff-9cd3f89490bba2f7d7e1e8174417074cc8f0fbdeaf3b603c0f19f57e0b2e8f2fR15
is this OK?
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.
.navbar-inverse .navbar-brand {color: white;}
.nexb-blue
.booleanwidget {width: 90%;}
You need to remove the .nexb-blue
line, it's not a correct syntax.
@pombredanne please have a look when you get a chance and merge it! Thanks! |
body {background: none;} | ||
.navbar {background: #363636; border-top: 3px solid #3298dc;} | ||
.navbar-inverse .navbar-brand {color: white;} | ||
.nexb-blue |
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.
.navbar-inverse .navbar-brand {color: white;}
.nexb-blue
.booleanwidget {width: 90%;}
You need to remove the .nexb-blue
line, it's not a correct syntax.
79b311c
to
d95fd9f
Compare
Changes applied. Will merge shortly! @tdruez thanks |
Borrowed from ScanCode.io
Reference: #1465
Thanks-to: Thomas Druez @tdruez