Rebase
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Dec 10 2024
Rebase
Rebase
Rebase
Rebase
Looks okay. Would be nice to be able to show progress, but I recognize we'll probably need to wait on backup size calculation for that.
Either one looks good to me!
Rebase
We're using a library https://www.npmjs.com/package/react-native-progress for displaying the multimedia upload progress, and we can consider using it here. I updated the diff with one of the options, but the NPM page presents some alternatives.
Update backup details and a loading indicator
Update the icon color
Rebase
Use window dimensions hook
Dec 9 2024
rebase before landing
fix frog callback return type to be Promise<FrogResponse> | FrogResponse
rebase
fix setting parent
rebase
rebase
rebase
rebase
In D14086#392872, @ashoat wrote:I haven't actually seen this utilized when we render the frame on the validator or warpcast, so I don't believe the title matters too much
Sounds like you don't know how it's used. It would be good to figure out before landing this... I worry that, for instance, we'll end up with an application titled "Comm Frame" in some mini-app directory.
I think it would probably be better to just use "Comm" if you truly have no clue what this is or how it is used, even after spending an hour or so trying to dig up information.
I haven't actually seen this utilized when we render the frame on the validator or warpcast, so I don't believe the title matters too much
rebase
change frame title to "Comm Frame" instead of "Comm Frame App"
But I don't know how to display this flow in this mode. I modified the colorsSelector so that I can see how it looks like.
Use the new icon
Looks great! Thanks for iterating on this :)
Added a placeholder icon. Will update the diff after https://linear.app/comm/issue/OPS-305/mockup-icon-for-restore-account-screen finalizes the design.
- The styling of the three fields looks strange. It's not clear to me if the values there should be updating, and I feel like the spacing should be improved. Do we want bullet points here?
- What are "backup details"?
The backup details is a header for the two fields that are displayed. Based on the discussion from https://linear.app/comm/issue/ENG-9680/create-a-restoration-screen it seems like getting the size might take some time, so if we decide to support only the timestamp for now, we shouldn't display the backup details header and the bullet points.
Rebase
Thanks for explaning :)
The QR code size was made to depend on the screen width, which is always less than the height. This allows the user to always be able to see the full QR even for the most uncommon aspect rations (sometimes the user would need to scroll this screen). Tested on two devices, one of which has 16:9 aspect.
Update the design