Commit 31d6368e authored by Savas Vedova's avatar Savas Vedova

Move CSS to page related bundle

Security dashboard css is very specific to its own page
therefore we've decided to move it into its own bundle
in order to improve page performance
parent e01351e1
...@@ -184,46 +184,3 @@ table { ...@@ -184,46 +184,3 @@ table {
border-top: 0; border-top: 0;
} }
} }
.vulnerability-list {
@media (min-width: $breakpoint-sm) {
.checkbox {
padding-left: $gl-spacing-scale-4;
padding-right: 0;
width: 1px;
+ td,
+ th {
padding-left: $gl-spacing-scale-4;
}
}
.detected {
width: 9%;
}
.status {
width: 8%;
}
.severity {
width: 10%;
}
.description {
max-width: 0;
}
.identifier {
width: 16%;
}
.scanner {
width: 10%;
}
.activity {
width: 5%;
}
}
}
...@@ -198,6 +198,7 @@ module Gitlab ...@@ -198,6 +198,7 @@ module Gitlab
config.assets.precompile << "page_bundles/pipeline_schedules.css" config.assets.precompile << "page_bundles/pipeline_schedules.css"
config.assets.precompile << "page_bundles/productivity_analytics.css" config.assets.precompile << "page_bundles/productivity_analytics.css"
config.assets.precompile << "page_bundles/profile_two_factor_auth.css" config.assets.precompile << "page_bundles/profile_two_factor_auth.css"
config.assets.precompile << "page_bundles/security_dashboard.css"
config.assets.precompile << "page_bundles/terminal.css" config.assets.precompile << "page_bundles/terminal.css"
config.assets.precompile << "page_bundles/todos.css" config.assets.precompile << "page_bundles/todos.css"
config.assets.precompile << "page_bundles/reports.css" config.assets.precompile << "page_bundles/reports.css"
......
@import 'page_bundles/mixins_and_variables_and_functions';
$security-filter-height: 90px; $security-filter-height: 90px;
$selection-summary-height: 68px; $selection-summary-height: 68px;
...@@ -38,9 +40,60 @@ $selection-summary-height: 68px; ...@@ -38,9 +40,60 @@ $selection-summary-height: 68px;
@include sticky-top-positioning(); @include sticky-top-positioning();
} }
// Due to position: sticky not being supported on Chrome (https://caniuse.com/#feat=css-sticky), .security-dashboard-accordion > ul {
// the property is assigned to the th element as a workaround @include gl-display-flex;
@include gl-flex-fill-1;
}
.security-charts > section {
flex: 1 1 40%;
margin: 1rem;
}
.vulnerability-list { .vulnerability-list {
@media (min-width: $breakpoint-sm) {
.checkbox {
padding-left: $gl-spacing-scale-4;
padding-right: 0;
width: 1px;
+ td,
+ th {
padding-left: $gl-spacing-scale-4;
}
}
.detected {
width: 9%;
}
.status {
width: 8%;
}
.severity {
width: 10%;
}
.description {
max-width: 0;
}
.identifier {
width: 16%;
}
.scanner {
width: 10%;
}
.activity {
width: 5%;
}
}
// Due to position: sticky not being supported on Chrome (https://caniuse.com/#feat=css-sticky),
// the property is assigned to the th element as a workaround
.card, .card,
thead th { thead th {
position: -webkit-sticky; position: -webkit-sticky;
...@@ -57,13 +110,3 @@ $selection-summary-height: 68px; ...@@ -57,13 +110,3 @@ $selection-summary-height: 68px;
@include sticky-top-positioning($security-filter-height + $selection-summary-height); @include sticky-top-positioning($security-filter-height + $selection-summary-height);
} }
} }
.security-dashboard-accordion > ul {
@include gl-display-flex;
@include gl-flex-fill-1;
}
.security-charts > section {
flex: 1 1 40%;
margin: 1rem;
}
- breadcrumb_title _("Security Dashboard") - breadcrumb_title _("Security Dashboard")
- page_title _("Security Dashboard") - page_title _("Security Dashboard")
- add_page_specific_style 'page_bundles/security_dashboard'
#js-group-security-dashboard{ data: group_level_security_dashboard_data(@group) } #js-group-security-dashboard{ data: group_level_security_dashboard_data(@group) }
- breadcrumb_title _("Vulnerability Report") - breadcrumb_title _("Vulnerability Report")
- page_title _("Vulnerability Report") - page_title _("Vulnerability Report")
- add_page_specific_style 'page_bundles/security_dashboard'
#js-group-vulnerabilities{ data: group_level_security_dashboard_data(@group) } #js-group-vulnerabilities{ data: group_level_security_dashboard_data(@group) }
- breadcrumb_title _("Security Dashboard") - breadcrumb_title _("Security Dashboard")
- page_title _("Security Dashboard") - page_title _("Security Dashboard")
- add_page_specific_style 'page_bundles/reports' - add_page_specific_style 'page_bundles/reports'
- add_page_specific_style 'page_bundles/security_dashboard'
#js-project-security-dashboard{ data: project_security_dashboard_config(@project) } #js-project-security-dashboard{ data: project_security_dashboard_config(@project) }
...@@ -4,5 +4,6 @@ ...@@ -4,5 +4,6 @@
- page_title @vulnerability.title - page_title @vulnerability.title
- page_description @vulnerability.description - page_description @vulnerability.description
- vulnerability_init_details = { vulnerability: vulnerability_details_json(@vulnerability, @pipeline)} - vulnerability_init_details = { vulnerability: vulnerability_details_json(@vulnerability, @pipeline)}
- add_page_specific_style 'page_bundles/security_dashboard'
#js-vulnerability-main{ data: vulnerability_init_details } #js-vulnerability-main{ data: vulnerability_init_details }
- breadcrumb_title _("Vulnerability Report") - breadcrumb_title _("Vulnerability Report")
- page_title _("Vulnerability Report") - page_title _("Vulnerability Report")
- add_page_specific_style 'page_bundles/security_dashboard'
#js-security-report-app{ data: project_security_dashboard_config(@project) } #js-security-report-app{ data: project_security_dashboard_config(@project) }
- page_title _('Security Dashboard') - page_title _('Security Dashboard')
- add_page_specific_style 'page_bundles/security_dashboard'
#js-security{ data: instance_security_dashboard_data } #js-security{ data: instance_security_dashboard_data }
- page_title _('Vulnerability Report') - page_title _('Vulnerability Report')
- add_page_specific_style 'page_bundles/security_dashboard'
#js-vulnerabilities{ data: instance_security_dashboard_data } #js-vulnerabilities{ data: instance_security_dashboard_data }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment