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.

Sign up for GitHub

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

Jump to bottom

Redesign settings pages #91

Merged
merged 21 commits into from
Apr 5, 2024
Merged

Redesign settings pages #91

merged 21 commits into from
Apr 5, 2024

Conversation

markfijneman
Copy link

No description provided.

@markfijneman
Set indicator color of all tabs to secondary
18fdcb4
@markfijneman
Display admin settings page in a single column on small screens
13f2f4d
@markfijneman
Adjust padding of cards in registration settings
5e4c5ce
@markfijneman
Style inputs in registration settings
1e196da
@markfijneman
Re-organize columns on registration settings to group registration an…
b54e53c 
…d calendar settings separately
@markfijneman
Hide inputs for registration and calendar passwords when disabled
2932733
@markfijneman
Adjust padding of cards in profile settings
82c45a7
@markfijneman
Add icon support to color picker button
52f85c5
@markfijneman
Style inputs in profile settings
3ed2a6e
@markfijneman
Moved Change password to left column in profile settings
61261d9
@markfijneman
Fix color picker button being too large in some cases
582d4a4
@markfijneman
Adjust padding for theme settings
64424c9
@markfijneman
Update styling for registered users settings page
db59339
@markfijneman
Removed borders from cards in settings pages
b4e84d2
@markfijneman
Added icons to notifications toggle
82976b9
@markfijneman
Added Discord logo to CTFNote token field
c6b8338
@markfijneman
Make color picker icon appear white in light mode
7483b5f
@markfijneman
Merge branch 'JJ-8:main' into settings-styling
e641328
JJ-8
JJ-8 requested changes Mar 24, 2024
View reviewed changes
Copy link
Owner

@JJ-8 JJ-8 left a comment

Choose a reason for hiding this comment

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

Very nice that everything is becoming more consistent! A few small remarks remain.

front/src/components/Admin/Users.vue Outdated Show resolved Hide resolved
front/src/components/Admin/Users.vue Outdated Show resolved Hide resolved
front/src/pages/Settings.vue Show resolved Hide resolved
@markfijneman markfijneman requested a review from JJ-8 March 27, 2024 12:09
@JJ-8 JJ-8 merged commit 2636be1 into JJ-8:main Apr 5, 2024
@markfijneman markfijneman deleted the settings-styling branch April 15, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JJ-8 JJ-8 JJ-8 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@markfijneman @JJ-8