search

marigold-dev / tzsafe-ui

TzSafe-UI frontend — to interact with multi-signatures wallets.
https://www.marigold.dev/tzsafe
10 stars 2 forks source link

[Refactor] contract execution refactor #63

Closed rueshyna closed 1 year ago

rueshyna commented 1 year ago
JulesGuesnon commented 1 year ago

Tried the UI, it looks likes it working so once the reviews above are done we can merge

JulesGuesnon commented 1 year ago

Actually, I thought it was like that on main, but it looks likes it's only on this branch. Imo everything should be aligned, but here there's two times some padding around the first input and the second one

image

rueshyna commented 1 year ago

Imo everything should be aligned

everything is aligned for now. but some complicated contract may look a bit weird. I'm generating some contract example to see. one of generated contract: KT1St7YCWTteGpaavrZV767fGfDHyZAPpFy6 KT18jGvnYQWCewajmP7TPefYGcbtSuJ1rfEY

JulesGuesnon commented 1 year ago

everything is aligned for now. but some complicated contract may look a bit weird

I don't understand what you mean by weird, can you take a screenshot or tell me how to reproduce it ?

rueshyna commented 1 year ago

I don't understand what you mean by weird, can you take a screenshot or tell me how to reproduce it ?

Per offline discussion, the layout is ok for now. will keep improving later if needs.