rueshyna
commented
1 year ago Overall looks good, but I do have a small problem with the design. It really looks giant compared to the transactions above. I think it can be a good idea to try:
- adding more space between the warning and the transactions table
- reducing a bit the line height of the text
I'm adding more warning messages this branch:
https://github.com/marigold-dev/tzsafe-ui/tree/cc%40warning_message_setting
I put messages in li tag. I'll also apply above suggestion with this branch. There will be easier not working on two branches. :)
the change:
refactor to add warning message
only show warning message when signing and resolving with approval result. the same change is applied to settings for the warning of the change.
when sign the proposal:
when reject the proposal:
when resolve the proposal (approve):
when resolve the proposal (reject):
cc: @tj825