no, this is not according to the guidelines, when using `new` in `Props` it should be defined as an factory method in companion object to avoid closing over unwanted things. Leave it as `Props[Counter]` here.