-
Notifications
You must be signed in to change notification settings - Fork 274
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add stack button on empty board #1865
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1865 +/- ##
=======================================
Coverage 56.90% 56.90%
=======================================
Files 77 77
Lines 3840 3840
=======================================
Hits 2185 2185
Misses 1655 1655 |
I think we should have this in a emptycontent layout here similar to the board not found message. You can use our new component from |
5e140b3
to
44d2dd1
Compare
44d2dd1
to
05db9cc
Compare
src/components/board/Board.vue
Outdated
@@ -23,6 +23,26 @@ | |||
<template> | |||
<div class="board-wrapper"> | |||
<Controls :board="board" /> | |||
|
|||
<EmptyContent v-if="stacksByBoard.length === 0" icon="icon-pause"> | |||
No lists, add one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be translatable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @jancborchardt for some nicer text maybe ;)
@@ -117,11 +140,10 @@ export default { | |||
this.$store.dispatch('orderStack', { stack: this.stacksByBoard[removedIndex], removedIndex, addedIndex }) | |||
}, | |||
|
|||
createStack() { | |||
addNewStack() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure that newStackTitle is emptied once the stack has been created.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See inline
done, let's merge |
src/components/board/Board.vue
Outdated
@@ -23,6 +23,26 @@ | |||
<template> | |||
<div class="board-wrapper"> | |||
<Controls :board="board" /> | |||
|
|||
<EmptyContent v-if="stacksByBoard.length === 0" icon="icon-pause"> | |||
{{ t('deck', 'No lists, add one') }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's do something more descriptive then here like:
No lists available
Create a new list to add cards to this board
Signed-off-by: Jakob Röhrl <jakob.roehrl@web.de>
Signed-off-by: Jakob Röhrl <jakob.roehrl@web.de>
Signed-off-by: Jakob Röhrl <jakob.roehrl@web.de>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
0b72dcd
to
ba0a482
Compare
Signed-off-by: Jakob Röhrl jakob.roehrl@web.de
It shows only on empty board up: