Is showing the same shape for different purposes a bad practice?

For instance, in this image I am using a line for both dividers and inputs.

enter image description here

Could there be any kind of confusion?

Maybe it's a better idea showing coloured lines, like this

enter image description here

Or to change the shape of inputs to a rectangle?

enter image description here

Thanks!

EDIT:

Here is how it is looking after some of your answers.

Also wanted to know if now, that I changed from the default image to another image, a border is still necessary? I can't seem to make a border/shadow there look good. Again, thanks.

enter image description here