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

VS Tizen extension user docs update for .Net and Hybrid App. #1612

Merged
merged 4 commits into from
Mar 29, 2022

Conversation

om90prakash
Copy link
Contributor

Signed-off-by: Om Prakash/Things Platform /SRI-Bangalore/Engineer/Samsung Electronics [email protected]

Change Description

Describe your changes here.

For instance,

  • Fix broken links of native application docs.
  • ...

Bugs Fixed

Provide links to bugs here

For instance,

API Changes

In case of ACR, put the link of ACR

For instance,

  • ACR: ACR-1120

Signed-off-by: Om Prakash/Things Platform /SRI-Bangalore/Engineer/Samsung Electronics <[email protected]>
@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1612

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/application/vstools/Tizen/dotnet.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: application, project, Tizen, file, gt, window, Debug, emulator, break, point, version, Studio, menu, profile, platform, option, dropdown, list, Solution, Explorer, source, code
---

Need help? YAML meta tag guide

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/application/vstools/Tizen/hybrid.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: project, Tizen, application, Project, Solution, Studio, Add, list, studio, Debug, break, point, button, Explorer, window, emulator, solution, selection, Native, .NET, file, source
---

Need help? YAML meta tag guide

Copy link
Contributor

@sangwook96 sangwook96 left a comment

Choose a reason for hiding this comment

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

Thank you very much !!

Copy link
Collaborator

@safir-srbd safir-srbd left a comment

Choose a reason for hiding this comment

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

@om90prakash Hello. Hope you are well. I have reviewed this PR, and have suggested some changes. Please incorporate them into the PR.

docs/application/vstools/Tizen/hybrid.md Outdated Show resolved Hide resolved
The following sections explain how to use Visual Studio Extension for Tizen to develop your hybrid applications. Hybrid application for Tizen supports to have different types(.NET/Native/Web) of project in same Solution/Workspace.


### Create Tizen .Net project
Copy link
Collaborator

Choose a reason for hiding this comment

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

'.NET' should be capitalized.

Suggested change
### Create Tizen .Net project
### Create Tizen .NET project


### Create Tizen .Net project

To create a Tizen .Net project: Refer to [Create Tizen .Net project](dotnet.md)
Copy link
Collaborator

Choose a reason for hiding this comment

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

'.NET' should be capitalized.

Suggested change
To create a Tizen .Net project: Refer to [Create Tizen .Net project](dotnet.md)
To create a Tizen .NET project: Refer to [Create Tizen .Net project](dotnet.md)


To add a Tizen Native project into the above created .NET project to make the Hybrid solution:

1. In the Visual Studio Solution Explorer, Right Click on Solution **Add &gt; New Project**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The words 'right click' should not be capitalized.

Suggested change
1. In the Visual Studio Solution Explorer, Right Click on Solution **Add &gt; New Project**.
1. In the Visual Studio Solution Explorer, right click on Solution **Add &gt; New Project**.


1. In the Visual Studio Solution Explorer, Right Click on Solution **Add &gt; New Project**.

2. For creating Native application, select **C++** and **Tizen** options in the dropdown list. Then, select **Tizen Native project** and click **Next**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The word 'Project' should be capitalized to maintain consistency with the accompanying image.

Suggested change
2. For creating Native application, select **C++** and **Tizen** options in the dropdown list. Then, select **Tizen Native project** and click **Next**.
2. For creating Native application, select **C++** and **Tizen** options in the dropdown list. Then, select **Tizen Native Project** and click **Next**.


![Version selection](media/native_platform_version.PNG)

5. The visual studio window with newly created project appears on the **Solution Explorer**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since 'Visual Studio' is a proper noun, it should be capitalized.

Suggested change
5. The visual studio window with newly created project appears on the **Solution Explorer**.
5. The Visual Studio window with newly created project appears on the **Solution Explorer**.


### Add Project Dependency

1. From Solution explorer Right click on **Startup Project** and select **Add Tizen Project Dependency**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

'Explorer' should be capitalized to maintain consistency with the rest of the document. A missing comma was also added.

Suggested change
1. From Solution explorer Right click on **Startup Project** and select **Add Tizen Project Dependency**.
1. From Solution Explorer, right click on **Startup Project** and select **Add Tizen Project Dependency**.


### Set Startup Project

1. From **Solution Explorer** Right click on the desired Project and select **Set as Startup Project**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

The words 'Right' and 'Project' should not be capitalized here. A missing comma was also added.

Suggested change
1. From **Solution Explorer** Right click on the desired Project and select **Set as Startup Project**.
1. From **Solution Explorer**, right click on the desired project and select **Set as Startup Project**.


![Add Project Dependency](media/hybrid_add_dependency.PNG)

2. A Window with project list for selection will be displayed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Corrected grammatical error.

Suggested change
2. A Window with project list for selection will be displayed.
2. A window with a project list for selection will be displayed.


### Debug your application in emulator

1. Open the .cs file in your visual studio application.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since 'Visual Studio' is a proper noun, it should be capitalized.

Suggested change
1. Open the .cs file in your visual studio application.
1. Open the .cs file in your Visual Studio application.

Copy link
Collaborator

@safir-srbd safir-srbd left a comment

Choose a reason for hiding this comment

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

@om90prakash I have added a few more changes.

![Visual Studio screen](media/hybrid_vs_screen.PNG)


### Set Startup Project
Copy link
Collaborator

Choose a reason for hiding this comment

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

Headings should follow sentence style case.
Ref: https://github.com/Samsung/tizen-docs/blob/master/reviewguide/review_points_guide.md#headings

Suggested change
### Set Startup Project
### Set startup project

1. From **Solution Explorer** Right click on the desired Project and select **Set as Startup Project**.


### Add Project Dependency
Copy link
Collaborator

Choose a reason for hiding this comment

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

Headings should follow sentence style case.
Ref: https://github.com/Samsung/tizen-docs/blob/master/reviewguide/review_points_guide.md#headings

Suggested change
### Add Project Dependency
### Add project dependency

Corrected Grammatical errors.

Co-authored-by: Safir Zawad <[email protected]>
@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1612

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/application/vstools/Tizen/hybrid.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: project, Tizen, application, Project, Studio, solution, Add, list, Solution, studio, Debug, break, point, Explorer, button, emulator, window, selection, file, .net, source, code
---

Need help? YAML meta tag guide

@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1612

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

@TizenDocsBot
Copy link
Collaborator

🔑 [Keyword] docs/application/vstools/Tizen/hybrid.md
No YAML metadata for keywords. You can insert keyword YAML meta at the start of this file as follows:

---
keyword: project, Tizen, application, Studio, Add, window, Project, click, list, solution, Debug, emulator, .net, break, point, button, file, Native, Solution, Explorer, gt, selection
---

Need help? YAML meta tag guide

Copy link
Collaborator

@safir-srbd safir-srbd left a comment

Choose a reason for hiding this comment

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

@om90prakash Thank you for incorporating the suggested changes. It appears you have missed out on two small suggestions, so I have made those suggestions again. Please incorporate them into this PR.


### Set startup project

1. From **Solution Explorer** right click on the desired project and select **Set as Startup Project**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Added a missing comma to improve readability.

Suggested change
1. From **Solution Explorer** right click on the desired project and select **Set as Startup Project**.
1. From **Solution Explorer**, right click on the desired project and select **Set as Startup Project**.


### Add project dependency

1. From Solution Explorer right click on **Startup Project** and select **Add Tizen Project Dependency**.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Added a missing comma to improve readability.

Suggested change
1. From Solution Explorer right click on **Startup Project** and select **Add Tizen Project Dependency**.
1. From Solution Explorer, right click on **Startup Project** and select **Add Tizen Project Dependency**.

@TizenDocsBot
Copy link
Collaborator

checkout the webpage https://docs1.stg.tizen.org/staging/1612

@TizenDocsBot
Copy link
Collaborator

Check Broken Link: Not found any broken link

Copy link
Collaborator

@safir-srbd safir-srbd left a comment

Choose a reason for hiding this comment

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

@om90prakash Thank you for incorporating all suggested changes.

@mijin85cho @asm-foysal I am approving this PR for merging.

@asm-foysal asm-foysal merged commit abc7fcc into Samsung:master Mar 29, 2022
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.

5 participants