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

Update dashboard header layout and tab component #4063

Merged
merged 3 commits into from
Feb 15, 2024

Conversation

briangregoryholmes
Copy link
Contributor

@briangregoryholmes briangregoryholmes commented Feb 15, 2024

The purpose of this PR was to simply add a Beta tag to the Pivot tab on the dashboard, but turned into one that refactors the Dashboard tab component with native functionality instead of extending headless-ui, whose limited functionality was presenting some issues. Additionally, this PR solves some layout oversights in the header component.

@briangregoryholmes briangregoryholmes changed the title update dashboard header and tab component Update dashboard header layout and tab component Feb 15, 2024
@briangregoryholmes briangregoryholmes self-assigned this Feb 15, 2024
@@ -97,20 +95,17 @@
{#if mockUserHasNoAccess}
<div class="mb-3" />
{:else}
<div class="-ml-3 px-1 pt-2 space-y-2">
<div class="-ml-3 px- pt-2 space-y-2">
Copy link
Contributor

Choose a reason for hiding this comment

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

px- looks like a typo

Copy link
Contributor

@ericpgreen2 ericpgreen2 left a comment

Choose a reason for hiding this comment

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

Looks good!

@briangregoryholmes briangregoryholmes merged commit 062b151 into main Feb 15, 2024
4 checks passed
@briangregoryholmes briangregoryholmes deleted the bgh/header-tab-update branch February 15, 2024 20:49
mindspank pushed a commit that referenced this pull request Feb 23, 2024
* update dashboard header and tab component

* remove unused variables

* typo
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