← 返回首页
Change example 13. by r12a · Pull Request #15 · w3c/string-search · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted
Cancel Submit feedback

Saved searches

Use saved searches to filter your results more quickly

Cancel Create saved search
Appearance settings
Resetting focus

Change example 13.#15

Open
r12a wants to merge 1 commit into
gh-pagesfrom
r12a-patch-1
Open

Change example 13.#15
r12a wants to merge 1 commit into
gh-pagesfrom
r12a-patch-1

Conversation

Copy link
Copy Markdown
Contributor

r12a commented Mar 21, 2023

No description provided.

r12a requested a review from aphillips March 21, 2023 16:10
Copy link
Copy Markdown
Contributor Author

r12a commented Mar 21, 2023

@xfq could you please enable Netlify for this repo (and as many others as you can).

aphillips requested changes Mar 21, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

aphillips left a comment

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

I haven't seen the preview yet, but the change looks reasonable. Note the need to move the graphics and fix the links.

Comment thread index.html
<td class="exampleChar">&#x627;&#x655;</td>
<td><code class="uname" translate="no">U+0627 ARABIC LETTER ALEF</code> + <code class="uname" translate="no">U+0655 ARABIC HAMZA BELOW</code></td>

<td class="exampleChar"><img src="https://r12a.github.io/c/Arabic/large/0625.png" alt="&#x065B;" style="height: 4rem; width: 4rem;"></td>
Copy link
Copy Markdown
Contributor

aphillips Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

Let's be sure to move the graphic to our repo and make the link relative?

Copy link
Copy Markdown
Contributor Author

r12a Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

Yes, i didn't do that pending approval for the approach, but we will need to do that before we can publish the doc to TR.

Comment thread index.html
<td class="exampleChar">&#x6cc;&#x65a;</td>
<td><code class="uname" translate="no">U+06CC ARABIC LETTER FARSI YEH</code> + <code class="uname" translate="no">U+065A ARABIC VOWEL SIGN SMALL V ABOVE</code></td>
<td><strong>Not canonically equivalent</strong><br/>(differences that <em>remain</em> after Unicode Normalization) Many of these are linked to user perception of whether the vowel is part of the base letter (<em lang="ar-Latn" translate="no">ijam</em>) vs. separable (<em lang="ar-Latn" translate="no">tashkil</em> or <em lang="ar-Latn" translate="no">hamza</em>)</td>
<td class="exampleChar"><img src="https://r12a.github.io/c/Arabic/large/0681.png" alt="&#x065B;" style="height: 4rem; width: 4rem;"></td>
Copy link
Copy Markdown
Contributor

aphillips Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

and here

Comment thread index.html
<td><strong>Not canonically equivalent</strong><br/>(differences that <em>remain</em> after Unicode Normalization) Many of these are linked to user perception of whether the vowel is part of the base letter (<em lang="ar-Latn" translate="no">ijam</em>) vs. separable (<em lang="ar-Latn" translate="no">tashkil</em> or <em lang="ar-Latn" translate="no">hamza</em>)</td>
<td class="exampleChar"><img src="https://r12a.github.io/c/Arabic/large/0681.png" alt="&#x065B;" style="height: 4rem; width: 4rem;"></td>
<td><span class="codepoint" translate="no"><span class="uname">U+062D ARABIC LETTER HAH</span> + <span class="uname">U+0654 ARABIC HAMZA ABOVE</span></span> Preferred usage for Kashmiri to represent a consonant+vowel sequence <span class="ipa">hə</span>. The hamza represents the vowel sound.</td>
<td><span class="codepoint" translate="no"><span class="uname">U+0681 ARABIC LETTER HAH WITH HAMZA ABOVE</span></span> Used for Pashto consonant <span class="ipa">d͡z</span>. Can then be combined with other diacritics to indicate vowels.</td>
Copy link
Copy Markdown
Contributor

aphillips Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

Consider entitizing the IPA?

Copy link
Copy Markdown
Contributor Author

r12a Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

We should apply a style to the ipa that puts /../ around the marked up item and probably ensure that we apply a webfont that can show all Latin characters, such as Noto Sans or SIL Gentium Plus.

Comment thread index.html
<td><code class="uname" translate="no">U+0626 ARABIC LETTER YEH WITH HAMZA ABOVE</code></td>
<td class="exampleChar">&#x6cc;&#x654;</td>
<td><code class="uname" translate="no">U+06CC ARABIC LETTER FARSI YEH</code> + <code class="uname" translate="no">U+0654 ARABIC HAMZA ABOVE</code></td>
<td class="exampleChar"><img src="https://r12a.github.io/c/Arabic/large/065B.png" alt="&#x065B;" style="height: 4rem; width: 4rem;"></td>
Copy link
Copy Markdown
Contributor

aphillips Mar 21, 2023

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality Hide comment

file location

Copy link
Copy Markdown
Member

xfq commented Mar 24, 2023

@xfq could you please enable Netlify for this repo (and as many others as you can).

Done. Feel free to push a commit to see the preview.

Copy link
Copy Markdown
Contributor

aphillips commented Aug 22, 2024

Nudging this PR. Can we address the comments and merge?

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

aphillips aphillips requested changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

Footer

© 2026 GitHub, Inc.