close
Skip to content

Correct comment inside script tag and style tag in razor file#48345

Merged
aeschli merged 2 commits intomicrosoft:masterfrom
Jasonlhy:master
May 2, 2018
Merged

Correct comment inside script tag and style tag in razor file#48345
aeschli merged 2 commits intomicrosoft:masterfrom
Jasonlhy:master

Conversation

@Jasonlhy
Copy link
Copy Markdown
Contributor

By default, all comments inside script tag and style tag are <!-- and --->
This pull request aims to correct the comment.

Before

<script>
    <!-- var a = 1;
    var a = 1; -->
</script>

<style>
    a {
        <!-- font-size: 1;  -->
    }
</style>

@foreach (var item in Model) {
   <tr>
      <td>
         @Html.ActionLink("Edit", "Edit", new { id = item.ID }) |
         @Html.ActionLink("Details", "Details", new { id = item.ID }) |
         @Html.ActionLink("Delete", "Delete", new { id = item.ID })
      </td>
		
      <td>
         @item.Name
      </td>
		
      <td>
         @String.Format("{0,g}", item.JoiningDate)
      </td>
   </tr>
}

After

<script>
    // var a = 1;
    // var a = 1;
</script>

<style>
    a {
        /* font-size: 1;  */
    }
</style>

@foreach (var item in Model) {
   <tr>
      <td>
         @Html.ActionLink("Edit", "Edit", new { id = item.ID }) |
         @Html.ActionLink("Details", "Details", new { id = item.ID }) |
         @Html.ActionLink("Delete", "Delete", new { id = item.ID })
      </td>
		
      <td>
         @item.Name
      </td>
		
      <td>
         @String.Format("{0,g}", item.JoiningDate)
      </td>
   </tr>
}

@msftclas
Copy link
Copy Markdown

msftclas commented Apr 22, 2018

CLA assistant check
All CLA requirements met.

@aeschli aeschli added this to the May 2018 milestone May 2, 2018
@aeschli aeschli added the languages-basic Basic language support issues label May 2, 2018
@aeschli aeschli merged commit b6d7254 into microsoft:master May 2, 2018
@aeschli
Copy link
Copy Markdown
Contributor

aeschli commented May 2, 2018

Thanks @Jasonlhy !

@aeschli aeschli added the verification-needed Verification of issue is requested label May 29, 2018
@github-actions github-actions Bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

languages-basic Basic language support issues verification-needed Verification of issue is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants