Skip to content
This repository has been archived by the owner on Jun 28, 2021. It is now read-only.

line-height updated. #553

Closed
wants to merge 2 commits into from
Closed

line-height updated. #553

wants to merge 2 commits into from

Conversation

ar5had
Copy link
Contributor

@ar5had ar5had commented Jan 6, 2017

This bug can be mixture of various style. It might be present because of font-weight . I think using percentage for line-height is not good. % are evil in my opinion.

This PR aims to solve issue-548.

@ar5had
Copy link
Contributor Author

ar5had commented Jan 6, 2017

@ahmedre @ATouhou Can you still replicate the bug ?

@ar5had ar5had closed this Jan 6, 2017
@ar5had ar5had reopened this Jan 6, 2017
@ahmedre
Copy link
Contributor

ahmedre commented Jan 6, 2017

Deployed to: http://staging.quran.com:33064

@mmahalwy
Copy link
Contributor

mmahalwy commented Jan 6, 2017

rebuild

@mmahalwy
Copy link
Contributor

mmahalwy commented Jan 6, 2017

okay i kicked a rebuild so we can take a look at it. Inshallah this solves it!

@ahmedre
Copy link
Contributor

ahmedre commented Jan 6, 2017

Deployed to: http://staging.quran.com:33065

@mmahalwy
Copy link
Contributor

mmahalwy commented Jan 6, 2017

Hmm, that did not work. I think it has something to do with a horizontal and vertical scroll going wrong.

@mmahalwy
Copy link
Contributor

mmahalwy commented Jan 6, 2017

Okay, I think it's something to do with the lazy load component making the scroll + rendering all weird.

@ar5had
Copy link
Contributor Author

ar5had commented Jan 6, 2017

Maybe. The repeated word isn't a new element but it is the same word. I checked it in console.

Another thing I have noticed is that the markup of surah page and other pages is not semantic. I donno how much that matters for quran.com but I believe it might be reason for small glitches that we have discovered or are yet to be discovered.

@mmahalwy
Copy link
Contributor

okay that did not fix it...

@mmahalwy
Copy link
Contributor

rebuild

@ahmedre
Copy link
Contributor

ahmedre commented Jan 10, 2017

Deployed to: http://staging.quran.com:33105

@mmahalwy mmahalwy closed this Jan 20, 2017
@ar5had ar5had deleted the font-bug branch January 30, 2017 04:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants