Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report types listed in a modal @ report plugins #3718

Merged
merged 21 commits into from
Nov 20, 2024
Merged

Report types listed in a modal @ report plugins #3718

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
989280e
Report types as dropdown list added for report plugins
Rieven Oct 23, 2024
ab0c8ef
remove redundant code
Rieven Oct 23, 2024
86a564e
Merge branch 'main' into feature/report-plugins-enhacement
Rieven Oct 23, 2024
a497a96
fix lang
Rieven Oct 23, 2024
705a3c4
fix html accessibility
Rieven Oct 23, 2024
9e59a78
Merge branch 'main' into feature/report-plugins-enhacement
Rieven Oct 24, 2024
c11d5bd
Merge branch 'main' into feature/report-plugins-enhacement
stephanie0x00 Oct 28, 2024
952ea9e
report types in modal
Rieven Nov 5, 2024
9e65c1a
Merge branch 'feature/report-plugins-enhacement' of github.com:minvws…
Rieven Nov 5, 2024
ea0ed19
fix link styling
Rieven Nov 6, 2024
0c969c6
remove margin
Rieven Nov 6, 2024
dff20db
00~Merge branch 'main' of github.com:minvws/nl-kat-coordination into …
Rieven Nov 6, 2024
8ae4e41
fix lang
Rieven Nov 6, 2024
5c52731
Merge branch 'main' of github.com:minvws/nl-kat-coordination into fea…
Rieven Nov 13, 2024
5f40420
Merge branch 'main' into feature/report-plugins-enhacement
underdarknl Nov 18, 2024
4cd924c
Merge branch 'main' into feature/report-plugins-enhacement
Rieven Nov 18, 2024
f20abd1
fixes for li labels
Rieven Nov 18, 2024
dad909a
remove some inline flexing
Rieven Nov 18, 2024
084643c
fix lang
Rieven Nov 18, 2024
b6901ff
fix padding
Rieven Nov 18, 2024
4914dbb
Merge branch 'main' into feature/report-plugins-enhacement
Rieven Nov 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions rocky/assets/css/components/report.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,8 +110,6 @@ span {
padding: var(--spacing-grid-050) var(--spacing-grid-100);
font-size: var(--tile-font-size);
font-weight: var(--list-item-completed-font-weight);
margin: var(--spacing-grid-050) var(--spacing-grid-100)
var(--spacing-grid-050) 0;

&.tags-color-grey-2 {
color: var(--colors-grey-800);
Expand Down
28 changes: 28 additions & 0 deletions rocky/assets/css/vendor_overrides/manon/tile.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,27 @@ ul.tiles {

&.plugins {
gap: 1.25rem;

.dropdown {
max-width: 100%;
border: none;
padding: 0;

.dropdown-list {
li {
padding: 0.5rem;
font-size: var(--tile-font-size);
}
}

button {
color: var(--colors-blue-600);
border: none;
border-radius: 0;
padding: 0.25rem 0 1rem;
font-size: var(--tile-font-size);
}
}
}

@extend %tile-style-overrides;
Expand Down Expand Up @@ -80,4 +101,11 @@ ul.tiles {
flex-direction: row;
flex-wrap: wrap;
gap: 0.5rem;
border: 0;
padding: 0;

p {
color: var(--colors-blue-600);
padding-left: 0;
}
}
38 changes: 27 additions & 11 deletions rocky/katalogus/templates/partials/plugin_tile.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,18 +30,34 @@
{% endif %}
<span class="plugin-author-link de-emphasized"><a target="_blank" href="https://openkat.nl" rel="noopener noreferrer">OpenKAT</a></span>
{% if show_report_types %}
<div class="tags">
<p>
<strong>{% translate "Required for:" %}</strong>
</p>
{% for plugin_id, report_types in plugin_report_types.items %}
{% if plugin_id == plugin.id %}
{% for report_type in report_types %}
<span class="label tags-color-{{ report_type.label_style }}">{{ report_type.name }}</span>
{% endfor %}
{% for plugin_id, report_types in plugin_report_types.items %}
{% if plugin_id == plugin.id %}
{% if report_types|length >= 2 %}
<div class="dropdown">
<button aria-controls="plugin-report-types"
aria-expanded="false"
class="dropdown-button ghost">
{% translate "Required for" %} {{ report_types|length }} {% translate "report types" %}
<span aria-hidden="true" class="icon ti-chevron-right"></span>
</button>
<div class="dropdown-list" id="plugin-report-types">
<ul>
{% for report_type in report_types %}<li>{{ report_type.name }}</li>{% endfor %}
</ul>
</div>
underdarknl marked this conversation as resolved.
Show resolved Hide resolved
</div>
{% else %}
<div class="tags">
<p>
<strong>{% translate "Required for: " %}</strong>
</p>
{% for report_type in report_types %}
Rieven marked this conversation as resolved.
Show resolved Hide resolved
<span class="label tags-color-{{ report_type.label_style }}">{{ report_type.name }}</span>
{% endfor %}
</div>
{% endif %}
{% endfor %}
</div>
{% endif %}
{% endfor %}
{% endif %}
{% if not remove_action_buttons %}
<div class="action-buttons">
Expand Down
10 changes: 9 additions & 1 deletion rocky/rocky/locale/django.pot
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1081,7 +1081,15 @@ msgid "Add"
msgstr ""

#: katalogus/templates/partials/plugin_tile.html
msgid "Required for:"
msgid "Required for"
msgstr ""

#: katalogus/templates/partials/plugin_tile.html
msgid "report types"
msgstr ""

#: katalogus/templates/partials/plugin_tile.html
msgid "Required for: "
msgstr ""

#: katalogus/templates/partials/plugin_tile_modal.html
Expand Down