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

feat: overhaul sidebar #274

Merged
merged 1 commit into from
Oct 23, 2024
Merged

Conversation

tylerslaton
Copy link
Contributor

@tylerslaton tylerslaton commented Oct 22, 2024

The sidebar now uses the ShadCN component. This reduces code complexity by a lot. Throughout the applications the various components for this are used and overall styling has changed quite a bit.

As such, the old sidebar components are no longer needed and are deleted.

In addition, the header now has breadcrumbs which are used to display the current page in a tree format replacing the previous method of just displaying a heading.

Finally, the sidebar has been updated to have a settings menu which contains a link to the OAuth applications page.

Screen.Recording.2024-10-23.at.12.53.33.PM.mov

The sidebar now uses the ShadCN component. This
reduces code complexity by a lot. Throughout the
applications the various components for this are
used and overall styling has changed quite a bit.

As such, the old sidebar components are no longer
needed and are deleted.

In addition, the header now has breadcrumbs which
are used to display the current page in a tree
format replacing the previous method of just
displaying a heading.

Finally, the sidebar has been updated to have a
settings menu which contains a link to the OAuth
applications page.

Signed-off-by: tylerslaton <[email protected]>
@tylerslaton tylerslaton marked this pull request as ready for review October 23, 2024 16:54
@tylerslaton tylerslaton merged commit e15fd92 into obot-platform:main Oct 23, 2024
1 check passed
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.

2 participants