Hacker News new | ask | show | jobs
by silisili 757 days ago
This looks pretty useful, kudos.

If I had any nit to make, it's about browser.NewBrowser. It stutters, and even with just New(), at first I thought it was making a client. Not sure what I'd go with, maybe Parse like net/url?

Again, just a minor nit, not trying to take away what you've done here at all.

2 comments

PageReader? HtmlReader?
Yes, it does right. Honestly, I couldn't find a better name. `Parse` sounds nice, so let me think about it.