Skip to content

Circle info for make active#2544

Merged
janno42 merged 9 commits intoe-valuation:mainfrom
AlexanderSchicktanz:CircleInfo_for_MakeActive
Apr 27, 2026
Merged

Circle info for make active#2544
janno42 merged 9 commits intoe-valuation:mainfrom
AlexanderSchicktanz:CircleInfo_for_MakeActive

Conversation

@AlexanderSchicktanz
Copy link
Copy Markdown
Collaborator

fix #2538

Pop-Up Info at hover for "Make active"

@niklasmohrin
Copy link
Copy Markdown
Member

This PR currently contains the changes both for this issue, but also from your other PR #2540. Can you rebase this branch onto the main branch from our repo? If you are unsure what to do, that's no problem, someone can help you at our meeting on Monday

Comment thread evap/static/ts/src/text-answer-warnings.ts
Comment thread evap/staff/templates/staff_semester_view.html Outdated
Comment thread evap/staff/templates/staff_semester_view.html Outdated
Comment thread evap/staff/templates/staff_semester_view.html Outdated
Comment thread evap/staff/templates/staff_semester_view.html
@niklasmohrin niklasmohrin requested a review from janno42 November 10, 2025 18:39
Comment thread evap/staff/templates/staff_semester_view.html Outdated
Comment thread evap/staff/templates/staff_semester_view.html Outdated
Comment thread evap/staff/templates/staff_semester_view.html Outdated
@niklasmohrin
Copy link
Copy Markdown
Member

@AlexanderSchicktanz Do you want to continue to work on this, or should we unassign you from the issue?

@niklasmohrin
Copy link
Copy Markdown
Member

@janno42 I committed my suggested (whitespace) change, so we are good to go here

@janno42 janno42 merged commit 6851df0 into e-valuation:main Apr 27, 2026
15 of 16 checks passed
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.

Active semester info

5 participants