layout working finally