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/link.rb | |
| 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/link.rb')
| -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 | 
