Skip to content
This repository has been archived by the owner on Apr 30, 2024. It is now read-only.

Refine Create Project from Template progress notification messages and increments #58

Closed
7 tasks done
RandomFractals opened this issue Jun 3, 2023 · 6 comments
Closed
7 tasks done
Labels
enhancement New feature or request onboarding Onboarding feature or improvement UX User experience feature or improvement

Comments

@RandomFractals
Copy link
Contributor

RandomFractals commented Jun 3, 2023

  • Refine Open new Workspace prompt when trying to create new project in an existing workspace with files.
  • Revert template repo cloning progress increment changes made in Prompt to Install Dependencies after new project from a template is created to get started with the new Evidence app #56
  • Change Progress message title to shorter Create Project
  • Shorten other notification messages for the different steps in template project creation process.
  • Show incremental info messages during repo cloning in the progress notification message just like we do in the Evidence Output view
  • Add started cloning template repo info message to the progress display
  • Bump Install Dependencies delay to 30 secs in build.ts

Originally posted by @RandomFractals in #57 (comment)

@RandomFractals RandomFractals added enhancement New feature or request UX User experience feature or improvement onboarding Onboarding feature or improvement labels Jun 3, 2023
RandomFractals added a commit that referenced this issue Jun 3, 2023
and add last progress report with 100 on finish
@RandomFractals
Copy link
Contributor Author

RandomFractals commented Jun 3, 2023

@archiewood see the changes applied above for the progress increment, open new workspace prompt, and create project from template progress messages.

The issue you recorded in #57 with Finished creating message lingering is very minor because eventually it goes away, and there is only 15 sec delay timer set for the github repo cloning. Eventually that message goes away, and you can expand it to see full message and the final Finished step.

I bumped that clone repo delay timer and progress display to 25 secs to reproduce it on windows 10 b/c that timing was set to expire perfectly in my prior testing.

See recorded finished step expand delay with the longer timer. We'll use the 15 sec as is now, and can revisit this when we switch to embedding template project content into extension package for faster cloning and creation.

evidence-create-project-from-template-progress-finished-2

@RandomFractals RandomFractals reopened this Jun 3, 2023
RandomFractals added a commit that referenced this issue Jun 3, 2023
- change with progress notification title
- shorten other progress messages
- show cloning template info messages in the progress notifications too
- bump install dependencies delay to 30 secs in build.ts
RandomFractals added a commit that referenced this issue Jun 3, 2023
@RandomFractals
Copy link
Contributor Author

Final Create project from a template UX from start to finish.

Note: We can't improve it more unless we embed template project in the extension package code under /templates.

evidence-create-project-from-template-and-run

@RandomFractals
Copy link
Contributor Author

Shorter template prompt and dialog title:

image

@RandomFractals
Copy link
Contributor Author

RandomFractals commented Jun 4, 2023

New error message display when the provided url is not a github url. We also reopen template dialog with the default template displayed for the users to use it instead, or to provide a valid github url:

image

RandomFractals added a commit that referenced this issue Jun 4, 2023
and skip cache info messages display when logging to Output view
@RandomFractals
Copy link
Contributor Author

Refined repo cloning git info messages display in Output view:

image

@archiewood
Copy link
Member

RandomFractals added a commit that referenced this issue Jun 6, 2023
it is better for this progress notification to linger longer like most vscode notifications stay open for a few seconds longer, than closing it prematurely
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request onboarding Onboarding feature or improvement UX User experience feature or improvement
Projects
None yet
Development

No branches or pull requests

2 participants