-
Notifications
You must be signed in to change notification settings - Fork 229
Rearrange the UI to show packages at the top #1753 #1756
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
e8a77ca
to
1c3e3b0
Compare
Signed-off-by: kunalsz <[email protected]>
@kunalsz LGTM! please sign off this commit. So I can merge |
1c3e3b0
to
1365cfa
Compare
@TG1999 Hey sorry about that. I just did that, you can now merge it. |
vulnerabilities/templates/index.html
Outdated
@@ -8,14 +8,14 @@ | |||
{% block content %} | |||
<section class="section pt-2"> | |||
<div class="container"> | |||
<div class="columns is-centered mb-5 mt-2"> | |||
<div class="columns is-centered mb-5"> |
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 belive we need margin from top here
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.
@TG1999 I have added the margin class as you said,you can review it now
Signed-off-by: kunalsz <[email protected]>
Rearranged the packages division up and vulnerabilities division down #1753