AS1024
/
Kanboard-Prod
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add a "Me" button to assignee form element

This commit is contained in:
Frederic Guillot 2016-01-31 11:05:47 -05:00
parent 3279768c1c
commit 9570793f68
34 changed files with 83 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ChangeLog
View File

@ -10,6 +10,7 @@ Breaking changes:
New features:
* Add a "Me" button to assignee form element
* Add external links for tasks with plugin api
* Add project owner (Directly Responsible Individual)
* Add configurable task priority

11
app/Helper/Task.php
View File

@ -45,6 +45,17 @@ class Task extends Base
return $this->taskPermission->canRemoveTask($task);
}
public function selectAssignee(array $users, array $values, array $errors = array(), array $attributes = array())
{
$attributes = array_merge(array('tabindex="3"'), $attributes);
$html = $this->helper->form->label(t('Assignee'), 'owner_id');
$html .= $this->helper->form->select('owner_id', $users, $values, $errors, $attributes);
$html .= '<a href="#" class="assign-me" data-target-id="form-owner_id" data-current-id="'.$this->userSession->getId().'" title="'.t('Assign to me').'">'.t('Me').'</a>';
return $html;
}
public function selectPriority(array $project, array $values)
{
$html = '';

2
app/Locale/bs_BA/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/cs_CZ/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/da_DK/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/de_DE/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/el_GR/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/es_ES/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/fi_FI/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/fr_FR/translations.php
View File

@ -1126,4 +1126,6 @@ return array(
'There is no external link for the moment.' => 'Il n\'y a pas de lien externe pour le moment.',
'Internal links' => 'Liens internes',
'There is no internal link for the moment.' => 'Il n\'y a pas de lien interne pour le moment.',
'Assign to me' => 'Assigner à moi',
'Me' => 'Moi',
);

2
app/Locale/hu_HU/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/id_ID/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/it_IT/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/ja_JP/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/my_MY/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/nb_NO/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/nl_NL/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/pl_PL/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/pt_BR/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/pt_PT/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/ru_RU/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/sr_Latn_RS/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/sv_SE/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/th_TH/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/tr_TR/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

2
app/Locale/zh_CN/translations.php
View File

@ -1123,4 +1123,6 @@ return array(
// 'There is no external link for the moment.' => '',
// 'Internal links' => '',
// 'There is no internal link for the moment.' => '',
// 'Assign to me' => '',
// 'Me' => '',
);

3
app/Template/board/popover_assignee.php
View File

@ -9,8 +9,7 @@
<?= $this->form->hidden('id', $values) ?>
<?= $this->form->hidden('project_id', $values) ?>
<?= $this->form->label(t('Assignee'), 'owner_id') ?>
<?= $this->form->select('owner_id', $users_list, $values, array(), array('autofocus')) ?><br/>
<?= $this->task->selectAssignee($users_list, $values, array(), array('autofocus')) ?>
<div class="form-actions">
<input type="submit" value="<?= t('Save') ?>" class="btn btn-blue"/>

3
app/Template/task_creation/form.php
View File

@ -48,8 +48,7 @@
<div class="form-column">
<?= $this->form->hidden('project_id', $values) ?>
<?= $this->form->label(t('Assignee'), 'owner_id') ?>
<?= $this->form->select('owner_id', $users_list, $values, $errors, array('tabindex="3"')) ?>
<?= $this->task->selectAssignee($users_list, $values, $errors) ?>
<?= $this->form->label(t('Category'), 'category_id') ?>
<?= $this->form->select('category_id', $categories_list, $values, $errors, array('tabindex="4"')) ?>

3
app/Template/task_modification/edit_task.php
View File

@ -44,8 +44,7 @@
<?= $this->form->hidden('id', $values) ?>
<?= $this->form->hidden('project_id', $values) ?>
<?= $this->form->label(t('Assignee'), 'owner_id') ?>
<?= $this->form->select('owner_id', $users_list, $values, $errors, array('tabindex="3"')) ?>
<?= $this->task->selectAssignee($users_list, $values, $errors) ?>
<?= $this->form->label(t('Category'), 'category_id') ?>
<?= $this->form->select('category_id', $categories_list, $values, $errors, array('tabindex="4"')) ?>

2
assets/css/app.css
View File

File diff suppressed because one or more lines are too long

2
assets/css/print.css
View File

File diff suppressed because one or more lines are too long

6
assets/css/src/task.css
View File

@ -329,3 +329,9 @@ div.color-square-selected {
height: 28px;
box-shadow: 3px 2px 10px 0 rgba(180,180,180,0.9);
}
/* Assign to me */
.assign-me {
font-size: 0.8em;
vertical-align: bottom;
}

2
assets/js/app.js
View File

File diff suppressed because one or more lines are too long

11
assets/js/src/Task.js
View File

@ -7,4 +7,15 @@ Task.prototype.listen = function() {
$(this).addClass("color-square-selected");
$("#form-color_id").val($(this).data("color-id"));
});
$(document).on("click", ".assign-me", function(e) {
e.preventDefault();
var currentId = $(this).data("current-id");
var dropdownId = "#" + $(this).data("target-id");
if ($(dropdownId + ' option[value=' + currentId + ']').length) {
$(dropdownId).val(currentId);
}
});
};