Submit GH bug: ask the user to do some checks before submitting a new issue

This commit is contained in:
Alexandre Flament 2022-07-03 11:03:20 +02:00
parent cf01f9e603
commit eb489c45ea
4 changed files with 100 additions and 34 deletions

View file

@ -1,21 +1,48 @@
// SPDX-License-Identifier: AGPL-3.0-or-later // SPDX-License-Identifier: AGPL-3.0-or-later
.stats_endpoint {
.github-issue-button {
display: block;
padding: 8px 16px;
font-family: sans-serif;
font-size: 16px;
color: white;
background-color: #238636;
border: #2ea043;
border-radius: 10px !important;
box-shadow: rgba(0, 0, 0, 0) 0 0 0 0;
}
.github-issue-button { .github-issue-button:hover {
display: block; background-color: #2ea043;
padding: 8px 16px; }
font-family: sans-serif;
font-size: 16px;
color: white;
background-color: #238636;
border: #2ea043;
border-radius: 10px !important;
box-shadow: rgba(0, 0, 0, 0) 0 0 0 0;
}
.github-issue-button:hover { .issue-hide {
background-color: #2ea043; display: none;
} }
.issue-hide { input[type=checked] {
display: none; position: absolute;
}
label {
margin: 1rem 1rem 1rem 0;
}
.step_content {
margin: 1rem 1rem 1rem 2rem;
}
.step1,
.step2 {
visibility: hidden;
}
.step1_delay {
transition: visibility 0s linear 4s;
}
#step1:checked ~ .step1,
#step2:checked ~ .step2 {
visibility: visible;
}
} }

View file

@ -1,6 +1,25 @@
// SPDX-License-Identifier: AGPL-3.0-or-later // SPDX-License-Identifier: AGPL-3.0-or-later
.engine-stats { .engine-stats {
border-spacing: 0;
border-collapse: collapse;
tr td,
tr th {
border-bottom: 1px solid var(--color-result-border);
padding: 0.25rem;
}
table.engine-tooltip {
border-spacing: 0;
border-collapse: collapse;
td,
th {
border: none;
}
}
.engine-name { .engine-name {
width: 20rem; width: 20rem;
} }
@ -13,23 +32,33 @@
.engine-reliability { .engine-reliability {
text-align: right; text-align: right;
} }
}
table.engine-error th.engine-error-type,
table.engine-error td.engine-error-type,
failed-test {
width: 10rem;
}
.engine-errors {
margin-top: 3rem;
table.engine-error { table.engine-error {
max-width: 1280px; max-width: 1280px;
margin: 1rem; margin: 1rem 0 3rem 0;
border: 1px solid gray; border: 1px solid var(--color-result-border);
} .ltr-text-align-left();
table.engine-error th.engine-error-type, tr th,
table.engine-error td.engine-error-type, tr td {
failed-test { padding: 0.5rem;
width: 10rem; }
}
table.engine-error span.log_parameters { & span.log_parameters {
border-right: 1px solid gray; border-right: 1px solid solid var(--color-result-border);
padding: 0 1rem 0 0; padding: 0 1rem 0 0;
margin: 0 0 0 0.5rem; margin: 0 0 0 0.5rem;
}
} }
} }

View file

@ -60,6 +60,16 @@ or manually by executing the searx/webapp.py file? -->
{{-'\n '}}* {{ test_name }}: {% for result in results%}`{{ result }}`,{% endfor -%} {{-'\n '}}* {{ test_name }}: {% for result in results%}`{{ result }}`,{% endfor -%}
{%- endfor -%} {%- endfor -%}
</textarea> </textarea>
<button type="submit" class="github-issue-button" title="{{ get_setting('brand.new_issue_url') }}">{{ _('Submit a new issue on Github including the above information') }}</button> <input type="checkbox" id="step1">
<label for="step1">{{ _('Start submiting a new issue on GitHub') }}</label>
<div class="step1 step_content">
<p><a href="https://github.com/searxng/searxng/issues?q=is%3Aissue+Bug:%20{{ engine_name }}" target="_blank" rel="noreferrer noreferrer">{{ _('Please check for existing bugs about this engine on GitHub') }}</a></p>
</div>
<input class="step1 step1_delay" type="checkbox" id="step2">
<label class="step1 step1_delay" for="step2" >{{ _('I confirm there is no existing bug about the issue I encounter') }}</label>
<div class="step2 step_content">
<p>{{ _('If this is a public instance, please specify the URL in the bug report') }}</p>
<button type="submit" class="github-issue-button" title="{{ get_setting('brand.new_issue_url') }}">{{ _('Submit a new issue on Github including the above information') }}</button>
</div>
</form> </form>
{% endmacro %} {% endmacro %}

View file

@ -91,15 +91,15 @@
</table> </table>
{% endif %} {% endif %}
<div> {% if selected_engine_name %}
{% if selected_engine_name %} <div class="engine-errors">
{% for secondary in [False, True] %} {% for secondary in [False, True] %}
{% set ns = namespace(first=true) %} {% set ns = namespace(first=true) %}
{% for error in engine_reliabilities[selected_engine_name].errors %} {% for error in engine_reliabilities[selected_engine_name].errors %}
{% if secondary == error.secondary %} {% if secondary == error.secondary %}
{% if ns.first %} {% if ns.first %}
{% set ns.first = false %} {% set ns.first = false %}
<h3>{% if secondary %}{{ _('Warnings') }}{% else %}{{ _('Errors and exceptions') }}{% endif %}</h3> <h2>{% if secondary %}{{ _('Warnings') }}{% else %}{{ _('Errors and exceptions') }}{% endif %}</h2>
{% endif %} {% endif %}
<table class="engine-error"> <table class="engine-error">
<tbody> <tbody>
@ -145,7 +145,7 @@
</table> </table>
{% endif %} {% endif %}
{{ new_issue(selected_engine_name, engine_reliabilities[selected_engine_name]) }} {{ new_issue(selected_engine_name, engine_reliabilities[selected_engine_name]) }}
{% endif %} </div>
</div> {% endif %}
{% endblock %} {% endblock %}