diff options
| author | Teddy Wing | 2017-06-15 14:37:14 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-06-15 14:37:14 +0200 |
| commit | 3b621ee93c5097c55bd4ce4c7bdecdaa18c6f9ab (patch) | |
| tree | e4c5fd925687018476e51435511b686b456a03b1 /lib | |
| parent | 5897637c559ba3d37e4f02b97149ff7dab927ef2 (diff) | |
| download | chouette-core-3b621ee93c5097c55bd4ce4c7bdecdaa18c6f9ab.tar.bz2 | |
TableBuilder: Make destroy link in gear menu render correctly
Previously we weren't correctly outputting the proper HTML to get the
:delete link rendering correctly with the icon and text. It should show:
[TRASH CAN ICON] Supprimer
but instead was showing the href.
To get this working, I added a `@content` attribute to `Link`. This
allows us to optionally specify non-text content to display in the link.
We then have delete-link-specific handling when rendering the gear
menu's HTML. In that particular case, the `<li>` needs to have a
'delete-action' class in order for the styling to work properly.
Created a new `destroy_link` helper method to allow us to reuse the
destroy link content in the header bar.
TODO: Collect Link#content and Link#name into the same property.
TODO: Rename `#destroy_link` to `#destroy_link_content`
Refs #3479
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/link.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/link.rb b/lib/link.rb index c875b85ac..3ce12ae24 100644 --- a/lib/link.rb +++ b/lib/link.rb @@ -1,10 +1,11 @@ class Link - attr_reader :name, :href, :method, :data + attr_reader :name, :href, :method, :data, :content - def initialize(name: nil, href:, method: nil, data: nil) + def initialize(name: nil, href:, method: nil, data: nil, content: nil) @name = name @href = href @method = method @data = data + @content = content end end |
