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

[MobileUI] Work around some issues with the initial swipe index #302

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Vogtinator
Copy link
Member

Sometimes ListView doesn't react to positionViewAtIndex even after completion.
Set currentIndex to the drawer works around that.

To avoid seeing the initial state briefly, make it invisible at first.

Additionally, show the config page for the initial setup. This is obviously more user friendly at the beginning, but I wonder whether that makes it less obvious how to get out of the configuration dialog. It would be nice to use the back button, but that appears broken because of #301 currently.

Needs some testing on Android and Windows at least whether this improves the situation.

Sometimes ListView doesn't react to positionViewAtIndex even after completion.
Set currentIndex to the drawer works around that.

To avoid seeing the initial state briefly, make it invisible at first.

Additionally, show the config page for the initial setup.
Somehow it looks odd if an area gets brighter while an adjacent area gets
darker. Avoid that by just not darkening the drawer.
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.

None yet

1 participant