fix ProjectRoleView tour help attachments (#1512)

bihealth · Nov 12, 2024 · 56b4bbb · 56b4bbb
1 parent 5865aea
commit 56b4bbb
Show file tree

Hide file tree

Showing 5 changed files with 12 additions and 9 deletions.
diff --git a/CHANGELOG.rst b/CHANGELOG.rst
@@ -31,6 +31,7 @@ Fixed
 - **Projectroles**
     - Deprecated ``SODAR_API_*`` settings required in tests (#1495)
     - Add workaround to ``ProjectInviteCreateView`` returning 404 with category and query string (#1510)
+    - Broken tour help attachments in ``ProjectRoleView`` (#1512)
 
 
 v1.0.2 (2024-09-09)

diff --git a/projectroles/templates/projectroles/_project_role_dropdown.html b/projectroles/templates/projectroles/_project_role_dropdown.html
@@ -2,7 +2,8 @@
 {% load projectroles_common_tags %}
 {% load projectroles_role_tags %}
 
-<button class="btn btn-secondary dropdown-toggle sodar-list-dropdown"
+<button class="btn btn-secondary dropdown-toggle sodar-list-dropdown
+               sodar-pr-role-btn"
         type="button" data-toggle="dropdown"
         aria-haspopup="true" aria-expanded="false">
   <i class="iconify" data-icon="mdi:gear"></i>

diff --git a/projectroles/templates/projectroles/_project_role_dropdown_owner.html b/projectroles/templates/projectroles/_project_role_dropdown_owner.html
@@ -1,7 +1,8 @@
 {% load rules %}
 {% load projectroles_common_tags %}
 
-<button class="btn btn-secondary dropdown-toggle sodar-list-dropdown"
+<button class="btn btn-secondary dropdown-toggle sodar-list-dropdown
+               sodar-pr-role-btn-owner"
         type="button" data-toggle="dropdown"
         aria-haspopup="true" aria-expanded="false">
   <i class="iconify" data-icon="mdi:gear"></i>

diff --git a/projectroles/templates/projectroles/_project_role_ops.html b/projectroles/templates/projectroles/_project_role_ops.html
@@ -1,5 +1,5 @@
 <div class="ml-auto">
-  <button class="btn btn-primary dropdown-toggle"
+  <button class="btn btn-primary dropdown-toggle sodar-pr-role-ops-btn"
           id="sodar-pr-role-ops-btn"
           type="button" data-toggle="dropdown"
           aria-haspopup="true" aria-expanded="false"

diff --git a/projectroles/templates/projectroles/project_roles.html b/projectroles/templates/projectroles/project_roles.html
@@ -141,17 +141,17 @@ <h3>
             showCancelLink: true
         });
     }
-    if ($('.sodar-pr-role-dropdown-owner').length) {
+    if ($('.sodar-pr-role-btn-owner').length) {
         tour.addStep('role_dropdown_owner', {
             title: 'Owner Dropdown',
             text: 'Ownership can be transferred to another user from this ' +
                   'dropdown.',
-            attachTo: '.sodar-pr-role-dropdown-owner left',
+            attachTo: '.sodar-pr-role-btn-owner left',
             advanceOn: '.docs-link click',
             showCancelLink: true,
         });
     }
-    if ($('.sodar-pr-role-dropdown').length) {
+    if ($('.sodar-pr-role-btn').length) {
         tour.addStep('role_dropdown', {
             title: 'Member Dropdown',
             text: 'For a non-owner member, you can update or remove the ' +
@@ -160,18 +160,18 @@ <h3>
                   'allowed. To remove such a role, navigate to the parent ' +
                   '{% get_display_name 'CATEGORY' %} and remove the ' +
                   'membership there.',
-            attachTo: '.sodar-pr-role-dropdown left',
+            attachTo: '.sodar-pr-role-btn left',
             advanceOn: '.docs-link click',
             showCancelLink: true,
         });
     }
-    if ($('#sodar-pr-role-ops-dropdown').length) {
+    if ($('#sodar-pr-role-ops-btn').length) {
         tour.addStep('role_ops', {
             title: 'Member Operations Dropdown',
             text: 'From this dropdown you can access operations for adding ' +
                   'new members to the {% get_display_name 'PROJECT' %} and ' +
                   'inviting users not yet on the site.',
-            attachTo: '#sodar-pr-role-ops-dropdown left',
+            attachTo: '#sodar-pr-role-ops-btn left',
             advanceOn: '.docs-link click',
             showCancelLink: true
         });