Skip to content

Utilize gitlab-ui components more accurately in artifacts_block.vue

Payton Burdette requested to merge pb-resolve-tech-debt-artifacts-block into master

What does this MR do?

  • Use GlButtonGroup instead of .btn-group for action buttons
  • Use GlButton with href prop instead of bootstrap classes on the GlLink component
  • Use gitlab-ui utility classes where possible

This is a developer related change and does not warrant a changelog

Related to #299912 (closed)

Screenshots (strongly suggested)

before after
Screen_Shot_2021-01-25_at_2.17.31_PM Screen_Shot_2021-01-25_at_2.28.41_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
Edited by Payton Burdette

Merge request reports

Loading