diff options
| author | Teddy Wing | 2017-06-15 11:22:41 +0200 |
|---|---|---|
| committer | Teddy Wing | 2017-06-15 11:22:41 +0200 |
| commit | a6ca6664092af35505b269052d685c08b4dde8b2 (patch) | |
| tree | 515b7d2049f49450c44c191f6e0b99102328f8a9 | |
| parent | 68e87d50bee6789a1a2377d05c9a9656569b99b6 (diff) | |
| download | chouette-core-a6ca6664092af35505b269052d685c08b4dde8b2.tar.bz2 | |
TableBuilderHelper: Add a couple TODOs
Refs #3479
| -rw-r--r-- | app/helpers/table_builder_helper.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/app/helpers/table_builder_helper.rb b/app/helpers/table_builder_helper.rb index 9384f8791..7e8a235b5 100644 --- a/app/helpers/table_builder_helper.rb +++ b/app/helpers/table_builder_helper.rb @@ -10,10 +10,12 @@ module TableBuilderHelper collection, columns, current_referential: nil, + # TODO: use this when building column headers sortable: true, selectable: false, # selection_actions: [] ## this has been gotten rid of. The element based on this should be created elsewhere links: [], # links: or actions: ? I think 'links' is better since 'actions' evokes Rails controller actions and we want to put `link_to`s here + # TODO: get rid of this argument. Going with params instead sort_by: {}, # { column: 'name', direction: 'desc' } cls: '' # can we rename this to "class"? # ^^ rename to html_options = {} at the end of the non-keyword arguments? Hrm, not a fan of combining hash args and keyword args |