Comments (7)
Yep, that's a bug. There are workarounds (like have the current color be a part of the id
attribute) but those are a hack. To be honest I'm kind of surprised this happens, this seems like a pretty basic test of the style handling code. Seems like the diffing code doesn't look at style changes when deciding to update an element?
Workaround:
import karax / [kbase, vdom, kdom, vstyles, karax, karaxdsl, jdict, jstrutils, jjson]
var s1 = style(StyleAttr.color, "blue")
proc createDom(data: RouterData): VNode =
result = buildHtml(tdiv):
p(key="test" & s1.getAttr(StyleAttr.color), style = s1):
text "change color!"
button:
text "Change Color"
proc onclick(ev: Event; n: VNode) =
echo "clicked"
s1.setAttr("color", "red")
setRenderer createDom
from karax.
Seems like the diffing code doesn't look at style changes when deciding to update an element?
I'm pretty sure that's what we're doing.
from karax.
That's why I'm surprised - I've literally seen that code, and had it work in many cases, but I'm not super sure what's going on here.
from karax.
Changing
s1.setAttr("color", "red")
to
s1 = style(StyleAttr.color, "red")
works as expected.
The bug seems to actually be that modifying the existing vstyle ref causes the diff to see the old style as equal to the new style, because they are both the same js array, that contains the new values. (see here, both a.style and b.style refer to the same array). To have this work, the vdom would need to retain a deep copy of the vstyle when it is applied to the dom. This seems like an expensive way to handle allowing re-use of the same Vstyles ref. What do you think @Araq?
from karax.
To have this work, the vdom would need to retain a deep copy of the vstyle when it is applied to the dom. This seems like an expensive way to handle allowing re-use of the same Vstyles ref.
Yes, performance of vstyle is really important. Can't be fixed.
from karax.
Ok had a couple ideas of how we could handle this situation more gracefully. Let me know your preference @Araq and I'll PR it.
Solution 1:
To allow reusing a vstyle as tried above:
Tag VStyles with a "version" field. When the vstyle is modified, increment the version. When a vstyle is rendered, keep a copy of that version number in the vnode. Compare vstyle obj and then version when testing for equality.
Solution 2:
To warn when using a vstyle as tried above, perhaps in debug mode only:
Tag VStyles with a "used" field. When a vstyle is rendered, set it to true. When testing for equality, if vstyles are equal check if the "used" field is true on the new vstyle obj. If true, show a warning about not re-using vstyles (it would be runtime only/in the browser but it's better than nothing).
Option 1 seems like it might benefit performance, as vstyles can be kept around and just modified slightly when needed. Option 2 is simpler to implement and less likely to cause issues.
from karax.
Option 1 seems better. I worry a bit of the overhead of yet another field for every VNode even though many don't have attached styles to them but I cannot think of a non-hacky way right that would avoid that.
from karax.
Related Issues (20)
- Broken functionalities when karax used with grammarly or languagetool (and possibly other extensions) HOT 8
- How to run a function on onclick event in a button? HOT 4
- Boolean Attributes in DSL HOT 6
- Event example does not work HOT 8
- Release 1.2.3 HOT 5
- view doesn't update when changing state in setInterval HOT 3
- IndexDefect on array HOT 3
- Karax input text becoming null HOT 1
- Async karax? HOT 3
- Empty content editable raises an error HOT 1
- How to attach and detach a custom a `Node` to the karax dom? HOT 1
- Karax does not update event handlers HOT 4
- stack size exceeded error on js target HOT 1
- `verbitam` content is appended every time it changes HOT 1
- runtime error on redraw in `removeAllEventHandlers` proc HOT 2
- SVG rendering HOT 2
- 1.3.2 has a tag and release, but karax.nimble version still 1.3.1 HOT 1
- inline closures in loop all capture the last instance for onclick HOT 4
- setDragImage type signature causing BigInt conversion issue HOT 2
- node wrongly deleted after a redraw if style param is used
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from karax.