debugger icon indicating copy to clipboard operation
debugger copied to clipboard

No UI refresh when travel debug / replay is enabled.

Open zpodlovics opened this issue 1 year ago • 3 comments

Description

No UI refresh when travel debug / replay is enabled.

Repro code

  1. Create a new awesome app using the elmish template as of https://github.com/elmish/templates
  2. Install redux devtools extension (firefox in my case)

Expected and actual results

Expected result: UI refresh.

Actual result:

The counter value should be 0 in this case.

Screenshot from 2023-07-30 21-57-21

Related information

  • elmish version: 4.0.1
  • fable-compiler version:4.1.4
  • fable-core version: 4.0
  • Operating system: Ubuntu 22.04

paket.lock.zip

It seems that the interaction between the components are complicated especially when react and stores are involved. Would it be impossible to provide a way to refresh the ui (functionality) for the debugger?

zpodlovics avatar Jul 30 '23 19:07 zpodlovics

Thanks for the report! I don't recall if I ever tried this in FF. It's possible some security feature is interfering with the plugin. What is that angry red icon in the status bar? Could you try this in opera/chrome/edge as well? Does it work with standalone DevTools?

et1975 avatar Jul 31 '23 18:07 et1975

The angry red icon on the status bar is the missing favicon. Screenshot from 2023-08-06 16-39-23

I also checked it with chrome (fresh install, only redux devtools installed):

Screenshot from 2023-08-06 16-40-30

And edge as well (fresh install, only redux devtools installed):

Screenshot from 2023-08-06 16-41-07

Checking it with standalone devtools is works in progress.

zpodlovics avatar Aug 06 '23 14:08 zpodlovics

I just tried the debugger on our ToDo sample and it works as expected. Looking at the template code I think |> Program.withReactBatched and |> Program.withDebugger are in reverse order. @zpodlovics please flip the order of these 2 instructions and try again? If that does it please open a PR into the template repo to fix this!

et1975 avatar Sep 14 '23 17:09 et1975