Support html safety for the #render_in method

[bradgessler]

Fixes issue where this code:

  def show
    render inline: (Views::Layouts::Article.new.render_in(view_context){ "<p>Hi</p>".html_safe }), content_type: "text/html"
  end

Would result in a rendition like this:

The updated method checks if the string is html_safe before deciding to call raw or text

[bradgessler]

Just tried your proposed changes in a Rails project and it works.

The "dumbest" thing I can think of to make this work outside of ActiveSupport is a #raw_render_in method. If that's gross then maybe I'd propose an Phlex.raw method that wraps a string in a Phlex::RawString (I don't think I got that name right) class that could then be checked by Phlex.

Putting that all together, the call would look something like this outside of Rails:

Views::Layouts::Article.new.render_in { Phlex.raw("<p>Hi</p>") }

[joeldrapper]

I’d be happy to have some kind of ActiveSupport feature detection that includes a module to override this method. 🤔

[joeldrapper]

👋 I thought of a way to add feature detection and it seems to work.