Skip to content

Remove head-variant="white" prop workaround from GlTable

Justin Ho Tuan Duong requested to merge justin_ho-remove-head-variant-white-prop-from-gl-tables into master

What does this MR do and why?

These were added to override some styles that made the header styles different from the original GlTable.

Those styles have since been removed and this workaround is no longer needed.

Changelog: changed

Screenshots or screen recordings

Page Before After
Project > Value Stream Analytics http://127.0.0.1:3000/gitlab-org/gitlab-shell/-/value_stream_analytics VSA_before VSA_after
Project > Clusters > Agent http://127.0.0.1:3000/gitlab-org/gitlab-shell/-/clusters?tab=agent Cluster_agent_before Cluster_agent_after
Project > Clusters http://127.0.0.1:3000/gitlab-org/gitlab-shell/-/clusters Clusters_before Clusters_after
Group > Members http://127.0.0.1:3000/groups/gitlab-org/-/group_members Group_members_before Group_members_after
Group > Compliance report http://127.0.0.1:3000/groups/gitlab-org/-/security/compliance_dashboard Compliance_report_before Compliance_report_after
Project > Security > Policies http://127.0.0.1:3000/jashkenas/underscore/-/security/policies Policies_before Policies_after

How to set up and validate locally

Open each page in the screenshots section.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports