158-jumping-cursor

[MartinKavik]

#158
It seems that jumping cursor is caused by fighting set_values - when I comment-out two calls, it works (see changes).

Also we can add returns to its body for a small performance improvement.

(This is only draft PR to show code changes)

[David-OConnor]

I need to do more homework on the consequences of removing these

[David-OConnor]

Haven't dove in yet, but of note:
-I can confirm this fixes the problem for me.
-Related merged PRs, that introduced this set_value.
#53
#54

[MartinKavik]

Issue should be resolved with this PR.

Smaller changes

• Resolved // todo: Unable to get this convenience function working from util.rs.
  • See new related todos in utils.rs:176.
• Updated example_e.rs:
  • It contains a form with many controls for testing inputs.
  • Backend processes received multipart data so this example can be used as example for file upload.

Main change

• HtmlElement value handling refactored:
  • events.rs: Many rows deleted, I think it should be used only for handling user's callbacks, not for changing DOMs (it's VirtualDOM's work).
  • websys_bridge: New function set_default_element_state which set default (initial) state for elements. This function should be as short as possible.
  • patch.rs: Resolved // todo ideally we unify attach as one method.
  • util.rs: I tried to properly implement get/set_value / get/set_checked for Html**Elementss.

Breaking changes

• There shouldn't be any.

[David-OConnor]

Nice. Much cleaner than the way I had it before.