Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Page Info Overlay #159

Merged
merged 1 commit into from
Aug 1, 2012
Merged

Page Info Overlay #159

merged 1 commit into from
Aug 1, 2012

Conversation

afarra
Copy link
Contributor

@afarra afarra commented Aug 1, 2012

Add option for overlaying page info (pg #, sura name, and juz2 #).

Beta for now. Just in case, surrounded the 2 entry points with catch-all to avoid FC's... at least until after Ramadan :)

It determines the empty gap between the top of the ImageView and the top of the actual bitmap. If the gap is large enough, it draws the text within that gap. If not, it uses the ayah-info database to get the actual location of the top of the first line, and utilizes that gap as well for drawing (i.e. between top of bitmap and top of highest glyph). A side effect is that on pages where a sura starts there is a huge gap (either because of basmallah or because of the decoration with sura name) and so it partly overlaps on that.

Should check for ayah-info files, but wasn't sure if it's a good idea because then the user would be asked every time they view a page. For now the "catch all" should take care of it..

ahmedre added a commit that referenced this pull request Aug 1, 2012
@ahmedre ahmedre merged commit de94504 into quran:master Aug 1, 2012
@ahmedre
Copy link
Contributor

ahmedre commented Aug 1, 2012

jazakAllah khair - we need to refactor stuff out of the HighlightingImageView - but waiting until we decide whether or not we'll have one more release before the end of ramadan (as it's a risky change for now).

ahmedre added a commit that referenced this pull request Aug 9, 2015
ahmedre added a commit that referenced this pull request Aug 9, 2015
ahmedre added a commit that referenced this pull request Oct 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants