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

Add welcome intro to panel view #154

Merged
merged 6 commits into from
Feb 5, 2024

Conversation

umami-umami
Copy link
Contributor

@umami-umami umami-umami commented Feb 1, 2024

Overview

This PR resolves #53 to add the welcome intro that is used in the overlay extension to the panel extension. A button was added to the nav bar of the panel app. Clicking the button displays the welcome card as an overlay.

Changes

  • Moved Welcome component from overlay extension components to shared components
  • Modified Welcome social icons to allow overflow for small screens
  • Added info icon as a button in the nav component
  • Created new component in the panel that re-uses the shared Welcome component
  • Show / hide the new welcome overlay component in the panel based on button click

Screenshots

Panel Screen Shot 2024-01-31 at 4 44 18 PM
Mobile (Small screen) Screen Shot 2024-01-31 at 4 43 16 PM
Video Overlay Screen Shot 2024-01-31 at 4 44 48 PM

@MattIPv4 MattIPv4 changed the title Add welcome intro to panel extension (Issue #53) Add welcome intro to panel extension Feb 4, 2024
Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

This is generally looking great and is what I had in mind for this! Most of my comments are nits :)

One general thing (couldn't find a logical place in the files to leave the comment), could we add a transition when the overlay is rendered on the panel, the same as the cards (they slide in)?

@umami-umami umami-umami requested a review from MattIPv4 February 4, 2024 23:10
Copy link
Member

@MattIPv4 MattIPv4 left a comment

Choose a reason for hiding this comment

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

LGTM, tyvm

@MattIPv4 MattIPv4 changed the title Add welcome intro to panel extension Add welcome intro to extension panel view Feb 5, 2024
@MattIPv4 MattIPv4 changed the title Add welcome intro to extension panel view Add welcome intro to panel view Feb 5, 2024
@MattIPv4 MattIPv4 merged commit 7dd8e3a into alveusgg:main Feb 5, 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.

Design alveus intro for the panel view
2 participants