Skip to content

Conversation

@jakobroehrl
Copy link
Contributor

@jakobroehrl jakobroehrl commented May 12, 2020

Signed-off-by: Jakob Röhrl [email protected]

It shows only on empty board up:
grafik

@codecov-io
Copy link

Codecov Report

Merging #1865 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1865   +/-   ##
=======================================
  Coverage   56.90%   56.90%           
=======================================
  Files          77       77           
  Lines        3840     3840           
=======================================
  Hits         2185     2185           
  Misses       1655     1655           

@juliusknorr
Copy link
Member

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 @nextcloud/vue for that: https://nextcloud-vue-components.netlify.app/#/Components/EmptyContent

@jakobroehrl jakobroehrl force-pushed the enh/addStackOnEmptyBoard branch from 5e140b3 to 44d2dd1 Compare July 2, 2020 12:42
@nextcloud nextcloud deleted a comment from juliusknorr Jul 8, 2020
@nextcloud nextcloud deleted a comment from juliusknorr Jul 8, 2020
@nextcloud nextcloud deleted a comment from juliusknorr Jul 8, 2020
@jakobroehrl jakobroehrl force-pushed the enh/addStackOnEmptyBoard branch from 44d2dd1 to 05db9cc Compare July 8, 2020 09:57
@nextcloud nextcloud deleted a comment from juliusknorr Jul 8, 2020
<Controls :board="board" />

<EmptyContent v-if="stacksByBoard.length === 0" icon="icon-pause">
No lists, add one
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be translatable

Copy link
Member

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 ;)

},
createStack() {
addNewStack() {
Copy link
Member

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.

Copy link
Member

@juliusknorr juliusknorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See inline

@jakobroehrl
Copy link
Contributor Author

done, let's merge

<Controls :board="board" />

<EmptyContent v-if="stacksByBoard.length === 0" icon="icon-pause">
{{ t('deck', 'No lists, add one') }}
Copy link
Member

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

@juliusknorr juliusknorr force-pushed the enh/addStackOnEmptyBoard branch from 0b72dcd to ba0a482 Compare July 31, 2020 20:10
@juliusknorr juliusknorr merged commit 137b6dd into master Jul 31, 2020
@juliusknorr juliusknorr deleted the enh/addStackOnEmptyBoard branch July 31, 2020 20:48
@juliusknorr juliusknorr mentioned this pull request Sep 1, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

addtional add stack button at the main area

4 participants