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.

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

Change color of submit error message #2034

Merged
merged 2 commits into from
Oct 22, 2023
Merged

Conversation

joendter
Copy link
Collaborator

@joendter joendter commented Oct 16, 2023

fix #2026

@joendter joendter changed the title fix #2026 Change color of submit error message Oct 16, 2023
@@ -154,7 +154,7 @@ <h3 class="mb-3">{{ evaluation.full_name }} ({{ evaluation.course.semester.name
<span class="fas fa-circle-info" data-bs-toggle="tooltip" title="{% trans 'The evaluation can be continued later using the same device and the same browser. But you have to submit it to send it to the server and make it count. After submitting, you can not edit the evaluation anymore.' %}"></span>
</div>
<button id="vote-submit-btn" type="submit" class="btn btn-success tab-selectable">{% trans 'Submit questionnaire' %}</button>
<p id="submit-error-warning" style="display: none">
<p id="submit-error-warning" style="display: none", class="text-danger">
Copy link
Member

@richardebeling richardebeling Oct 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HTML attributes are typically separated just by whitespace. I don't know if the comma is legal here, but for consistency in the code base, I'd rather have no comma here.

Copy link
Member

@janno42 janno42 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✔️ Meets requirements
✔️ UI functionality checked

@janno42 janno42 merged commit 2e780a3 into e-valuation:main Oct 22, 2023
@janno42
Copy link
Member

janno42 commented Oct 22, 2023

Thank you for your first contribution to EvaP! 🎆

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Change color of submit error message
3 participants