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

UI Picker dialog doesn't stop loading repo= URL #3329

Open
8 tasks
ewpatton opened this issue Jan 17, 2025 · 2 comments · May be fixed by #3353
Open
8 tasks

UI Picker dialog doesn't stop loading repo= URL #3329

ewpatton opened this issue Jan 17, 2025 · 2 comments · May be fixed by #3353

Comments

@ewpatton
Copy link
Member

Describe the bug

When the UI picker is presented to users on first run and there is a repo= query parameter, the presentation of the dialog does not pause the startup sequence, causing the project to be loaded and shown behind the picker dialog.

Affects

  • Designer
  • Blocks editor
  • Projects Explorer
  • Companion
  • Compiled apps
  • Buildserver
  • Debugging
  • Other... (please describe)

Expected behavior

No additional work should be done until the selection is made.

Steps to reproduce

  1. Build a fresh copy of master
  2. Copy one of our existing tutorial URLs from the website and substitute localhost:8888 for ai2.appinventor.mit.edu
  3. Go through the login, TOS pages to get to the UI picker dialog. Note the project loading in the background

Note that this may also affect loading from the gallery but I did not test that.

@cindyloo
Copy link

I opened up
https://ai2.appinventor.mit.edu/?locale=en&repo=http%3A%2F%2Fappinventor.mit.edu%2Fyrtoolkit%2FAI%2FaiaFiles%2FimageClassification%2FWhatIsIt.asc in incognito mode. I went through the login, but am unclear as to why it would be a bad thing for the repo to load. I did not see the picker dialog..

@cindyloo
Copy link

it must be a new user so that ShowUIPicker is True in UserSettings

    return userSettings.getSettings(SettingsConstants.USER_GENERAL_SETTINGS)
            .getPropertyValue(SettingsConstants.SHOW_UIPICKER).equalsIgnoreCase("True");
  }```

called in `handleQueryString()`

@ewpatton ewpatton linked a pull request Jan 27, 2025 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants