Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

[TabLayout] setScrollPosition is not working correctly

See original GitHub issue

Calling setScrollPosition() with updateSelectedText = true and updateIndicatorPosition = true doesn’t set the indicator to the specified position:

Actual result	Expected result

Material Library version: 1.8.0-alpha03

Issue Analytics

State:
Created 10 months ago
Comments:6 (6 by maintainers)

Top GitHub Comments

1reaction

imhappicommented, Nov 24, 2022

Ah I see that the onLayout call isn’t expected and is caused by updating the tab visuals to the view selection

I do agree that although this method really shouldn’t be being used, the previous behavior should be re-instated, will re-open this issue and look into this more closely after the holidays

0reactions

pubiqqcommented, Nov 24, 2022

If your logic is correct, why do you need updateIndicatorPosition at all if it is ignored, and the position of the indicator is updated in onLayout() anyway?

I agree, the method has a very unfortunate name, it has a dual purpose, has some problems with documentation, etc. But this method used to work properly, and I think consumers expect it to continue to work or some alternative will be given.