Skip to content

Fix spacing between columns

Savas Vedova requested to merge 215921-fix-spacing-vuln-list into master

What does this MR do?

Fixes the spacing between columns in the vulnerability list in the security dashboards. The new Security Dashboard uses the gl-table component from gitlab-ui. That manages the spacings between the columns much better. However, since this dashboard will be disabled as soon as we release the new security dashboard, I didn't refactor it to use gl-table component.

The fix was to change the columns to a percentage width. That should fix the spacing in the Group and Instance Level dashboards. However, it also changes slightly the Project Level dashboard because we're reusing the same component in all three dashboards. I don't think it's a big deal if we slightly increase the spacing in the Project Level but I will leave this to the UX people. You can see the screenshots below.

NOTE In order to view this dashboard, you'll need to disable the feature flag if it's enabled. You can do so by:

# In the gitlab folder execute:
bin/rails console
# Once the console is ready execute:
Feature.disable(:first_class_vulnerabilities)
# After waiting a few seconds refresh the page
# To reenable the feature flag:
Feature.enable(:first_class_vulnerabilities)
# Type `exit` to exit
exit

Screenshots

How it looks on the Project Level after the changes:

image

This is the Group Level:

image

This is the Instance Level:

image

Mobile view:

image

This is how the mobile view was before:

image

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Yorick Peterse

Merge request reports