From 51e4e29b8c9b91e870d1c2f15f8c413cd7d516a7 Mon Sep 17 00:00:00 2001
From: Jordi Boggiano <j.boggiano@seld.be>
Date: Thu, 9 Apr 2026 13:32:29 +0200
Subject: [PATCH] Make maintainer management forms more visible

---
 templates/package/view_package.html.twig | 140 +++++++++++------------
 1 file changed, 70 insertions(+), 70 deletions(-)

diff --git a/templates/package/view_package.html.twig b/templates/package/view_package.html.twig
index ac936e69c..a0c002ee6 100644
--- a/templates/package/view_package.html.twig
+++ b/templates/package/view_package.html.twig
@@ -165,6 +165,76 @@
                     {% set isGitHubRepoUrl = package.isGitHub() %}
                     {% set displayOpenIssues = version and version.support is not null and version.support.issues is defined and isGitHubRepoUrl and version.support.issues starts with repoUrl %}
 
+                    {% if is_granted('remove_maintainer', package) or is_granted('add_maintainer', package) %}
+                        <div>
+                            {% if is_granted('add_maintainer', package) %}
+                                {{ form_start(addMaintainerForm, {
+                                    attr: { id: 'add-maintainer-form', class: (show_add_maintainer_form|default(false) ? '': 'hidden') },
+                                    action: path('add_maintainer', { 'name': package.name })
+                                }) }}
+                                    <div>
+                                        <h4>Add Maintainer</h4>
+                                        <p>
+                                            {{ form_label(addMaintainerForm.user, "Username") }}
+                                            {{ form_errors(addMaintainerForm.user) }}
+                                            {{ form_widget(addMaintainerForm.user) }}
+                                        </p>
+                                        {{ form_rest(addMaintainerForm) }}
+                                        <input class="btn btn-block btn-success btn-lg" type="submit" value="Add Maintainer" />
+                                    </div>
+                                {{ form_end(addMaintainerForm) }}
+                            {% endif %}
+
+                            {% if is_granted('remove_maintainer', package) %}
+                                {{ form_start(removeMaintainerForm, {
+                                    attr: { id: 'remove-maintainer-form', class: (show_remove_maintainer_form|default(false) ? '': 'hidden') },
+                                    action: path('remove_maintainer', { 'name': package.name })
+                                }) }}
+                                    <div>
+                                        <h4>Remove Maintainer</h4>
+                                        <p>
+                                            {{ form_label(removeMaintainerForm.user, "Username") }}
+                                            {{ form_errors(removeMaintainerForm.user) }}
+                                            {{ form_widget(removeMaintainerForm.user) }}
+                                        </p>
+                                        {{ form_rest(removeMaintainerForm) }}
+                                        <input class="btn btn-block btn-danger btn-lg" type="submit" value="Remove Maintainer" />
+                                    </div>
+                                {{ form_end(removeMaintainerForm) }}
+                            {% endif %}
+                        </div>
+                    {% endif %}
+
+                    {% if is_granted('transfer_package', package) %}
+                        <div>
+                            {{ form_start(transferPackageForm, {
+                                attr: {
+                                    id: 'transfer-package-form',
+                                    class: (show_transfer_package_form|default(false) ? '': 'hidden'),
+                                },
+                                action: path('transfer_package', { 'name': package.name })
+                            }) }}
+                                <div>
+                                    <h4>Transfer Ownership</h4>
+                                    <div class="maintainers-collection-wrapper">
+                                        {{ form_errors(transferPackageForm.maintainers) }}
+                                        <ul class="maintainers-list" data-prototype="{{ form_widget(transferPackageForm.maintainers.vars.prototype)|e('html_attr') }}">
+                                            {% for maintainerField in transferPackageForm.maintainers %}
+                                                <li>
+                                                    {{ form_errors(maintainerField) }}
+                                                    {{ form_widget(maintainerField) }}
+                                                </li>
+                                            {% endfor %}
+                                        </ul>
+                                        <button type="button" class="btn btn-info btn-sm add-maintainer-item">Add Maintainer</button>
+                                    </div>
+                                    {{ form_rest(transferPackageForm) }}
+                                    <input class="btn btn-block btn-success btn-lg" type="submit" value="Transfer Ownership" />
+                                </div>
+                            {{ form_end(transferPackageForm) }}
+                        </div>
+                    {% endif %}
+
                     <div class="row package-aside">
                         <div class="details col-xs-12 col-sm-6 col-md-12">
                             <h5>Package info</h5>
@@ -296,76 +366,6 @@
                 </div>
             </div>
 
-            {% if is_granted('remove_maintainer', package) or is_granted('add_maintainer', package) %}
-                <div class="row">
-                    {% if is_granted('add_maintainer', package) %}
-                        {{ form_start(addMaintainerForm, {
-                            attr: { id: 'add-maintainer-form', class: 'col-sm-6 col-md-3 col-md-offset-9 ' ~ (show_add_maintainer_form|default(false) ? '': 'hidden') },
-                            action: path('add_maintainer', { 'name': package.name })
-                        }) }}
-                            <div>
-                                <h4>Add Maintainer</h4>
-                                <p>
-                                    {{ form_label(addMaintainerForm.user, "Username") }}
-                                    {{ form_errors(addMaintainerForm.user) }}
-                                    {{ form_widget(addMaintainerForm.user) }}
-                                </p>
-                                {{ form_rest(addMaintainerForm) }}
-                                <input class="btn btn-block btn-success btn-lg" type="submit" value="Add Maintainer" />
-                            </div>
-                        {{ form_end(addMaintainerForm) }}
-                    {% endif %}
-
-                    {% if is_granted('remove_maintainer', package) %}
-                        {{ form_start(removeMaintainerForm, {
-                            attr: { id: 'remove-maintainer-form', class: 'col-sm-6 col-md-3 col-md-offset-9 ' ~ (show_remove_maintainer_form|default(false) ? '': 'hidden') },
-                            action: path('remove_maintainer', { 'name': package.name })
-                        }) }}
-                            <div>
-                                <h4>Remove Maintainer</h4>
-                                <p>
-                                    {{ form_label(removeMaintainerForm.user, "Username") }}
-                                    {{ form_errors(removeMaintainerForm.user) }}
-                                    {{ form_widget(removeMaintainerForm.user) }}
-                                </p>
-                                {{ form_rest(removeMaintainerForm) }}
-                                <input class="btn btn-block btn-danger btn-lg" type="submit" value="Remove Maintainer" />
-                            </div>
-                        {{ form_end(removeMaintainerForm) }}
-                    {% endif %}
-                </div>
-            {% endif %}
-
-            {% if is_granted('transfer_package', package) %}
-                <div class="row">
-                    {{ form_start(transferPackageForm, {
-                        attr: {
-                            id: 'transfer-package-form',
-                            class: 'col-sm-6 col-md-3 col-md-offset-9 ' ~ (show_transfer_package_form|default(false) ? '': 'hidden'),
-                        },
-                        action: path('transfer_package', { 'name': package.name })
-                    }) }}
-                        <div>
-                            <h4>Transfer Ownership</h4>
-                            <div class="maintainers-collection-wrapper">
-                                {{ form_errors(transferPackageForm.maintainers) }}
-                                <ul class="maintainers-list" data-prototype="{{ form_widget(transferPackageForm.maintainers.vars.prototype)|e('html_attr') }}">
-                                    {% for maintainerField in transferPackageForm.maintainers %}
-                                        <li>
-                                            {{ form_errors(maintainerField) }}
-                                            {{ form_widget(maintainerField) }}
-                                        </li>
-                                    {% endfor %}
-                                </ul>
-                                <button type="button" class="btn btn-info btn-sm add-maintainer-item">Add Maintainer</button>
-                            </div>
-                            {{ form_rest(transferPackageForm) }}
-                            <input class="btn btn-block btn-success btn-lg" type="submit" value="Transfer Ownership" />
-                        </div>
-                    {{ form_end(transferPackageForm) }}
-                </div>
-            {% endif %}
-
             {% if versions|length %}
                 <div class="row versions-section">
                     <div class="version-details col-md-9">