elastic-rock / KeepScreenOn

An Android screen timeout quick settings tile
GNU General Public License v3.0
131 stars 4 forks source link

MIUI sets timeout to 10 minutes when screen timeout settings are opened and KeepScreenOn is enabled #11

Closed islam2hamy closed 1 year ago

islam2hamy commented 1 year ago

Hello ,

restore the previous timeout value dosn't work

Steps : add the app quick settings tile activet it deactivet it

the screen timeout dosn't restore the previous timeout value

android : 13 miui : 14.0.12 app v. : 1.5.0

elastic-rock commented 1 year ago

My first question is, whether the bug occurs in version 1.6.0? If no, it would be more than ideal but that is rather a hope. :slightly_smiling_face:

If yes, I just want to ask you based on what do you assume that the screen timeout is not restored to the previous value? I tried it on MIUI version 14.0.2 (also Android 13) with version 1.5.0 and didn't encounter any issues. :thinking:

islam2hamy commented 1 year ago

My first question is, whether the bug occurs in version 1.6.0?

Yes

If yes, I just want to ask you based on what do you assume that the screen timeout is not restored to the previous value? I tried it on MIUI version 14.0.2 (also Android 13) with version 1.5.0 and didn't encounter any issues. 🤔

the screen timeout in the phone setting is still 10 m

ps. system language is arabic

elastic-rock commented 1 year ago

The maximum timeout value in the system settings is 10 minutes on MIUI v14. When I activated the tile and then visited the settings tab where screen timeout can be set, the system set the timeout by itself to 10 minutes, because 10 minutes is the maximum the user can choose from the settings. If one doesn't visit the settings when the tile is active, it works without problems (on my side). I don't know why they do it that way, because on pixel devices, it just shows null and doesn't do stuff you don't want it to do. It may be that you are experiencing this issue. Does the timeout restore correctly, when you don't visit the settings? If you need to, you can check the current timeout using adb shell settings get system screen_off_timeout.

islam2hamy commented 1 year ago

The maximum timeout value in the system settings is 10 minutes on MIUI v14. When I activated the tile and then visited the settings tab where screen timeout can be set, the system set the timeout by itself to 10 minutes, because 10 minutes is the maximum the user can choose from the settings. If one doesn't visit the settings when the tile is active, it works without problems (on my side).

yes , and that is fine with me

I don't know why they do it that way, because on pixel devices, it just shows null and doesn't do stuff you don't want it to do. It may be that you are experiencing this issue.

yes, but not matter , 10 minutes is ok for me

Does the timeout restore correctly, when you don't visit the settings?

deactivate the tile doesn't restore the setting , e.x. 30 second

Turhvjbufv commented 1 year ago

+1 Deactivating through the quick settings tile also doesn't restore previous value (1m) on s10

elastic-rock commented 1 year ago

I have pushed out a new update (v1.11.0) which includes a small label inside the app that shows the current screen timeout. Could you please make a screen recording reproducing the bug and returning to the app (to see the label) each time the timeout value supposedly changed? Thank you.

P.S. The update has not yet reached F-Droid but you may get it from GitHub as the signing keys are the same.

islam2hamy commented 1 year ago

i have found that , opening the screen timeout in the phone setting making the tile deactiveted

Steps : add the app quick settings tile activet it open the screen timeout in the phone setting Go back to the tile

the tile is deactiveted !

Turhvjbufv commented 1 year ago

I have pushed out a new update (v1.11.0) which includes a small label inside the app that shows the current screen timeout. Could you please make a screen recording reproducing the bug and returning to the app (to see the label) each time the timeout value supposedly changed? Thank you.

P.S. The update has not yet reached F-Droid but you may get it from GitHub as the signing keys are the same.

The new update seems to have fixed the problem, but I didn't play much with it since I don't have much time currently

elastic-rock commented 1 year ago

Any updates on the matter?

Turhvjbufv commented 1 year ago

For me the new update fixed the problem

islam2hamy commented 1 year ago

Any updates on the matter?

For me

i have found that , opening the screen timeout in the phone setting making the tile deactiveted Steps : add the app quick settings tile activet it open the screen timeout in the phone setting Go back to the tile the tile is deactiveted !

elastic-rock commented 1 year ago

Any updates on the matter?

For me

i have found that , opening the screen timeout in the phone setting making the tile deactiveted Steps : add the app quick settings tile activet it open the screen timeout in the phone setting Go back to the tile the tile is deactiveted !

But otherwise, is it working as expected?

islam2hamy commented 1 year ago

Any updates on the matter?

For me

i have found that , opening the screen timeout in the phone setting making the tile deactiveted Steps : add the app quick settings tile activet it open the screen timeout in the phone setting Go back to the tile the tile is deactiveted !

But otherwise, is it working as expected?

yes

elastic-rock commented 1 year ago

The fact that opening the settings panel sets the timeout to 10min is a feature of MIUI and not something that can be prevented by the app. Therefore, I am closing this issue as everything else is working as expected. If something doesn't work, I will be happy to help.