Skip to content
This repository has been archived by the owner on Nov 26, 2020. It is now read-only.

FolioReaderCenter.getCurrentChapterName() issue, applied recursive search for toc reference. #342

Merged
merged 7 commits into from
Jul 12, 2018

Conversation

wesleycgs
Copy link
Contributor

Same error that was fixed in FolioReaderCenter.getCurrentChapter().
Fixed FolioReaderCenter.getCurrentChapterName(), applied recursive search for toc reference.

Copy link
Member

@hebertialmeida hebertialmeida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me, the only thing is the question about the removal of contentInsetAdjustmentBehavior

@@ -157,10 +157,6 @@ open class FolioReaderCenter: UIViewController, UICollectionViewDelegate, UIColl
enableScrollBetweenChapters(scrollEnabled: true)
view.addSubview(collectionView)

if #available(iOS 11.0, *) {
collectionView.contentInsetAdjustmentBehavior = .never
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a fix for a bug on iOS 11 collectionView scroll, why did you remove?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please refer to this for more info: 0f161dd

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, sory sorry, i reverted incorrect commit.
Fixed! 658e0ba

Copy link
Member

@hebertialmeida hebertialmeida left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@hebertialmeida hebertialmeida merged commit 4742907 into FolioReader:master Jul 12, 2018
Julyyq added a commit to Julyyq/FolioReaderKit that referenced this pull request Apr 7, 2019
* 'master' of github.com:FolioReader/FolioReaderKit: (51 commits)
  Migrating documentation from cocoadocs to github pages
  Removing extra spaces
  Fix fast scroll bug (FolioReader#379)
  Migrated the Xcode project to use swift 4.2
  Updated Cartfile to not use JSQWebViewController
  Bump minimum version support to iOS 9 and removed legacy JSQWebViewController
  Update FolioReaderKit.podspec
  Update README.md
  Update CHANGELOG.md
  Changed dependencies to work with iOS 12.1 and Xcode 10.1 (FolioReader#377)
  Fix crash on select segment at index. Swift 4.2 (FolioReader#361)
  Update issue templates
  method to get current progress between 0 and 100 (FolioReader#348)
  FolioReaderCenter.getCurrentChapterName() issue, applied recursive search for toc reference. (FolioReader#342)
  Fixed FolioReaderWebView, menu controller still showing colors menu after select any color in menu. (FolioReader#339)
  Fix iOS 11 auto adjust inset bug and closes #332 #312 #261
  Update Cocoapods version
  Add Note function (FolioReader#335)
  Update LICENSE
  Fixed getCurrentChapter(), not returning some chapters, closes #302 thanks to @wesleycgs
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants