Roman

Proposal. There could be `nil pointer dereference` in these methods. Check if req, res and req.URL are not `nil`.

Transport: metrics.Metrics.String remove redundant index increment in the loop

1

The removed line does nothing. Index `i` is incremented by `range` clause and increment inside the range clause body doesn't affect it.

Return error when default URL cannot be parsed

3

This changes are more as a proposal. I see the point why this error check is not so important (we parse default url), but error check can help if someone...

feat(viewport): horizontal scroll

12

implement #235