Make SsrNode attribute order stable (#323)

This makes it possible to use SSR'd output as snapshot test by making the attribute order deterministic.

* Stable output of render_to_string

* Run cargo fmt & cargo clippy

* The output result of `render_to_string` is now fixed

* Rearrange the import order

.gitignore

@@ -9,3 +9,7 @@ Cargo.lock
 
 # VS Code folder
 .vscode
+
+# Jetbrain IDE
+.idea/
+*.iml

examples/hydrate/src/main.rs

@@ -61,7 +61,7 @@ fn app() -> View<G> {
 fn main() {
     console_error_panic_hook::set_once();
     console_log::init_with_level(log::Level::Debug).unwrap();
-    
+
     let s = sycamore::render_to_string(|| view! { App() });
     log::info!("{}", s);
 

packages/sycamore/src/builder/agnostic/mod.rs

@@ -142,17 +142,17 @@ where
         if TypeId::of::<G>() == TypeId::of::<crate::SsrNode>() {
             // If Server Side Rendering, insert beginning tag for hydration purposes.
             self.element.append_child(&G::marker_with_text("#"));
-            // Create end marker. This is needed to make sure that the node is inserted into the right
-            // place.
+            // Create end marker. This is needed to make sure that the node is inserted into the
+            // right place.
             let end_marker = G::marker_with_text("/");
             self.element.append_child(&end_marker);
             render::insert(
                 &self.element,
                 View::new_dyn(child),
                 None,
                 Some(&end_marker),
-                true, /* We don't know if this is the only child or not so we pessimistically set
-                       * this to true. */
+                true, /* We don't know if this is the only child or not so we pessimistically
+                       * set this to true. */
             );
             return self;
         }
@@ -172,8 +172,8 @@ where
                 View::new_dyn(child),
                 initial,
                 None,
-                true, /* We don't know if this is the only child or not so we pessimistically set
-                       * this to true. */
+                true, /* We don't know if this is the only child or not so we pessimistically
+                       * set this to true. */
             );
             return self;
         }

packages/sycamore/src/generic_node/ssr_node.rs

@@ -6,7 +6,7 @@ use std::hash::{Hash, Hasher};
 use std::iter::FromIterator;
 use std::rc::{Rc, Weak};
 
-use ahash::AHashMap;
+use indexmap::map::IndexMap;
 use once_cell::sync::Lazy;
 use wasm_bindgen::prelude::*;
 
@@ -129,7 +129,7 @@ impl GenericNode for SsrNode {
 
     fn element(tag: &str) -> Self {
         let hk = get_next_id();
-        let mut attributes = AHashMap::new();
+        let mut attributes = IndexMap::new();
         if let Some(hk) = hk {
             attributes.insert("data-hk".to_string(), format!("{}.{}", hk.0, hk.1));
         }
@@ -363,7 +363,7 @@ impl WriteToString for SsrNode {
 #[derive(Debug, Clone, Eq, PartialEq)]
 pub struct Element {
     name: String,
-    attributes: AHashMap<String, String>,
+    attributes: IndexMap<String, String>,
     children: Vec<SsrNode>,
 }
 

packages/sycamore/src/utils/hydrate/mod.rs

@@ -8,7 +8,8 @@ thread_local! {
     static HYDRATION_CONTEXT: RefCell<Option<HydrationRegistry>> = RefCell::new(None);
 }
 
-/// Run the closure inside a hydration context. If already inside a hydration context, creates a nested context.
+/// Run the closure inside a hydration context. If already inside a hydration context, creates a
+/// nested context.
 pub fn with_hydration_context<F, R>(f: F) -> R
 where
     F: FnOnce() -> R,