Skip to content

Use CopyableField component in Issue Sidebar email address component

Tom Quirk requested to merge use-copyable_field-in-issue-email into master

What does this MR do?

Follow-up to !5682 (closed). Uses the new copyable_field.vue component for the presentation of copy_email_to_clipboard.vue.

Additionally, copyable_field has a new prop: clipboardTooltipText, to allow the developer to set custom tooltip text for the clipboard button (in this case, Copy email address).

Related Issue: #324610 (closed)

Screenshots (strongly suggested)

Screenshots demonstrate no user-facing change

Before After
Screen_Shot_2021-03-26_at_9.42.52_pm Screen_Shot_2021-03-26_at_9.45.13_pm
Screen_Shot_2021-03-26_at_9.53.51_pm Screen_Shot_2021-03-26_at_9.55.24_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 Tom Quirk

Merge request reports

Loading