Skip to content
Snippets Groups Projects

Added initial todo functionality to collapsed sidebar

Merged Dimitrie Hoekstra requested to merge create-collapsed-todo-button into master

What does this MR do?

Adds todo functionality to the collapsed sidebar in the issue and mr view

Are there points in the code the reviewer needs to double check?

it still needs:

  • check on accessibility aria stuff
  • tooltip
  • loading spinner icon should temp replace the icons
  • should work/show state across states of sidebar (open/closed)

Why was this MR needed?

#24805 (closed)

Screenshots (if relevant)

2017-03-24_16.09.02

image

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #24805 (closed)

Merge request reports

Pipeline #7270576 passed

Pipeline passed for 89d1a1c0 on create-collapsed-todo-button

Test coverage 69.26% (23.86%) from 2 jobs

Merged by Filipa LacerdaFilipa Lacerda 8 years ago (Mar 29, 2017 1:41pm UTC)

Loading

Pipeline #7317369 failed

Pipeline failed for 412c9e8b on master

Test coverage 45.43% (23.86%) from 2 jobs

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading