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.

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

解决示例铃声播放问题,增加推送图片拖放到相册 #266

Closed
wants to merge 0 commits into from

Conversation

uuneo
Copy link
Contributor

@uuneo uuneo commented Dec 12, 2024

  1. 保持只有一个音频正在播放,并且点击相同音频停止播放
  2. 推送图片下拉后支持拖放到相册或者其他需要的地方
    12月12日

@uuneo
Copy link
Contributor Author

uuneo commented Dec 12, 2024

另外,复制的提示文字我改到上面显示了,(今天测试了一下,如果图片太高会造成滚动,提示文字如果在底部看不到)

@Finb
Copy link
Owner

Finb commented Dec 16, 2024

哈喽,保持只有一个音频正在播放测试完了,没有问题,非常感谢

但推送图片的改动有些问题,这个特性还没设计好,现在尽可能不去改动
拖动图片这个交互有点别扭和不直观,希望能去掉
图片控件的高度最好根据屏幕高度去设置UIImageView的最大高度,配合contentMode=.scaleAspectFit 完全展示图片(避免拖动)

总之图片相关的改动希望能先去掉,先合并下这个PR。最多加一下根据屏幕高度设置 UIImageView 的最大高度,提在另一个PR

@uuneo
Copy link
Contributor Author

uuneo commented Dec 16, 2024

好的,没问题!这个位置其实不应该加这个交互,起因是因为app内部并没有查看图片缓存,或者保存图片的功能,我用这个主要的场景是直接浏览器的图片推到手机,这个只是在使用起来做一个短暂的过度,还有图片高度的问题,我现在是使用的宽度来做基准,这样可以充满整个通知界面,如果说换成高度做基准的话,两边可能产生白边,而且可能长图会看不清楚!感谢你的回复,我先删除相关的代码

@uuneo uuneo force-pushed the master branch 4 times, most recently from 180a9e9 to 51ca40e Compare December 16, 2024 09:36
@Finb
Copy link
Owner

Finb commented Dec 16, 2024

谢谢,我看NotificationViewController.swift 一起被提交上来了,但实际只是空格改成了\t。
所以还需要帮忙执行下

git reset --soft 6c4577e
git restore --staged notificationContentExtension/NotificationViewController.swift
git restore notificationContentExtension/NotificationViewController.swift

然后重新提交下。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants