Merge pull request #50 from basemate/action-class-and-id-attrs

Allow actions to accept id and class attributes

Author: jonasjabari

app/concepts/action/view/action.haml

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

docs/components/action.md

@@ -85,6 +85,34 @@ failure: {
 }
 ```
 
+### ID (optional)
+
+This parameter accepts a string of ids that the action component should have:
+
+```ruby
+id: 'my-action-id'
+```
+
+which renders as an HTML `id` attribute, like so:
+
+```html
+<a id="my-action-id">...</a>
+```
+
+### Class (optional)
+
+This parameter accepts a string of classes that the action component should have:
+
+```ruby
+class: 'my-action-class'
+```
+
+which renders as an HTML `class` attribute, like so:
+
+```html
+<a class="my-action-class">...</a>
+```
+
 ### Notify
 
 Not in use right now.

spec/usage/components/action_spec.rb

@@ -386,6 +386,33 @@ def page2
 
   end
 
+  it 'accepts class and id attributes and returns them as the corresponding HTML attributes' do
+
+    class ExamplePage < Page::Cell::Page
+
+      def response
+        components {
+          action action_config do
+            button text: "Click me!"
+          end
+        }
+      end
+
+      def action_config
+        return {
+          id: 'action-id',
+          class: 'action-class'
+        }
+      end
+
+    end
+
+    visit "/example"
+
+    expect(page).to have_css('a#action-id.action-class')
+
+  end
+
   it 'action_path: passing path as a string (not recommended)' do
 
     Rails.application.routes.append do