heckyel/mediagoblin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove the slightly un-intuitive "Resolve" button on the report handling page.

Browse Source 
This button just uses JavaScript to reveal the subsequent actions. Better to
show these upfront.

Signed-off-by: Ben Sturmfels <ben@sturm.com.au>
This commit is contained in:
Rodrigo Martins 2021-04-10 15:24:11 +10:00 committed by Ben Sturmfels
parent 93084dfe28
commit 9bcebe68c7
No known key found for this signature in database
GPG Key ID: 023C05E2C9C068F0
2 changed files with 20 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
mediagoblin/static/js/setup_report_forms.js
View File

@ -24,14 +24,12 @@ function init_report_resolution_form() {
} }
init_user_banned_form(); init_user_banned_form();
$('form#resolution_form').hide() $('form#resolution_form').hide()
$('#open_resolution_form').click(function() { $('form#resolution_form').toggle();
$('form#resolution_form').toggle(); $.each(hidden_input_names, function(key, list){
$.each(hidden_input_names, function(key, list){ $.each(list, function(index, name){
$.each(list, function(index, name){ $('label[for='+name+']').hide();
$('label[for='+name+']').hide(); $('#'+name).hide();
$('#'+name).hide(); });
});
});
}); });
$('#action_to_resolve').change(function() { $('#action_to_resolve').change(function() {
$('ul#action_to_resolve li input:checked').each(function() { $('ul#action_to_resolve li input:checked').each(function() {

30
mediagoblin/templates/mediagoblin/moderation/report.html
View File

@ -128,21 +128,7 @@
{{ report.report_content }} {{ report.report_content }}
</div> </div>
</div> </div>
{% if not report.is_archived_report() and {% if report.is_archived_report() %}
not (report.reported_user.has_privilege('admin') and
not request.user.has_privilege('admin')) %}
<input type=button class="button_action" value="{% trans %}Resolve{% endtrans %}" id=open_resolution_form />
<form action="" method="POST" id=resolution_form>
{{ wtforms_util.render_divs(form) }}
{{ csrf_token }}
<input type=submit class="button_action_highlight" id="submit_this_report" value="{% trans %}Resolve This Report{% endtrans %}"/>
</form>
<script>
$(document).ready(function() {
init_report_resolution_form();
});
</script>
{% elif report.is_archived_report() %}
<h2><img src="{{ request.staticdirect('/images/icon_clipboard.png') }}" <h2><img src="{{ request.staticdirect('/images/icon_clipboard.png') }}"
alt="" alt=""
title="Under a GNU LGPL v.3 or Creative Commons BY-SA 3.0 license. title="Under a GNU LGPL v.3 or Creative Commons BY-SA 3.0 license.
@ -154,13 +140,25 @@
<pre> <pre>
<p>{{ report.result }}</p> <p>{{ report.result }}</p>
</pre> </pre>
{% else %} {% elif report.reported_user.has_privilege('admin')
and not request.user.has_privilege('admin') %}
<input type=button disabled=disabled value="{% trans %}Resolve This Report{% endtrans %}"/> <input type=button disabled=disabled value="{% trans %}Resolve This Report{% endtrans %}"/>
<p> <p>
{% trans -%} {% trans -%}
You cannot take action against an administrator You cannot take action against an administrator
{%- endtrans %} {%- endtrans %}
</p> </p>
{% else %}
<form action="" method="POST" id=resolution_form>
{{ wtforms_util.render_divs(form) }}
{{ csrf_token }}
<input type=submit class="button_action_highlight" id="submit_this_report" value="{% trans %}Resolve This Report{% endtrans %}"/>
</form>
<script>
$(document).ready(function() {
init_report_resolution_form();
});
</script>
{% endif %} {% endif %}
{% endif %} {% endif %}
{% endblock %} {% endblock %}