Skip to content

Replace deprecated buttons on epic detail view

Taurie Davis requested to merge epic-detail-view-deprecated-button into master

What does this MR do?

Updates:

  • “Close epic” to not utilize .btn-warning-secondary
  • “New epic” to not utilize .btn-success
  • “Comment” split button to not utilize .btn-success
  • “Turn on” button in Confidentiality popover within sidebar to not utilize .ban-warning

Screenshots (strongly suggested)

before after
Screen_Shot_2021-03-29_at_3.45.05_PM Screen_Shot_2021-03-29_at_3.46.02_PM
Screen_Shot_2021-03-29_at_3.46.41_PM Screen_Shot_2021-03-29_at_3.48.00_PM
Screen_Shot_2021-03-29_at_3.48.54_PM Screen_Shot_2021-03-29_at_3.50.42_PM

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

Merge request reports

Loading