quran / quran.com-frontend

quran.com frontend
https://quran.com
MIT License
998 stars 361 forks source link

Basic styling issues #489

Closed ar5had closed 7 years ago

ar5had commented 8 years ago

s1

In the above picture, you can see there is no padding for first li element with text surahs. It doesnt look good.

Second thing to note down here is the width of container having ayats. Their width is greater than 100%. I guess removing negative margin and some styles, will remove this bug.

s2

I think navbar is holding too much things, sidebars can be used here which will offer same control in better way. I did not like the styling here. The toggle button (hamburger sign) and font-size of surahs (see second picture) doesnt look good. I will try to post some good options in future for these issues.

mmahalwy commented 8 years ago

@arshdkhn1 thanks for the feedback! I agree, the navbar needs some change. It's very confusing. Ideas?

ar5had commented 8 years ago

@mmahalwy I think sidebar will be the best option. By sidebar I meant something like this- afsd

mmahalwy commented 8 years ago

I agree! @arshdkhn1 want to take it on?

ar5had commented 8 years ago

@mmahalwy surely.

mmahalwy commented 8 years ago

thank you!