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

[Suggestion]: 优化Screen On/Off 事件 #399

Closed
ywjheart opened this issue Feb 15, 2024 · 10 comments
Closed

[Suggestion]: 优化Screen On/Off 事件 #399

ywjheart opened this issue Feb 15, 2024 · 10 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@ywjheart
Copy link

是否有人曾提过类似的问题?

否(No)

你觉得APP有什么不足之处?

目前已经支持两类Screen 事件, On和Off,能否额外增加2个事件, screen unlocked, screen locked。 因为screen unlocked才是实际使用手机吧。

你觉得该怎么去完善会比较好?【非必答】

No response

@ywjheart ywjheart added the help wanted Extra attention is needed label Feb 15, 2024
@pppscn
Copy link
Owner

pppscn commented Feb 16, 2024

微信图片_20240216214810

已经细分四种事件

**【注意】如果有设置锁屏密码,通常按下电源键息屏时会立即锁定,此时触发的是 locked(不会触发 off);没密码时才会触发 off **

等每周构建版在线编译完成,下载覆盖安装试试

/~https://github.com/pppscn/SmsForwarder/actions/workflows/Weekly_Build.yml

@ywjheart
Copy link
Author

Pixel 6P测试正常,创建了4个任务,包括用NovaLauncher的一键锁屏,不同锁屏方式主要差异在screen off事件。

@ywjheart
Copy link
Author

对不起,刚才测试不仔细,延时似乎无效,虽然不影响使用。默认延迟设置为5分钟,用一键锁屏或者指纹解锁,其实notification是立刻发出的,而不是根据延时设置。

@ywjheart ywjheart reopened this Feb 16, 2024
@ywjheart
Copy link
Author

screen on: 设置为0 可以正确触发。
screen locked、unlocked: 可以正确触发,但延时设置似乎无效。
screen off:不是很容易触发,没什么影响。

@pppscn
Copy link
Owner

pppscn commented Feb 16, 2024

screen locked、unlocked: 可以正确触发,但延时设置似乎无效。

这个问题重新下载测试

screen off:不是很容易触发,没什么影响。

【注意】如果有设置锁屏密码,通常按下电源键息屏时会立即锁定,此时触发的是 locked(不会触发 off);没密码时才会触发 off

@ywjheart
Copy link
Author

ywjheart commented Feb 16, 2024

使用最新构建,设置屏幕超时30s, Pixel6P可以触发Off 事件(看起来没问题了)。
另外我注意到只有On、Off可以设置为0 mins, Locked/Unlocked 无论改成多少,submit,save之后再回来检查依然是5 mins,不过触发实际上是立刻的。(等于这个无变化)

@pppscn
Copy link
Owner

pppscn commented Feb 17, 2024

使用最新构建,设置屏幕超时30s, Pixel6P可以触发Off 事件(看起来没问题了)。 另外我注意到只有On、Off可以设置为0 mins, Locked/Unlocked 无论改成多少,submit,save之后再回来检查依然是5 mins,不过触发实际上是立刻的。(等于这个无变化)

抱歉,我太自大了,直接ctrl+c/v了事,没有测试,忘记初始化赋值了,继续测试吧

@ywjheart
Copy link
Author

我把screen lock, unlocked的延时设置为2mins,测试正常。应该是修好了。

@pppscn
Copy link
Owner

pppscn commented Feb 17, 2024

微信图片_20240217185934

继续测试

@ywjheart
Copy link
Author

我给4个事件设置了1分钟延迟,其中screen locked和screen off设置了recheck. 测试后可以看到只有screen on和screen unlock 消息, 没有screen off 和screen locked消息,所以它们被正确跳过执行了. 我也阅读了你的对应commits. 我也部分检查了导出的logs. 那些被取消执行的情况在log中会记录:recheck condition is not pass.

我认为功能已经正确实现, issue可以closed.

@pppscn pppscn closed this as completed Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants