Skip to content
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

551_show_tag_title_on_cards_overview #561

Merged
merged 2 commits into from
Jul 31, 2018

Conversation

philippe-levan
Copy link
Collaborator

Signed-off-by: Philippe Le Van philippe.levan@kibatic.com

Summary

cards labels visible under the title

image

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Tests (unit, integration, api and/or acceptance) are included
    N/A (only UI)
  • Documentation (manuals or wiki) has been updated or is not required
    N/A (only UI)

@codecov
Copy link

codecov bot commented Jul 30, 2018

Codecov Report

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

@@           Coverage Diff           @@
##           master     #561   +/-   ##
=======================================
  Coverage   80.19%   80.19%           
=======================================
  Files          46       46           
  Lines        1601     1601           
=======================================
  Hits         1284     1284           
  Misses        317      317

css/style.scss Outdated
width: 15px;
height: 20px;
padding: 0 4px;
margin: 0 2px 1px 0;
Copy link
Member

Choose a reason for hiding this comment

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

Lets probably use a 2px bottom margin as well, so they are nicly aligned when using multiple lines.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done :-)

@juliusknorr
Copy link
Member

Look really good. Just one small comment. 😉

cc @nextcloud/deck

Signed-off-by: Philippe Le Van <philippe.levan@kibatic.com>
Signed-off-by: Philippe Le Van <philippe.levan@kibatic.com>
@juliusknorr juliusknorr force-pushed the 551_show_tag_title_on_cards_overview branch from 6a4126a to 508674f Compare July 31, 2018 07:21
@juliusknorr juliusknorr merged commit 9f7bef9 into master Jul 31, 2018
@juliusknorr
Copy link
Member

Thanks a lot, this is a big improvement 👍

@juliusknorr juliusknorr deleted the 551_show_tag_title_on_cards_overview branch July 31, 2018 07:57
@philippe-levan
Copy link
Collaborator Author

Thanks for the review and the merge !

@juliusknorr juliusknorr mentioned this pull request Oct 8, 2018
33 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.

2 participants