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.

Sign up for GitHub

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

Jump to bottom

你把我的Pull request合并了，然后又把我的代码revert了 #41

Closed
songlipeng2003 opened this issue Mar 12, 2018 · 6 comments
Closed

你把我的Pull request合并了，然后又把我的代码revert了 #41

songlipeng2003 opened this issue Mar 12, 2018 · 6 comments

Comments

@songlipeng2003
Copy link
Contributor

你把我的Pull request合并了，然后又把我的代码revert了，所以最新的代码ios的图片还是有问题
@FWiner
Copy link

FWiner commented Mar 13, 2018

@songlipeng2003 我fork下来合了你改的代码算是暂时解决问题了。。研究了下貌似也没啥问题，谢谢啊

@songlipeng2003
Copy link
Contributor Author

我们已经在正式环境使用，目前没有发现问题

@GuoDapeng
Copy link
Contributor

我现在没时间解决这个让我不是很明白的两个方法，各有利弊，但是我需要你的那个 PR 里面的方法，我担心你在做其他方面的改进时会混到这次 PR 里。所以我建立了新的分支存放了你提交的 PR。我是这么想的，我最近事情很多，所以很抱歉不能很好的解决这个问题。很感谢你这么关注这件事情。

@GuoDapeng
Copy link
Contributor

我一定给出一个既能选择不牺牲画质又能选择更高效率的方法。

@giantss
Copy link
Owner

giantss commented May 10, 2018

#58

@lovelyelfpop
Copy link
Collaborator

@songlipeng2003 用最新的插件看看

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@songlipeng2003 @lovelyelfpop @GuoDapeng @FWiner @giantss