idea: clean up Builder UI #149
Conversation
kasparkallas
requested review from
philipandersson,
vmichalik,
elvijsTDL and
0xYoanna
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Playwright Tests tests failed after 19m 7s Run Details
This message was posted automatically by
currents.dev | Integration Settings
|
|
I'm torn, on one side it is less context for each step but now we have more steps. This would be a good A/B test imo! Anyways, since we have scrolling in the sidebar, maybe scrolling should move you to the next tab. Or having an arrow + progress bar to show that you are moving forward. Sense of progression and hinting of direction in tabs could be improved imo. |
the NFT tab is on the way too. Though we don't have a big enough sample-size for A/B testing |
vercel
bot
temporarily deployed
to
Preview – widget-gated-community-demo
| }, | ||
| i, | ||
| ) => ( | ||
| <Zoom |
There was a problem hiding this comment.
Different delay based on index would look awesome. Index * 100 or something
There was a problem hiding this comment.
I see what you mean and it's a cool idea but it's slightly more complicated.
The main use for the animation is to animate the latest payment option that was added. The last one to be added has the highest index.
It will show up on top though as I reverse the list after the .map call.
apps/widget-builder/src/components/payment-option-view/PaymentOptionView.tsx
Outdated
Show resolved
Hide resolved
| const demoStyling: DisplaySettings = { | ||
| ...defaultDisplaySettings, | ||
| darkMode: faker.datatype.boolean(), | ||
| primaryColor: faker.color.rgb() as `#${string}`, |
There was a problem hiding this comment.
faker generates random values right? I would use default values instead because faker could generate some really ugly configurations.
There was a problem hiding this comment.
Nah, it's fun. I'm open to adding a "Reset default theme" button.
vercel
bot
temporarily deployed
to
Preview – widget-gated-community-demo
Makes more sense this way, IMO. Thoughts?