Very very minor nitpick, magit-status theme (gray, brown, black) is not very welcoming. I don't use themes so I guess these are defaults magit values. You need to put some rainbow in there :)
Another reason why the default theme (both in it's light and dark variant) are a bit ugly is that they should work even for users who have customized individual faces. Sticking with gray backgrounds seems like a safe bet.