Skip to content

fix: namespace css classes

Scott Hampton requested to merge 32-namespace-css into master

In order to reduce CSS clashes when injecting onto other pages, we are adding extra scope to make the selectors more unique.

Closes #32 (closed)

What's in this MR?

  • Most CSS classes are now prefixed by #gitlab-review-container.
// Before
.gitlab-button-secondary:hover {
  // [...]
}

// After
#gitlab-review-container .gitlab-button-secondary:hover {
  // [...]
}
// Before
.gl-fixed {
  // [...]
}

// After
#gitlab-review-container .gl-fixed,
#gitlab-review-container.gl-fixed {
  position: fixed;
}
Edited by Scott Hampton

Merge request reports