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

Switch activity subscription from well to modal #1765

Merged

Conversation

tomudding
Copy link
Member

To make it easier to subscribe to activities, the subscription form is now a modal. This prevents the page from scrolling and allows for some magic auto re-openings of the modal if there is an error.

This also changes the states the activity subscription button can be in to ensure that it is easier to see what will happen.

Can subscribe as GEWIS member (logged in)
image
image

Can subscribe as GEWIS member (logged out)
image

Can subscribe as external participant
image
image

Subscribed and can unsubscribe
image
image

Subscribed and cannot unsubscribe
image

Not subscribed and cannot (un)subscribe
image

Closes GH-1752.

To make it easier to subscribe to activities, the subscription form
is now a modal. This prevents the page from scrolling and allows
for some magic auto re-openings of the modal if there is an error.

This also changes the states the activity subscription button can
be in to ensure that it is easier to see what will happen.
@tomudding tomudding merged commit 22e8d95 into GEWIS:main Nov 19, 2023
4 checks passed
github-actions bot added a commit that referenced this pull request Nov 19, 2023
Tom Udding: Merge pull request #1765 from tomudding/feature/activity-sign-up-through-modal

Switch activity subscription from well to modal

Co-authored-by: tomudding <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move sign up 'well' to modal
1 participant