Skip to content

Pin vulnerability list header to the page top

Alexander Turinske requested to merge 215355-pin-header into master

What does this MR do?

Pin selection summary/list header to the page top

  • create class that sets the header immediately below the header and performance bar and updates the height depending on if the selection summary is visible or not

Screenshots

After

Firefox

Project-level security dashboard with header only

project_header

Project-level security dashboard with header and selection summary

project_header_selection_summary

Group-level security dashboard with header (no selection summary on this page)

group_header

Instance-level security dashboard with header (no selection summary on this page)

instance_header

Chrome

Project-level security dashboard with header only

chrome_project_security_dashboard

Project-level security dashboard with header and selection summary

chrome_project_security_dashboard_with_selection_summary

Instance/Group-level security dashboard with header (no selection summary on this page) chrome_instance_group_security_dashboard

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

Related to #215355 (closed)

Edited by Alexander Turinske

Merge request reports