NethermindEth / zksync-remix-plugin

Remix plugin for zkSync

MIT License

12 stars 10 forks source link

UI revamp changes #153

Closed satyambnsal closed 1 month ago

satyambnsal commented 1 month ago

Changes summary

147

148

149

150

151

114

38

varex83 commented 1 month ago

I have some problems with paddings/margin here, can you fix it to be centered?

varex83 commented 1 month ago

also, remove this shadow under the env section

varex83 commented 1 month ago

nitpick, make it more table-like, and use tx hash instead of txid

varex83 commented 1 month ago

can we have the same width of dropdown as selector?

satyambnsal commented 1 month ago

@varex83 Thank you for the thorough review. I've addressed all the points raised, with one exception: changing the Transaction history table structure to a more table-like format. This modification would require good CSS changes, so I propose we handle this in a separate PR. Here is how it looks after changing TxId to TxHash

satyambnsal commented 1 month ago

Review changes done.