Ryex / ic10emu

A Stationeers IC10 code editor and emulator
https://ic10emu.dev/
Apache License 2.0
7 stars 2 forks source link

Frontend rework #15

Closed Ryex closed 5 months ago

Ryex commented 6 months ago

Staging for rework merge. feel free to review if you care.

netlify[bot] commented 6 months ago

Deploy Preview for ic10emu ready!

Name Link
Latest commit fac6616b2c9379d78b14234be3a35ff226446dad
Latest deploy log https://app.netlify.com/sites/ic10emu/deploys/661992bfa441440008e9c5cc
Deploy Preview https://deploy-preview-15--ic10emu.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Ryex commented 6 months ago

Sweet, the deploy preview works! note, I likely won't be using netlify for the main app but it's super nice for this.

Emilgardis commented 6 months ago

Renaming a device doesn't update it immediately until ic has been stepped/run. The dropdown also shows the old name until you pick it again.

Also, there's no way to add a device but I understand that's still wip

Ryex commented 5 months ago

alright bed time for tonight

Renaming a device doesn't update it immediately until ic has been stepped/run. The dropdown also shows the old name until you pick it again.

this should be fixed now, just needed to send events

Also, there's no way to add a device but I understand that's still wip

That will be the next step after this merges. I need to flesh out device templates on the backend first.

Emilgardis commented 5 months ago

this should be fixed now, just needed to send events

hmm, still looks broken to me:

image
Ryex commented 5 months ago

le sigh https://github.com/shoelace-style/shoelace/issues/1971

Ryex commented 5 months ago

Ok, this is pretty close to Merge ready

that select update issue will need to wait on a upstream fix or some other workaround.

Actual adding of devices will be done later after some work on device templates and building VM state from a data structure is done.

@Emilgardis @ComputerDruid Thank you so much for the previous reviews if you would like to give it one more look over that would be awesome.