ruby-china / ruby-china-ios Goto Github PK

我见数字尾巴的交互：

• 当目前 Tab 已经是 Tabitem 所处的页面时，再次点击页面对应的 Tabitem 会触发 Pull Refresh；
• 如果不是，切换到哪个页面，没有刷新。

Router仓库里面的Example跑起来crash

未登录情况下点击评论里的like会在webview里面进行登录。登录进去的结果是我可以评论发帖点赞，但是由于app端我没登录，没有我的access_token，所以我还是得再登录一次。蛋疼。

https://github.com/ruby-china/homeland/blob/36d9301a1f85edd3f9a4cdda10c700c06b7772a7/app/assets/javascripts/app.coffee#L126

这里直接在页面上跳转了而没通过controller返回401，所以没有弹出app登录的页面。

建议app上直接不render回复和点赞的按钮算了。

重现方式是先登录线上的，然后，切成本地，在再次运行

相比话题列表不一样，通知用户每次点进去当然是期望看到新内容的，所以有必要每次都刷新

想参与到开发当中，想问问有没有一些标准，代码和内容。

• 从话题返回列表的时候，需要触发一下 PullToRefresh 以便载入新数据
• 分页一次加载的数量需要增加到 40;
• （赞、关注、收藏）CSS 上隐藏页面的；
• 已关注、已赞的样式不够明显，需要显示赞的数量；
• PKHUD 的样式不够好，如果可以的话该用 MBProgressHUD
• 搜索界面带过去的 access_token 参数拼接不对，出现两个 ? 了

未登陆、或登陆过期的时候，重新打开应用

应用会自动触发 PullToRefresh，这个时候不会弹出登陆，而是等待了一会儿 Crash 了
WebView 的界面也需要检查

我昨晚打开的列表，几天早上打开 App，哪个位置还是昨晚的内容。

看起来像是线程不对的问题

找一个有 location 的 用户
点击 @汉字 就挂了。

有些文章非常长，同时回复也很多，要阅读后面的需要拖动很久才可以，需要增加一个按钮，快速到顶部和底部。

不知道 iOS UI 里面有没有类似的场景例子可以参考。

现在如果网络状态不佳的情况，PullToRefresh 需要等很久才会失败，应该设定一个限时，例如 (3s)，超时以后保持之前的数据不变（目前可能会是白屏）

此外，Turbolinks 由于列表没有用 WebView 了，话题查看页面首次打开的时候慢多了（因为 WebKit 还没初始化），还得想个法优化一下

目前是 24h 有效期的，目前没刷新，会导致每天都需要登录。

OAuth 2 的机制是可以用 refreshToken 来获得新的 accessToken 延长有效期的。

如：帖子楼层的锚点，点击后会再次打开当前页，也没有跳转到目的楼层，而是显示的首屏。

目前 refresh access token 操作只在应用启动时做了，还需要修复上面说的情况。

但我看本地日志，都已经正常返回了的，猜测会不会是 d5f3b3e 这个哪里修改错了

否则 WebView 里面的登录状态依然还存在

API：

https://ruby-china.org/api/v3/topics.json

问题版本：Testflight的当前测试版本

经过我在本地debug的结果来看，原因是OAuth2.shard.currentUser在登录后仍可能是nil
导致 https://github.com/ruby-china/ruby-china-ios/blob/master/ruby-china-ios/Controllers/FavoriteTopicsViewController.swift#L23 let userLogin = OAuth2.shared.currentUser!.login 强制解包失败，造成闪退，似乎App会记录最后一次页面，于是造成无限闪退。

推测：登录方法 会触发 self.reloadCurrentUser() 该方法调用的 UserService.me 为异步方法，如果网络环境较差没有及时响应则会导致accessToken 正确设置，isLogin() 返回 true，但是 currentUser 仍为 nil，通过短路强制解包currentUser的相关代码后，侧边栏为未登录状态，而通知等tab内容显示正常，可以验证推测。

• iphone6s
• ios 10.2.2
  

最近增加了 Youtube 视频嵌入的功能，iOS 客户端打开那种页面会跑到一个空白页面去。

https://ruby-china.org/topics/31241

ruby-china/homeland@8167fe7

由于每个人的头像都不一样，放到 NavBar 上效果不是太好，将它移到 SideMenu NavBar 的 Left 位置。

同时以后 SideMenu 的 NavBar 背景色可能会和主界面的不一样

如题，如果有设计稿我可以帮忙加

提示“注册成功，你可以登录了！” 其实并没有注册！

height为1的bottomBorder。。。