Minor fixes in about.html and other static pages under / #389
Changes from 3 commits
ee934e1
fc25a00
101facb
9bbce18
ba83f83
9b8577b
ed6e0c7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -53,7 +53,7 @@ <h2>Audit your web page</h2> | |
<h2>Analyze for improvements</h2> | ||
</div> | ||
<p> | ||
Review the audits report to identify areas in which your page can be improved. Each day we'll generate a new report for you. | ||
Review the audits report to identify areas in which your page can be improved. Each day we’ll generate a new report for you. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is the smart quote on purpose? just curious There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I was actually trying to figure out if this is a thing in the existing content, but wasn't able to check the code out on my laptop. In any case, I think we need to revert this for now. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It uses smart quotes throughout the rest of the text, so I assumed it was missing. Can’t rebase now, just about to board my plane to the US and on my phone only. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I fixed them across the repo now. @robdodson PTAL |
||
</p> | ||
</div> | ||
<div class="profile-step"> | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1 @@ | ||
/google/src/head/depot/google3/devrel/web/dev/devsite.js | ||
C:/google/src/head/depot/google3/devrel/web/dev/devsite.js | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think this is going to break on all non-Windows systems. Could you revert this change? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Huh, sorry, this seems like VS Code magic. Undone. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apparently 'borne' is actually correct english. TIL