Didn't you post this before? Looks very familiar.
I don't see any issues with the routing.
Using vias to connect top & bottom ground layers is fine, I do it all the time.
Couple of the LED names are sitting on the outer grey ring, maybe move those a little. With Eagle I would Smash the part and then move the name.
Some of the names are overlapping the part outlines, I would fix those.
Some of the parts - you appear to have values there instead of a reference designater (C1, C2, etc). I would fix that.
Some of the names overlap vias. Move the names a little or they will not be readable.
It's the little cosmetic stuff like that which makes the board look sharp when all done. Having the Ref Des for each part makes the board much easier to debug as well if needed.