Made several El methods chainable. Fixed bug in El.set_text.

Author: David-OConnor

CHANGELOG.md

@@ -9,8 +9,11 @@ and `El::new_text` respectively
 in addition to semicolons
 - Fixed typos in a few attributes (Breaking)
 - Fixed a bug where an HTML namespace was applied to raw html/markdown elements
-- New conditionaln syntax added in `class!` macro: ``
+- New conditional syntax added in `class!` macro, similar to `Elm`'s `classList`
 - `Listener` now implements `MessageMapper`
+- `El methods` `add_child`, `add_style`, `add_attr`, and `set_text` now return the elements,
+allowing chaining
+- Fixed a bug with `set_text`
 
 
 ## v0.3.6

CONTRIBUTING.md

@@ -9,4 +9,9 @@ Required tools to build and test:
 
 Before submitting a PR, please run `cargo make verify`: This will run `fmt`, `clippy`, build all examples, and run all tests.
 
-We're currently moving quickly and making breaking API changes, when appropriate.
+Recommended starting points:
+- Open issues
+- Adding and improving tests
+- Friction points in your Seed apps
+- `//todo` comments in code
+

examples/server_integration/Cargo.lock

@@ -41,8 +41,7 @@ fn update(msg: Msg, model: &mut Model, _: &mut Orders<Msg>) {
 /// This func demonstrates use of custom element tags, and the class! and
 /// id! convenience macros
 fn misc_demo() -> El<Msg> {
-    let mut custom_el = El::empty(Tag::Custom("mytag".into()));
-    custom_el.set_text(""); // Demo of set_text.
+    let custom_el = El::empty(Tag::Custom("mytag".into())).set_text(""); // Demo of set_text.
 
     let mut attributes = attrs! {};
     attributes.add_multiple(At::Class, &["a-modicum-of", "hardly-any"]);

examples/window_events/src/lib.rs

@@ -835,23 +835,28 @@ impl<Ms> El<Ms> {
     }
 
     /// Add a new child to the element
-    pub fn add_child(&mut self, element: El<Ms>) {
+    pub fn add_child(mut self, element: El<Ms>) -> Self {
         self.children.push(element);
+        self
     }
 
     /// Add an attribute (eg class, or href)
-    pub fn add_attr(&mut self, key: String, val: String) {
+    pub fn add_attr(mut self, key: String, val: String) -> Self {
         self.attrs.vals.insert(key.into(), val);
+        self
     }
 
     /// Add a new style (eg display, or height)
-    pub fn add_style(&mut self, key: String, val: String) {
+    pub fn add_style(mut self, key: String, val: String) -> Self {
         self.style.vals.insert(key, val);
+        self
     }
 
     /// Replace the element's text node. (ie between the HTML tags)
-    pub fn set_text(&mut self, text: &str) {
-        self.text = Some(text.into())
+    pub fn set_text(mut self, text: &str) -> Self {
+        // todo: Allow text to be impl ToString?
+        self.children.push(El::new_text(text));
+        self
     }
 
     /// Shortcut for finding the key, if one exists