Skip to content

Circle info for make active#2544

Open
AlexanderSchicktanz wants to merge 9 commits intoe-valuation:mainfrom
AlexanderSchicktanz:CircleInfo_for_MakeActive
Open

Circle info for make active#2544
AlexanderSchicktanz wants to merge 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

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