devedbox / AXWebViewController

AXWebViewController is a webViewController to browse web content inside applications. It’s a lightweight controller on iOS platform based on WKWebView (UIWebView would be the base Kit under iOS 8.0). It added navigation tool bar to refresh, go back, go forward and so on. It support the navigation style on WeChat. It is a simple-using and convenient web view controller using inside applications.
MIT License
797 stars 133 forks source link

一些更新 #54

Closed tentenlee100 closed 6 years ago

tentenlee100 commented 6 years ago
  1. 更新到xcode 9,xcode 8會無法使用,因為會有xcode9的檢查版本語法。
  2. 修正使用cocoapods use_frameworks!時圖片與語系無法載入的問題
  3. 新增設定showsNavigationCloseBarButtonItem,在設定AXWebViewControllerNavigationBarItem的時候不會顯示"關閉"按鈕,預設是顯示(為了體感不想讓使用者覺得怪怪的)
  4. 新增設定showsNavigationBackBarButtonItemTitle,在設定AXWebViewControllerNavigationBarItem的時候不會顯示"返回"按鈕文字,還是可以點,預設是顯示(整體app若都沒顯示返回的話,就這邊會顯示也怪怪的)
  5. 新增設定checkUrlCanOpen,不會檢查是否可否開啟,一率都開啟url,預設是會檢查只有在ios 8之後有效。(因為ios 8之後不需要檢查也不會crash了,有開模擬器測試過)
devedbox commented 6 years ago

谢谢!我将会在最快的时间完成审查合并!

tentenlee100 commented 6 years ago

@devedbox 請問一下是要加在哪邊呀? 因為我不知道哪邊有問題.... 抱歉 你是說 checkUrlCanOpen這個設定限制ios8之後嗎??

devedbox commented 6 years ago

是的,就是这个checkUrlCanOpen,还有些其他问题,这些更新你都测试过了吗?

tentenlee100 commented 6 years ago

@devedbox 加上NS_AVAILABLE_IOS這我沒用過...不知道加上後我的.m裡面的檔案需不需要修改。 我也不知道怎麼去測試...

還是我只要加上這段就好了?

/// Check url can open default YES ,only work after iOS 8.
@property(assign, nonatomic) BOOL checkUrlCanOpen NS_AVAILABLE_IOS(8_0) ;

這幾個是有稍微測試過了,新設定我自己都在用了,目前還沒有發生什麼問題。

devedbox commented 6 years ago

@tentenlee100 👌,我再审阅审阅,没问题的话就先把你的合并了,我再稍微做点修改。若有修改,将会告知并发版!谢谢支持!

devedbox commented 6 years ago

这是加上API available之后的代码:

/// Check url can open default YES ,only work after iOS 8.
@property(assign, nonatomic) BOOL checkUrlCanOpen API_AVAILABLE(ios(8.0));
tentenlee100 commented 6 years ago

@devedbox 我想到一件事情,這設定加下去不是會造成他有設定的話會直接crash? 這樣他在支援低版本的時候反而會有問題吧? 但是內部程式其實都已經判別好了,就算他app最低支援在ios7有使用也不會用到這個設定。 這樣子是不是不要加個宣告比較好呀?

tentenlee100 commented 6 years ago

另外 這版本上來之後,README.md 也要稍微更新一下,用這版本只支援xcode 9了 8就會不認識xcode9專屬的語法了

devedbox commented 6 years ago

了解你的意思了,crash的话不至于,现在使用API_AVAILABLE(...)的语法,在xcode9上如果不判定版本的话,编译器会报错,而不会在运行时去判定,这是新加入的新功能; README.md需要更新,这两天会抽空更新。

最近项目比较繁忙,劳你费心了! 哈哈😆

tentenlee100 commented 6 years ago

方便release新的版本號嗎? 這樣update README會比較清楚

devedbox commented 6 years ago

Check out v0.7.0.