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

Design maintenance Jul 7 2023 #266

Merged
merged 5 commits into from
Jul 10, 2023

Conversation

micheal-parks
Copy link
Member

@micheal-parks micheal-parks commented Jul 7, 2023

Done in a session with nat. Mostly involves notification and modal tweaks.

@DTCurrie
Copy link
Member

DTCurrie commented Jul 7, 2023

What was the fix for the notify "art" bug?

@micheal-parks
Copy link
Member Author

micheal-parks commented Jul 10, 2023

@DTCurrie

We restructured slots to be more intuitive, but this will actually require a follow up in app. There's an example added that creates a notification that would exist on the history page without that visual bug 🎉

(it was a flexbox + slot issue)

Copy link
Member

@mrloureed mrloureed left a comment

Choose a reason for hiding this comment

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

LGTM!

title='This is the title.'
message='This is the message.'
title='You have an incoming life update'
message='LifeOS has been upgraded to a new version that includes great new things such as friends and family and you will not be alone anymore.'
Copy link
Member

Choose a reason for hiding this comment

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

🤣

@micheal-parks micheal-parks merged commit 356f6f6 into viamrobotics:main Jul 10, 2023
5 checks 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.

3 participants