Skip to content

Commit e679bfb

Browse files
Move maintainer actions below the list of maintainers
1 parent 9fc3165 commit e679bfb

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

css/app.scss

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1085,18 +1085,15 @@ input:focus:invalid:focus, textarea:focus:invalid:focus, select:focus:invalid:fo
10851085
.package .details #add-maintainer{
10861086
margin-top: 3px;
10871087
margin-bottom: 7px;
1088-
float: right;
10891088
font-size: 34px;
10901089
}
10911090
.package .details #transfer-package {
10921091
margin-top: 7px;
10931092
margin-bottom: 7px;
10941093
margin-right: 4px;
1095-
float: right;
10961094
font-size: 34px;
10971095
}
10981096
.package .details #remove-maintainer {
1099-
float: right;
11001097
font-size: 35px;
11011098
margin-top: 5px;
11021099
}

templates/package/view_package.html.twig

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -144,10 +144,14 @@
144144
{% for maintainer in package.maintainers -%}
145145
<a href="{{ path('user_profile', {'name': maintainer.username}) }}"><img width="48" height="48" title="{{ maintainer.username }}" src="https://www.gravatar.com/avatar/{{ maintainer.email|gravatar_hash }}?s=48&amp;d=identicon" srcset="https://www.gravatar.com/avatar/{{ maintainer.email|gravatar_hash }}?s=96&amp;d=identicon 2x"></a>
146146
{% endfor %}
147-
{% if is_granted('remove_maintainer', package) %}<a title="Remove Maintainer" id="remove-maintainer" href="{{ path('remove_maintainer', {'name': package.name}) }}"><i class="glyphicon glyphicon-remove"></i></a>{% endif %}
147+
</p>
148+
149+
{% if is_granted('remove_maintainer', package) or is_granted('add_maintainer', package) or is_granted('transfer_package', package) %}
150+
<h5>Maintainer actions</h5>
148151
{% if is_granted('add_maintainer', package) %}<a title="Add Maintainer" id="add-maintainer" href="{{ path('add_maintainer', {'name': package.name}) }}"><i class="glyphicon glyphicon-plus"></i></a>{% endif %}
152+
{% if is_granted('remove_maintainer', package) %}<a title="Remove Maintainer" id="remove-maintainer" href="{{ path('remove_maintainer', {'name': package.name}) }}"><i class="glyphicon glyphicon-remove"></i></a>{% endif %}
149153
{% if is_granted('transfer_package', package) %}<a title="Transfer Package" id="transfer-package" href="{{ path('transfer_package', {'name': package.name}) }}"><i class="glyphicon glyphicon-transfer"></i></a>{% endif %}
150-
</p>
154+
{% endif %}
151155

152156
<h5>Details</h5>
153157
{% set repoUrl = package.browsableRepository %}
@@ -338,7 +342,7 @@
338342
</li>
339343
{% endfor %}
340344
</ul>
341-
<button type="button" class="btn btn-info btn-sm add-maintainer-item">Add Another Maintainer</button>
345+
<button type="button" class="btn btn-info btn-sm add-maintainer-item">Add Maintainer</button>
342346
</div>
343347
{{ form_rest(transferPackageForm) }}
344348
<input class="btn btn-block btn-success btn-lg" type="submit" value="Transfer Ownership" />

0 commit comments

Comments
 (0)