Skip to content

Separate emoji and activity controls

Marcel van Remmerden requested to merge separate-emoji-activity-controls into master

What does this MR do and why?

Separates the controls for emoji/creating a new branch from the controls related to the activity log (sort/filter/timeline).

Screenshots or screen recordings (2022-09-12 03:05)

These screenshots are with the :work_items_hierarchy feature enabled.

Issue
Description Before After
Wide display.

🎉 Emoji award controls and "Create merge request" moved below description.
🎉 "Activity" header with activity sort/filter contols.
96260_before_issue 96260_after_issue
Small display (top of page).

🎉 Lots of room for emoji award control and "Create merge request" dropdown.
96260_before_issue_upper 96260_after_issue_sm_upper
Small display (top of "notes").

🎉 Previous clutter is cleaned up.
96260_before_issue_lower 96260_after_issue_sm_lower
Epic
Description Before After
Wide display.

🎉 Emoji award controls moved below description.
🎉 "Activity" header with activity sort/filter contols.
96260_before_epic 96260_after_epic
Small display (top of page).

96260_before_epic_upper 96260_after_epic_sm_upper
Small display (top of "notes").

🎉 Looks much better and lots of less clutter.
96260_before_epic_lower 96260_after_epic_sm_lower
MR
Description Before After
Wide display.

🎉 Emoji award controls moved below description.
🎉 "Activity" header with activity sort/filter contols.
96260_before_mr 96260_after_mr
Small display (top of page).

96260_before_mr_sm_upper 96260_after_mr_sm_upper
Small display (top of "notes").

96260_before_mr_sm_lower 96260_after_mr_sm_lower
Incident
Description Before After
Wide display.

🎉 Emoji award controls moved below description.
🎉 "Activity" header with activity sort/filter contols.
🎉 "Show Timeline View Toggle" button is intact next to sort/filter control.
96260_before_incident 96260_after_incident
Small display (top of page).

96260_before_incident_sm_upper 96260_after_incident_sm_upper
Small display (top of "notes").

🎉 So much cleaner!
96260_before_incident_sm_lower 96260_after_incident_sm_lower
Incident on "Timeline" Tab
Description Before After
🎉 Nothing changed. 96260_before_incident_timeline 96260_after_incident_timeline

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Original screenshots (outdated)

This MR scope has changed since it was first opened. These are the original screenshots of the MR in it's original state.

Desktop

Before After
image image
image image
image image
image image

Mobile

Before After
image image
image image
image image
image image

Related to ✨ Beautifying our UI: Create:Code Review ✨ (#364574 - closed)

Edited by Annabel Dunstone Gray

Merge request reports