Hiccdown Development Notes

Showing only those parts of the discussion which lead to #308 and its comments.

See full discussion instead
  Log in or sign up to participate in this discussion.
With an account, you can revise, criticize, and comment on ideas.

Discussions can branch out indefinitely. Zoom out for the bird’s-eye view.
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 year ago·#303· Collapse

Hiccdown methods should live in Rails helpers as class methods. That way, the problem described in #302 is solved – methods can be referenced unambiguously:

ProductsHelper.index
StoresHelper.index
Criticized1 criticim(s)
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 year ago·#305· Collapse

Does that mean they wouldn’t have access to the view_context? If so, calling helper methods from inside these class methods wouldn’t be possible.

Criticism of #303
Dennis Hackethal’s avatar
Dennis HackethalOP revised about 1 year ago·#308· Collapse
2nd of 3 versions

If so, there might be a way to bind them to the view_context. Or I could definitely pass the view_context explicitly as the first parameter:

So instead of

@helper_module.instance_method(@action_name).bind_call(view_context)

I would do

@helper_module.send(@action_name, view_context)

And the parameter list of each Hiccdown method would start accordingly:

module ProductsHelper
  def self.index vc #, …
    # …
  end
end
Criticism of #305Criticized1 criticim(s)
Dennis Hackethal’s avatar
Dennis HackethalOP, about 1 year ago·#311· Collapse

Superseded by #310. This comment was generated automatically.

Criticism of #308