iced_web icon indicating copy to clipboard operation
iced_web copied to clipboard

UI is not updating/working with some web extensions turned on

Open orsenkucher opened this issue 5 years ago • 3 comments

Something within Dark Reader chrome extension is blocking iced from updating the UI. As you can see, two radio buttons are selected and slider value was not updated, despite slider itself being moved. Click on button does nothing as well
image Demo is perfectly fine when extension is turned off and page reloaded. Also every other page on internet behaves normally with extension working. So I assume iced XOR dark reader are yielding this.

orsenkucher avatar Sep 28 '20 21:09 orsenkucher

might be a consequence of some broader problem somehow utilized by extension 🤷‍♂️

orsenkucher avatar Sep 28 '20 22:09 orsenkucher

I can confirm this. DarkReader is adding an own style element after each style element found in the body. It seems that Dodrio isn't aware of a change from outside, wich breaks the VDOM.

dodrio/change-list.js:90 Uncaught TypeError: Cannot read property 'firstChild' of null
    at Array.pushFirstChild (dodrio/change-list.js:90)
    at ChangeList.applyChangeRange (dodrio/change-list.js:231)
    at ChangeList.applyChanges (dodrio/change-list.js:220)
    at imports.wbg.__wbg_applyChanges_80aff4692bcb5634 (tour.js:281)
    at /tour/tour_bg.wasm:wasm-function[125]:0x63f2d
    at /tour/tour_bg.wasm:wasm-function[305]:0x8715e
    at /tour/tour_bg.wasm:wasm-function[638]:0x96d16
    at __wbg_elem_binding2 (tour.js:27)
    at real (tour.js:444)

If I add an additional style element by myself to iced.rs like this:

...
<div style="width: 100%; height: 100%">
    <style>...</style>
    <style></style><!-- the style I've added -->
    <div class="c" ...>
        ...
    </div>
</div>
...

... I get a similar error:

dodrio/change-list.js:97 Uncaught TypeError: Cannot read property 'nextSibling' of null
    at Array.popPushNextSibling (dodrio/change-list.js:97)
    at ChangeList.applyChangeRange (dodrio/change-list.js:231)
    at ChangeList.applyChanges (dodrio/change-list.js:220)
    at imports.wbg.__wbg_applyChanges_80aff4692bcb5634 (tour.js:281)
    at /tour/tour_bg.wasm:wasm-function[125]:0x63f2d
    at /tour/tour_bg.wasm:wasm-function[305]:0x8715e
    at /tour/tour_bg.wasm:wasm-function[638]:0x96d16
    at __wbg_elem_binding2 (tour.js:27)
    at real (tour.js:444)

There is already an open issue at the dodrio repository.

Kaiden42 avatar Oct 03 '20 18:10 Kaiden42

i feel like dodrio is abandoned/unmaintained at this point, so this isn't going to be fixed anytime soon.

Absolucy avatar Jan 05 '21 21:01 Absolucy