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

Can't move up or down #2

Closed
Kronos256 opened this issue Mar 3, 2021 · 12 comments
Closed

Can't move up or down #2

Kronos256 opened this issue Mar 3, 2021 · 12 comments
Labels
bug Something isn't working

Comments

@Kronos256
Copy link

When the app is installed and enable I can't move up or down on my TCL TV home screen. In settings and app drawer it works just fine but not on the home screen I tried reinstalling it but nothing changes as soon as I enable it I can't move up or down only left and right works on the home screen.

@virresh
Copy link
Owner

virresh commented Mar 4, 2021

Would you be able to help with debugging the issue / provide android logs?

I don't own a TCL TV so don't know what's the exact issue.

@Kronos256 Kronos256 reopened this Mar 10, 2021
@Kronos256
Copy link
Author

Kronos256 commented Mar 10, 2021

Would you be able to help with debugging the issue / provide android logs?

I don't own a TCL TV so don't know what's the exact issue.

Sorry I wasn't home last week so I couldn't send a LogCat.
I just tested the new release (v1.02), I thought it fixed my issue but it didn't.

As promised here is the LogCat:
https://filebin.ca/5uSss4XGYrQb/logcat_03-10-2021_17-01-17.zip

Let me know if you need anything else.

@virresh
Copy link
Owner

virresh commented Mar 12, 2021

Thanks Kronos.
Couple of questions:

  1. Are you sure this is the latest release? I can see only parts of the log (i.e keypresses only). Many useful events that were logged in my mouse emulation engine are not visible here. It's possible they might've been eaten up due to system clogging, but there isn't even a single one of them....
    The logs that I'm after are essentially this line of code:

    Log.i(LOG_TAG, "Root Node ======>>>>>" + ((node != null) ? node.toString() : "null"));

  2. Can you confirm the package name of your tv launcher. I might make some exceptions just for that app and release it with next set of changes. Will ask you to confirm when they're ready.

@lmpina
Copy link

lmpina commented Mar 30, 2021

This may have something to do with the software display size setting (which affects the size of objects on the screen). On a typical TV box, there are four display size adjustments that can generally be made in Hidden Settings. When display size is at default (2nd setting), MATVT can scroll in all four directions. When the display size is maximized to largest (4th size), MATVT can only scroll up and down. It can not scroll left and right. When display size is reduced to larger (3rd size), MATVT is able to scroll in all four directions again. I know that I've described the exact opposite of your problem, but display size, magnification and resolution are all worth checking...

@virresh
Copy link
Owner

virresh commented Mar 31, 2021

Thanks for the inputs lmpina
Ideally MATVT should be able to scroll irrespective of display size, but the description you gave makes sense a bit.
I will have to adjust the length of swipe sent when scrolling to a larger value when on a larger display screen. (Seems like the finger movement during swipe isn't sufficient for large screens).
If the issue is really what I think it is,
(i) On the largest screen size, scrolling left and right instead sends clicks on the underlying action button?
(ii) The left right scrolling issue is reproducible everywhere and not just on the TV Home screen?

@lmpina
Copy link

lmpina commented Apr 1, 2021

For clarification purposes, two Settings apps were factory installed by a recent firmware update on the H96Max RK3318 shown in "Factors affecting pointer accuracy."

  1. Larger gear icon - Settings, version 9.com.android.settings
    2, Smaller gear icon - Settings, v1.0, com.android.tv.settings

The path to the Display size setting interacting with MATVT (Largest size causing no vertical scroll) is:

Smaller gear icon → Advanced (Display size, Screen saver, Device theme) → Display → Display size

20210331_171933

Changing the Display size here is like hitting Control + and Control - in a browser. It makes everything easier to read on the screen, just what's needed at a leanback distance when someone needs new glasses.

Also, for what it's worth. Larger gear icon → Display → Resolution was set to Auto; Larger gear icon Display → Zoom was adjusted so that the red, unfilled, test rectangle (adjusting VHeight and HWidth) was perfectly fit to the screen.

This issue is no big deal, IMHO, since MATVT is NOT the only app to balk at the Largest size, and it's very easy to resolve - just back off to Larger or less.

@sportica
Copy link

sportica commented Apr 4, 2021

I had the same problem with Kronos256. And it was fixed when I changed the feedback type to 'feedbackHaptic' in the accessibility service config file.

feedbacktype

@virresh
Copy link
Owner

virresh commented Apr 8, 2021

Thanks @sportica !

I'll update the feedback type.
I wonder if the feedback type would clash on some other devices though, but we'll see about it when we get there. This fix should suffice in the meanwhile 😄

@CaioTheBrain
Copy link

The same happens to me in a Mi Box running Android TV 9, Android TV Home version 3.1.2-366946414-f and Android TV Core Services version 3.1.1-362554353-f.

Once Mouse Toggle Service is enabled at accessibility settings, the D-pad control can't move up or down in the homescreen, only left and right.

@virresh virresh added the bug Something isn't working label Jun 20, 2021
@virresh
Copy link
Owner

virresh commented Jun 26, 2021

@CaioTheBrain @Kronos256
Released v1.04 (https://github.com/virresh/matvt/releases/tag/v1.0.4) with the above fix

Do try and let me know if there you still same issues. Thanks @sportica once again!

@CaioTheBrain
Copy link

v 1.0.4 fixed it for me.

Thank you @virresh.

@virresh
Copy link
Owner

virresh commented Jun 27, 2021

Great
I'll close this issue then.
Feel free to comment / re-open if needed.

@virresh virresh closed this as completed Jun 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

5 participants