-
Notifications
You must be signed in to change notification settings - Fork 981
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
Implementation of new UI toggle #13384
Conversation
Jenkins BuildsClick to see older builds (18)
|
48ac7fc
to
dbc3c7e
Compare
b95ef1b
to
4302c02
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you also please add new wallet screen
Yes, the new wallet redesign is enabled. home-stack.cljs |
Thank you @flexsurfer for reviewing PR |
94% of end-end tests have passed
Failed tests (5)Click to expand
Passed tests (79)Click to expand
|
85% of end-end tests have passed
Not executed tests (85)Failed tests (13)Click to expand
Passed tests (76)Click to expand
|
60% of end-end tests have passed
Failed tests (2)Click to expand
Passed tests (3)Click to expand
|
@Parveshdhull thanks for your work! It's so exciting to touch the new UI for the first time! 🤩 Found some issues, but not sure if all of them should be fixed at this stage. So please let me know if some of them are not relevant atm and weren't meant by "smoke test the new UI for basic flow". I'll continue testing with that in mind. ISSUE 1: The new UI switches to the old one when switching mode (light/dark) in SettingsSteps:
The tabs have become "old" again, although in the settings video_2022-05-25_14-05-57.mp4ISSUE 2: The
|
0% of end-end tests have passed
Failed tests (1)Click to expand
|
57% of end-end tests have passed
Failed tests (6)Click to expand
Passed tests (8)Click to expand
|
9b71059
to
4c8b80f
Compare
Got it, thanks for your answer @Parveshdhull ! |
fixes #13370
Summary:
PR implement toggle for enabling new UI at runtime
Note:
Development:
screen-overwrites
function will replace current screens on enabling UI togglenavigation2
dispatch is used for now:chat.ui/navigate-to-chat-nav2
for opening chats without topbar for switcher. You can remove this call if you need the top bar in chat.cc: @briansztamfater @jo-mut
Testing:
status: ready