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

Added close button to settings - working with browser history #1204

Conversation

SebiWrn
Copy link
Contributor

@SebiWrn SebiWrn commented Oct 23, 2023

Motivation and Context

Fixes #1181

Description

Added a Button to exit the settings page. Currently uses browser history but can be extended to use custom history.

Steps for Testing

  • Log into any account
  • Open settings page of account
  • Press the small "x" at the top right of the page

Prerequisites:

  • 1 Account

Screenshots

exit_button_settings

@SebiWrn SebiWrn linked an issue Oct 23, 2023 that may be closed by this pull request
@github-actions
Copy link

Your Testserver will be ready at https://1204.test.live.mm.rbg.tum.de in a few minutes.

Logins
Kurs1 Kurs2 Kurs3 Kurs4
public public loggedin enrolled
prof1 prof1 prof2 prof1
prof2
student1
student2
student3
student1
student2
student2
student3
student1
student2

Copy link
Collaborator

@alexanderstephan alexanderstephan left a comment

Choose a reason for hiding this comment

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

Code looks and works nicely as well! :)

@alexanderstephan
Copy link
Collaborator

alexanderstephan commented Oct 25, 2023

Btw, you might want to set your git config properly. As no email is set, your commit is not linked to your account.
Edit: Hm, okay, another email is Co-Author, but you still might wanna take a look.

@alexanderstephan alexanderstephan merged commit e337d87 into dev Oct 25, 2023
9 checks passed
@alexanderstephan alexanderstephan deleted the 1181-button-to-quit-the-settings-at-httpsliverbgtumdesettings branch October 25, 2023 19:06
YiranDuan721 pushed a commit that referenced this pull request Oct 25, 2023
SebiWrn added a commit that referenced this pull request May 7, 2024
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.

Button to quit the settings at https://live.rbg.tum.de/settings
2 participants