Skip to content

Use note header component in event item component

Daniel Tian requested to merge 209993-use-note-header into master

NOTE: This MR requires that !28898 (merged) be merged first.

What does this MR do?

Throughout the first class vulnerabilities feature, we have a common header for a system note. It can either be a single line:

Single line system note
Screenshot_from_2020-04-13_11-09-02

or on two lines where the top line is the header:

Two line system note
Screenshot_from_2020-04-13_11-09-12

There is already a app/assets/javascripts/notes/components/note_header.vue component that provides the header and handles some special cases. This MR is to switch history_entry.vue to use note_header.vue instead of duplicating what it does. It also makes some changes to note_header.vue to make it more flexible and reusable.

How to test locally

https://gitlab.com/snippets/1956676

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Daniel Tian

Merge request reports