Skip to content

Conversation

@jdneo
Copy link
Contributor

@jdneo jdneo commented Jan 29, 2022

Signed-off-by: sheche sheche@microsoft.com

Signed-off-by: sheche <sheche@microsoft.com>
@jdneo
Copy link
Contributor Author

jdneo commented Jan 29, 2022

Hi @jperedadnr,

This is the PR to initialize the getting started document for VS Code.

One question: I noticed that the existing screenshots all have a white colored margin as the background, the screenshots in this PR does not have that, do I need to add them? Or could I directly use css margin style?

@jdneo
Copy link
Contributor Author

jdneo commented Feb 7, 2022

@jperedadnr would you mind to take a look at this PR?

@jperedadnr
Copy link
Contributor

Ok, we'll look into it.
About screenshots, we typically take them on macOS, which adds this shadow effect, but it is not mandatory.

@jdneo
Copy link
Contributor Author

jdneo commented Feb 11, 2022

Hi @jperedadnr,

We post a tweet on Twitter yesterday about JavaFX support in VS Code, looks like people are really existing about JavaFX support in VS Code. The tweet's PV data looks amazing too:

0d57fbab-9364-4945-a55f-45cdff054e96

It would be really appreciated if the team can get this PR reviewed and merged, which I think can provide more help to the JavaFX and VS Code developers. 😄

@abhinayagarwal
Copy link
Collaborator

abhinayagarwal commented Feb 17, 2022

The screenshots are very close to the paragraph text.

image

Adding the following style to gluon.css makes it look much better:

/* Visual Code */
#IDE-VSCode img {
    padding: 1em;
}

image

Copy link
Collaborator

@abhinayagarwal abhinayagarwal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall the PR LGTM. Left a few minor comments.


<a href="images/ide/vscode/gradle/vscode05.png" target="_blank"><img src="images/ide/vscode/gradle/vscode05.png" alt="Gradle dependencies"/></a>

As for any other Gradle dependencies, these jars can be found in the local <kbd>.gradle</kbd> repository.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get this statement. What other Gradle dependencies are we referring to?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied this from the Eclipse tutorial, see:

  • Non-modular with Maven > 2. Verify the pom
  • Non-modular with Gradle > 2. Modify the build

I guess it intends to tell the readers that the build tool will manage all the dependencies from different projects into those folders?

I agree that this statement somehow introduces confusions and I'm ok to remove it.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In that case I would keep these as it is and update the sentences for all IDE together in a later PR.


<a href="images/ide/vscode/maven/vscode04.png" target="_blank"><img src="images/ide/vscode/maven/vscode04.png" alt="Update the pom"/></a>

As for any other maven dependencies, these jars can be found in the local <kbd>.m2</kbd> repository.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get this statement. What other Maven dependencies are we referring to?


<a href="images/ide/vscode/modular/maven/vscode04.png" target="_blank"><img src="images/ide/vscode/modular/maven/vscode04.png" alt="Update the pom"/></a>

As for any other maven dependencies, these jars can be found in the local <kbd>.m2</kbd> repository.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get this statement. What other Maven dependencies are we referring to?


<a href="images/ide/vscode/modular/gradle/vscode05.png" target="_blank"><img src="images/ide/vscode/modular/gradle/vscode05.png" alt="Gradle dependencies"/></a>

As for any other Gradle dependencies, these jars can be found in the local <kbd>.gradle</kbd> repository.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't get this statement. What other Gradle dependencies are we referring to?

 - fix the css style
 - refine the wording

Signed-off-by: sheche <sheche@microsoft.com>
@jdneo
Copy link
Contributor Author

jdneo commented Feb 18, 2022

@abhinayagarwal Thank you for reviewing the PR. I've addressed some of the comments.

As of the statements of other Maven dependencies and other Gradle dependencies, I'm ok to remove them.

Waiting for your suggestion and I'll make follow-up actions if needs.

@abhinayagarwal abhinayagarwal merged commit 703b59f into openjfx:master Feb 20, 2022
@jdneo jdneo deleted the cs/vscode branch February 21, 2022 00:51
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