rebase before landing

rebase before landing

will make sure ci passes before landing

will make sure ci passes before landing

will make sure ci passes before landing

will make sure ci passes before landing

address feedback

will make sure ci passes before landing

update

fix merge conflict with .wasm file

rebase before landing

Hm, radio button doesn't have much contrast when selected. Guessing that component will get updated later as part of this work?

rebase before landing

In D10570#305644, @ashoat wrote:

Several things look weird about this modal:

1. The "Are you sure" line isn't aligned, neither with the icon nor with the header text
2. The spacing between the "Are you sure" line and the one above seems too light given the spacing elsewhere
3. It doesn't have a separation between the button area and the text area. I thought Ted's most recent designs for modals introduced this, but I'm not sure if it was meant to apply universally or not

Not sure this diff is meant to address any of this. Feel free to land without addressing, but in that case please link a relevant Linear issue that explicitly mentions all of the above before landing

D10276 makes this diff obsolete

In D11087#320698, @kamil wrote:

Looks good, two nits:

1. The file from the test plan is not visible
2. Can you also check if it works on the web? Logs on the worker are enough - here is how you can access them

rebase before landing

rebase before landing

rebase before landing

rebase before landing

address feedback + rebase before landing

address comments + rebase before landing

rebase before landing

address feedback + rebase before landing

rebase before landing

rebase before landing

Can you make sure removing form doesn't affect functionality?

rebase before landing

rebase before landing

rebase before landing

rebase before landing