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

iOS 7 styling, uses the UINavigationBar at the top, cleaned up bar buttons, Cocoapods installation #53

Closed
wants to merge 40 commits into from

Conversation

KiranPanesar
Copy link

Hi all,

Decided to fork this project and do some necessary refactoring.

Here's the change list (from the new readme):

  • Optimised UI for iOS 7 style.
  • Implemented fast pagination as per this pull request.
  • Ditched the ReaderMainToolbar for the top toolbar. The viewer now uses the current UINavigationController's nav bar. Note: this means the the viewer is required to be shown in a UINavigationController instance. See README for info.
  • Ditched cluttered bar button items in favour of two UIBarButtonItems.
  • Added ability to open PDF in other apps.
  • Moved all framework files into dedicated folder (ReaderFramework).
  • Moved assets into Reader.bundle file in /ReaderFramework.
  • Added UIPopover approach for iPad actions (see here).
  • Installation via Cocoapods.

@PreetySingh
Copy link

Can we change the size of the pdf being rendered. In one of my apps I want to have a toolbar on right. So it should not be of width 1024 but lesser.
When I adjusted the size of reader content view the image is not shown completely (The right side is cropped not shown on screen).
I even modified scrollView bounds but it didn't worked out.

Please suggest me something. Thank you for your cooperation.

MaxHasADHD and others added 4 commits June 24, 2014 18:39
Merged a fork that added iCloud file support and updated some of the UI
Added Non-retina assets
Set prefersStatusBarHidden to NO in ThumbsViewController
@vfr vfr closed this Sep 16, 2014
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.

4 participants