Review

a lot of code changed
also no need to be specific. use ViewGroup here