Instances for ToWidget

[andrewthad]

The ToWidget typeclass could be more helpful than it currently is. Notably, there are sensible instances for Text, lazy Text, and text Builder that are not provided. I think that one objection to introducing such classes is that there is a piece of syntax in the shakespeare quasiquoter ^{..} that calls ToWidget on its argument. Adding more instances would allow the end user to feed text into this construct, which might be undesirable.

I don't use the quasiquoter for templating, so the downside of it doesn't bother me. The reason I would like these instances is because there are a number of situations where I have to call toWidget . toHtml in my code that could be abbreviated to toWidget if these instances were provided. If this change is at odds with the goals of the TH templating, then I wouldn't want to mess that up for people, but if not, I would find these instances useful.

[snoyberg]

I could go either way on it. How about asking on the mailing list, and if there are no objections, sending a PR?

[andrewthad]

I'm now on the mailing list, and I've asked there, so we can see what others think.

[andrewthad]

As requested in the google groups conversation, I've put up a PR.