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

Update the tools page #593

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from
Open

Update the tools page #593

wants to merge 2 commits into from

Conversation

xfq
Copy link
Member

@xfq xfq commented Jun 21, 2024

I'll redirect https://www.w3.org/International/tools to the new page after merging this PR.


Preview

@xfq xfq requested a review from r12a June 21, 2024 06:22
Copy link

netlify bot commented Jun 21, 2024

Deploy Preview for i18n-drafts ready!

Name Link
🔨 Latest commit ae5329c
🔍 Latest deploy log https://app.netlify.com/sites/i18n-drafts/deploys/66ab78d4a114270009e1eee5
😎 Deploy Preview https://deploy-preview-593--i18n-drafts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

pages/tools.en.html Outdated Show resolved Hide resolved
Browser-based app. Converts between characters, Unicode code point numbers, UTF-8 and UTF-16 code units in hex, Numeric Character References (hex and decimal), percent-encoded text, and other character escape formats.</p></li>
<li>
<p><a href="https://r12a.github.io/uniview/">UniView</a><br>
Browser-based app.
UniView is a Unicode character viewer/picker that displays characters from a selected range or search pattern, provides information about them, and allows you to assemble strings from them like in a character map. You can also create customizable lists of characters for pasting into documents. UniView supports the latest version of the Unicode Standard, including the supplementary planes as well as the BMP. Characters are displayed as graphics by default, but you can also display them using an installed font.</p>
</li>
<li>
<p><a href="https://r12a.github.io/apps/encodings/">Encoding converter</a><br>
<p><a href="https://r12a.github.io/app-analysestring/index.html">Analyse string tool</a><br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Seems a bit twisted. Perhaps:

Suggested change
<p><a href="https://r12a.github.io/app-analysestring/index.html">Analyse string tool</a><br>
<p><a href="https://r12a.github.io/app-analysestring/index.html">String analysis tool</a><br>

Copy link
Member Author

Choose a reason for hiding this comment

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

I discussed it with @r12a, and since this is the name of the tool, he suggested keeping the original name.

Copy link
Contributor

Choose a reason for hiding this comment

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

Well, the name of the app is "Analyse string", so i think @xfq has it right. However, a possible compromise may be to move the word 'tool' outside the link text.

<p><a href="https://r12a.github.io/apps/encodings/">Encoding converter</a><br>
<p><a href="https://r12a.github.io/app-analysestring/index.html">Analyse string tool</a><br>
Browser-based app.
Lists the characters in text that you paste into the large box at the top, and produces information about each character.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Lists the characters in text that you paste into the large box at the top, and produces information about each character.
List the characters in the text you supply (typically by pasting into the large box at the top), producing information about each character.

Copy link
Member Author

Choose a reason for hiding this comment

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

To make it more consistent, I used third-person singular like the rest of this page.

@r12a What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

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

@aphillips your suggestions don't follow the pattern of the blurbs in the rest of the page, so i think the original is good.

Co-authored-by: Addison Phillips <[email protected]>
Browser-based app. Converts between characters, Unicode code point numbers, UTF-8 and UTF-16 code units in hex, Numeric Character References (hex and decimal), percent-encoded text, and other character escape formats.</p></li>
<li>
<p><a href="https://r12a.github.io/uniview/">UniView</a><br>
Browser-based app.
UniView is a Unicode character viewer/picker that displays characters from a selected range or search pattern, provides information about them, and allows you to assemble strings from them like in a character map. You can also create customizable lists of characters for pasting into documents. UniView supports the latest version of the Unicode Standard, including the supplementary planes as well as the BMP. Characters are displayed as graphics by default, but you can also display them using an installed font.</p>
</li>
<li>
<p><a href="https://r12a.github.io/apps/encodings/">Encoding converter</a><br>
<p><a href="https://r12a.github.io/app-analysestring/index.html">Analyse string tool</a><br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Well, the name of the app is "Analyse string", so i think @xfq has it right. However, a possible compromise may be to move the word 'tool' outside the link text.

<p><a href="https://r12a.github.io/apps/encodings/">Encoding converter</a><br>
<p><a href="https://r12a.github.io/app-analysestring/index.html">Analyse string tool</a><br>
Browser-based app.
Lists the characters in text that you paste into the large box at the top, and produces information about each character.
Copy link
Contributor

Choose a reason for hiding this comment

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

@aphillips your suggestions don't follow the pattern of the blurbs in the rest of the page, so i think the original is good.

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