Skip to content

Conversation

Eskibear
Copy link
Contributor

As discussed, we will have 4 steps for users to follow:

  • create a new project/open a sample project
  • explore projects, e.g. view symbols
  • run projects
  • view live information

I'll update SVG files with new commits once they are ready.

@Eskibear
Copy link
Contributor Author

Changes in cf7043e :

  • Step 1: add a button to git.clone(vscode's command) petclinic into a location you choose. vscode handles the UX.
  • Step 2: remove the instruction about adding "@" in the workspace symbols inputbox as we now pre-input "#@" when you click the command.
  • Step 3: add a button to run the project
  • Step 4: add buttons to open beans and mappings views. Memory view is not showing all the time, we can show the memory graph in the image, telling users that we have this fancy feature.

@martinlippert
Copy link
Member

The changes look good to me, just missing the SVG files.

@Tanyatang0
Copy link

Here are the SVG files.
Step1_commandPalette
Step1_commandPalette
Step2_Explore your project
Step2_Explore your project
Step3_Run your project
Step3_Run your project
Step4_View live information from your running application
Step4_View live information from your running application
Attached is the image zip file. @Eskibear
Steps image file.zip

@Eskibear

This comment was marked as outdated.

Signed-off-by: Yan Zhang <[email protected]>
Signed-off-by: Yan Zhang <[email protected]>
@Eskibear Eskibear marked this pull request as ready for review December 5, 2022 11:44
@Eskibear
Copy link
Contributor Author

Eskibear commented Dec 5, 2022

@martinlippert I just updated the themed SVGs (particularly optimized for not being strange in all built-in themes). Please take a look.

@martinlippert
Copy link
Member

Looks all good to me. Let's merge this.

@martinlippert martinlippert merged commit 832cfc3 into spring-projects:main Dec 5, 2022
@martinlippert martinlippert added this to the 4.17.0.RELEASE milestone Dec 5, 2022
@Eskibear Eskibear deleted the walkthru branch December 6, 2022 02:28
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.

3 participants