Skip to content

Commit 367d327

Browse files
committed
SDG Issuelib - code review tweaks
1 parent 11bab68 commit 367d327

File tree

8 files changed

+25
-15
lines changed

8 files changed

+25
-15
lines changed

app/assets/javascripts/hera/behaviors.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,7 @@ document.addEventListener('turbo:load', function () {
1616
});
1717

1818
// If project id is changed in project path
19-
if (
20-
!/^\/projects\/1(\/|$)/.test(window.location.pathname) &&
21-
!/^\/addons\/issuelib(\/|$)/.test(window.location.pathname)
22-
) {
19+
if (!/^\/projects\/1(\/|$)/.test(window.location.pathname)) {
2320
$('[data-behavior~=project-teaser]').removeClass('d-none');
2421
}
2522

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
class StaticPagesController < AuthenticatedController
22
include ProjectScoped
33

4-
def issuelib; end
4+
def issuelib_index; end
55

6-
def import; end
6+
def issuelib_import; end
77
end

app/helpers/hera_helper.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ def flash_messages
6161

6262
def navbar_brand
6363
if !defined?(Dradis::Pro)
64-
link_to main_app.root_path, class: 'navbar-brand' do
64+
link_to main_app.project_path(current_project), class: 'navbar-brand' do
6565
image_tag 'logo_small.png', alt: 'Dradis CE logo', class: 'p-lg-0'
6666
end
6767
else

app/views/issues/_empty_import_ce.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<ul class="fa-ul">
33
<li><i class="fa-solid fa-li fa-chevron-right"></i><a href="https://dradis.com/integrations/mediawiki.html?utm_source=ce&utm_medium=app" target="_blank">MediaWiki</a></li>
44
<li><i class="fa-solid fa-li fa-chevron-right"></i>
5-
<%= link_to projects_1_issue_import_path do %>
5+
<%= link_to static_issuelib_path do %>
66
<i class="fa-solid fa-book"></i> Built-in IssueLibrary
77
<% end %>
88
</li>

app/views/issues/_import_box.html.erb

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,22 @@
1919
</p>
2020
<p class="mb-2">Libraries:</p>
2121
<ul class="fa-ul">
22-
<li><i class="fa-solid fa-li fa-chevron-right"></i><a href="https://dradis.com/integrations/mediawiki.html?utm_source=ce&utm_medium=app" target="_blank">MediaWiki</a></li>
2322
<li><i class="fa-solid fa-li fa-chevron-right"></i>
24-
<%= link_to projects_1_issue_import_path do %>
23+
<a href="https://dradis.com/integrations/mediawiki.html?utm_source=ce&utm_medium=app" target="_blank">
24+
MediaWiki
25+
</a>
26+
</li>
27+
<li><i class="fa-solid fa-li fa-chevron-right"></i>
28+
<%= link_to static_issuelib_path do %>
2529
<i class="fa-solid fa-book"></i> Built-in Issue Library
2630
<% end %>
2731
</li>
2832
</ul>
33+
<form action="<%= static_issuelib_import_path %>" class="form-inline mb-4" method="get">
34+
<div class="inset-button-wrapper">
35+
<input type="text" class="form-control search-query", placeholder="Search Issue Library" />
36+
</div>
37+
</form>
2938
<% else %>
3039
<% Dradis::Plugins::with_feature(:import).each do |plugin| %>
3140
<div>

app/views/layouts/hera/navbar/main_nav/_tools_menu.html.erb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,10 @@
55
</a>
66

77
<ul class="dropdown-menu" aria-labelledby="tools-dropdown">
8+
<%= render_view_hooks('tools_menu') %>
89
<% if !defined?(Dradis::Pro) %>
9-
<li><%= link_to 'Issue Library', addons_issuelib_path, class: 'dropdown-item' %></li>
10+
<li><div class="dropdown-divider"></div></li>
11+
<li><%= link_to 'Issue Library', static_issuelib_path, class: 'dropdown-item' %></li>
1012
<% end %>
11-
<%= render_view_hooks('tools_menu') %>
1213
</ul>
1314
</li>

app/views/static_pages/import.html.erb renamed to app/views/static_pages/issuelib_import.html.erb

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@
44
<li class="breadcrumb-item">
55
<%= link_to 'Dashboard', main_app.root_path %>
66
</li>
7+
<li class="breadcrumb-item">
8+
<%= link_to 'Issue Library', main_app.static_issuelib_path %>
9+
</li>
710
<li class="breadcrumb-item active">
811
Import Issues
912
</li>

app/views/static_pages/issuelib.html.erb renamed to app/views/static_pages/issuelib_index.html.erb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,17 +20,17 @@
2020
</div>
2121
<div id="il-page-tips" class="page-tips show collapse">
2222
<div class="bg-info">
23-
<p>Maintain a library of issues that can be easily accessed, and achieve consistency across your projects.</p>
23+
<p>Maintain a library of findings that can be easily accessed, and achieve consistency across your projects.</p>
2424
<p>Authors can be given permission to access or manage the Issue Library.</p>
25-
<p class="text-end card-link">More about <a href="https://dradis.com/support/guides/issuelibrary/" target="_blank">Issue Library</a>.</p>
25+
<p class="text-end card-link">More about <a href="https://dradis.com/tools/issuelib.html" target="_blank">Issue Library</a>.</p>
2626
</div>
2727
</div>
2828
<% end %>
2929

3030
<div class="row d-flex align-items-start justify-content-between">
3131
<div class="col-12 col-md-9">
3232
<h1>Build Your Issue Library</h1>
33-
<h2 class="mb-3">Maintain reusable issues to ensure consistency across all of your projects</h2>
33+
<h2 class="mb-3">Maintain reusable findings to ensure consistency across all of your projects</h2>
3434
</div>
3535
<div class="col-12 col-md-3 d-flex justify-content-start justify-content-md-end mb-4 mb-md-0">
3636
<a

0 commit comments

Comments
 (0)