sygmaprotocol / sygma-widget

Transfer widget for the sygmaprotocol
5 stars 2 forks source link

feat: overlay #79

Closed wainola closed 8 months ago

wainola commented 8 months ago

Description

Implenentation of loader overlay

Related Issue Or Context

Closes: #66

How Has This Been Tested? Testing details.

Types of changes

Checklist:

wainola commented 8 months ago

@mpetrunic contrary to what I thought, since we are handling the state of the opening and close of the overlay in the widget, it doens't make sense to have event propagation from the overlay itself to signal change of the loading state. Because of that implementation of the component is pretty straigthforward. Possible improvement is to move the spiner itself to a separate component, so we can use that for the button. Wdyt?

mpetrunic commented 8 months ago

ugh, I think you need to fix those conflicts 🙈