Revert "Show spinner when opening/creating a project, take #2 (#6827)"

[farmaazon]

This reverts commit 0ed78f9.

Fixes #7163

As it turned out, the issue with missing retries is quite important, because IDE does not launch on weak hardware. Reverting that commit is an easy way to fix it (and the loading spinner could be implemented in a better way, with cooperation with the new dashboard).

Pull Request Description

Important Notes

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

• The documentation has been updated, if necessary.
• [ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
• All code follows the
  Scala,
  Java,
  and
  Rust
  style guides. In case you are using a language not listed above, follow the Rust style guide.
• All code has been tested:
  • Unit tests have been written where possible.
  • If GUI codebase was changed, the GUI was tested when built using ./run ide build.

[farmaazon]

I expected the revert will just remove the "second" loading screen, but it's still there giving strange effect:

strange-2023-07-06_14.51.04.mp4

@PabloBuchu @somebody1234 as you know the dashboard more or less, do you know if there is some code in dashboard which makes the second loading screen? I haven't found anything in rust code.

[somebody1234]

oh you mean does the dashboard explicitly run a loading spinner? afaict no - the dashboard code actually doesn't have a full-screen loading spinner (nor one with the logo)
it's possible the entry point triggers the spinner somehow via the loader, but i don't think that should happen either.

i can see some logs are seemingly duplicated, may that be related?
it reminds me of React logging things twice in its debug mode, so it may be caused by debug mode?

[somebody1234]

if it's possible to set a breakpoint on show_progress_indicator that might help with debugging?
alternatively you could try -authentication=false -feature=preview.new-dashboard=false -startup.project=<project here> which should be equivalent (and so should theoretically show only one spinner, if the dashboard is probably the cause)

[jdunkerley]

Testing on a weak VM this fix works and can get a not very usable but at least working instance of Enso.

[farmaazon]

Hmm, the effect is gone after merging develop. I like it.

Now, I only need an owner's accept. @MichaelMauderer or @wdanilo