Hiccdown Development Notes

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.

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

#303 · · Dennis HackethalOP, 10 months ago · context · Criticized2

collapse · hide comments

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.

#305 · · Dennis HackethalOP revised 10 months ago · ← 2nd of 2 versions → · Criticism of #303Criticized1

collapse · hide comments

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 #, …
    vc.some_helper_method
  end

  def some_helper_method
    # …
  end
end

#310 · · Dennis HackethalOP revised 10 months ago · ← 3rd of 3 versions → · Criticism of #305

collapse

That would be mixing class methods an instance methods in Rails helper modules, which typically only contain instance methods. Not idiomatic Rails usage.

#312 · · Dennis HackethalOP, 10 months ago · Criticism of #303

collapse

#327 applies here, too: no access to instance variables inside helper class methods.

#332 · · Dennis HackethalOP, 9 months ago · Criticism of #303

collapse