search

UESTC-Miniapp / UESTC-Miniapp-FrontEnd

电子科技大学小程序前端代码库
4 stars 1 forks source link

Bug fix #23

Closed Vizards closed 5 years ago

Vizards commented 5 years ago

对益达大佬的这个项目挺感兴趣，修了一点 issue 里的 bug

如果不允许直接提交到 master 的话，麻烦 maintainers 新建个分支，以后的 commit 也会在这个分支

Yidadaa commented 5 years ago

woc 居然能吸引到 vizards 大佬来贡献代码，有点兴奋。 这个项目是和一个18级学弟一起开发的，目的是为了锻炼下学弟的 coding 能力，同时保持下新鲜感。刚开始实现后端部分的时候做过调研，曾经很多次在搜索页看到过你的项目，没想到今天能看到你的 pr，属实有些激动。

Yidadaa commented 5 years ago

这项目最近才搞出一个像样（能用）的版本，开发流程还没有规范起来，如果有兴趣一起开发的话，我会在 merge 了这个 pr 后锁定master分支，然后统一在dev分支上开发。

MrThanlon commented 5 years ago

我知道不应该太过强迫症。如果直接设置背景色会导致在页面底部继续上拉同样不好看

Vizards commented 5 years ago

@MrThanlon 小程序可以分别设置顶部和底部背景色。但是仅 iOS 支持，预计后面更新会支持安卓，可以接受？

https://developers.weixin.qq.com/miniprogram/dev/api/ui/background/wx.setBackgroundColor.html

MrThanlon commented 5 years ago

@Vizards 根据益达大佬的说法，安卓版并不能下拉(我没有安卓设备无法测试)，所以可以不用考虑安卓版的背景色。我觉得可以这么做。 另外还有一点，href替换成onclick事件后似乎会导致在请求结束才执行页面跳转，在界面上看起来像是响应变慢，这是不能接受的。