-
Notifications
You must be signed in to change notification settings - Fork 264
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
missing translation for 'Add a new stack' #169
Conversation
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
Codecov Report
@@ Coverage Diff @@
## master #169 +/- ##
=======================================
Coverage 72.34% 72.34%
=======================================
Files 32 32
Lines 846 846
=======================================
Hits 612 612
Misses 234 234 |
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.
You missed this one:
deck/templates/part.board.mainView.php
line 39
@pixelipo you are right! Thanks! |
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
templates/part.boardlist.php
Outdated
@@ -3,7 +3,7 @@ | |||
<thead> | |||
<tr> | |||
<td class="cell-board-bullet"></td> | |||
<td class="cell-board-title" width="90%"><?php p($l->t('Board title')); ?></td> | |||
<td class="cell-board-title" width="90%"><?php p($l->t('Boards')); ?></td> |
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.
I don't think this is correct, @artemanufrij - it's a row title, and the row displays board titles.
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.
@juliushaertl what do you think about?
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.
@pixelipo @juliushaertl may be just "Title"?
Title
Members
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.
Title
is better.
By the way, Files app uses Name
for the same row in their table
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.
@pixelipo ok, I change it to"Title"
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.
Yep just title sounds better.
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
Signed-off-by: Artem Anufrij <artem.anufrij@live.de> fixed pixelipo's comment label->tag Signed-off-by: Artem Anufrij <artem.anufrij@live.de> 'Board title' -> 'New Board title' Signed-off-by: Artem Anufrij <artem.anufrij@live.de> 'Board title' -> 'Boards' in Boardlist Signed-off-by: Artem Anufrij <artem.anufrij@live.de> [tx-robot] updated from transifex renamed column header 'Boards' -> 'Title' Signed-off-by: Artem Anufrij <artem.anufrij@live.de> Merge branch 'master' into missing_translation
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
templates/part.board.mainView.php
Outdated
@@ -34,10 +34,10 @@ | |||
<div class="stack" ng-repeat="s in stacks" data-as-sortable-item | |||
data-columnindex="{{$index}}" id="column{{$index}}" | |||
style=""> | |||
<h2 data-as-sortable-item-handle> | |||
<h2 data-as-sortable-item-handle> |
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.
you got some weird space left before the <h2>
tag
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.
@pixelipo fixed
Signed-off-by: Artem Anufrij <artem.anufrij@live.de>
Signed-off-by: Artem Anufrij artem.anufrij@live.de