[REFACTOR] migrate all components to 1.0.0 html attributes and properties implementation

Author: Nils Henning

CHANGELOG.md

@@ -502,7 +502,7 @@ end
 
 - [ ] Please add an unique ID to each `async` component usage, even if matestack is currntly autogeneratin an ID if not applied. This will be removed in future releases
 
-#### 12. New `toggle` component is replacing `async` shown_on, hide_on, hidef_afer...
+#### 12. New `toggle` component is replacing `async` shown_on, hide_on, hide_after...
 
 We decided to move all pure clientside view state manipulation logic from the `async` component to a new component called `toggle`. `async` should now only take care of serverside rerendering based on events.
 

app/concepts/matestack/ui/core/action/action.haml

@@ -1,3 +1,3 @@
-%a{@tag_attributes, "@click.prevent": "perform", "href": action_path}
+%a{ html_attributes, "@click.prevent": "perform", "href": action_path }
   - if block_given?
     = yield

app/concepts/matestack/ui/core/async/async.rb

@@ -13,7 +13,10 @@ def initialize(*args)
       if @included_config.present? && @included_config[:isolated_parent_class].present?
         @component_config[:parent_class] = @included_config[:isolated_parent_class]
       end
-      @tag_attributes.merge!({
+    end
+
+    def children_wrapper_attributes
+      html_attributes.merge({
         "v-if": "showing",
         id: @component_config[:component_key]
       })

app/concepts/matestack/ui/core/async/children_wrapper.haml

@@ -1,2 +1,2 @@
-%div{@tag_attributes.merge(class: "matestack-async-component-root")}
+%div{ children_wrapper_attributes.merge(class: "matestack-async-component-root") }
   =yield

app/concepts/matestack/ui/core/bdo/bdo.haml

@@ -1,5 +1,5 @@
-%bdo{@tag_attributes}
-  - if options[:text].blank? && block_given?
+%bdo{ html_attributes }
+  - if text.blank? && block_given?
     = yield
   - else
-    = options[:text]
+    = text

app/concepts/matestack/ui/core/bdo/bdo.rb

@@ -1,11 +1,6 @@
 module Matestack::Ui::Core::Bdo
   class Bdo < Matestack::Ui::Core::Component::Static
-    requires :dir
-
-    def setup
-      @tag_attributes.merge!({
-        "dir": options[:dir]
-      })
-    end
+    html_attributes :dir
+    optional :text
   end
 end

app/concepts/matestack/ui/core/button/button.rb

@@ -1,6 +1,7 @@
 module Matestack::Ui::Core::Button
   class Button < Matestack::Ui::Core::Component::Static
-    html_attributes :disabled
+    html_attributes :autofocus, :disabled, :form, :formaction, :formenctype, :formmethod, 
+      :formnovalidate, :formtarget, :name, :type, :value
     optional :text
   end
 end

app/concepts/matestack/ui/core/collection/content/content.rb

@@ -7,11 +7,9 @@ def setup
     end
 
     def response
-      components {
-        div @tag_attributes do
-          yield_components
-        end
-      }
+      div @tag_attributes do
+        yield_components
+      end
     end
 
   end

app/concepts/matestack/ui/core/collection/content/next/next.haml

@@ -1,3 +1,3 @@
-%a{@tag_attributes, "@click": "next()"}
+%a{ html_attributes , "@click": "next()" }
   - if block_given?
     = yield

app/concepts/matestack/ui/core/collection/content/page/link/link.haml

@@ -1,3 +1,3 @@
-%a{@tag_attributes, "@click": "goToPage(#{@options[:page]})"}
+%a{ html_attributes, "@click": "goToPage(#{page})" }
   - if block_given?
     = yield