シャイニーカラーズのユニットに読み仮名を追加 #495
Merged
+10
−0
Conversation
This file contains 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
Member
arrow2nd
commented
Mar 15, 2022
arrow2nd
commented
- 各ユニットに読み仮名を追加しました (ref. ユニットのよみがな #422)
banjun
reviewed
Mar 15, 2022
| @@ -83,6 +90,7 @@ | |||
| <rdf:Description rdf:about="Team%2EStella"> | |||
| <schema:name rdf:datatype="http://www.w3.org/2001/XMLSchema#string">Team.Stella</schema:name> | |||
| <rdfs:label rdf:datatype="http://www.w3.org/2001/XMLSchema#string">Team.Stella</rdfs:label> | |||
| <imas:nameKana xml:lang="ja">ちーむ すてら</imas:nameKana> | |||
There was a problem hiding this comment.
マージブロックってほどでは全然ないんですが
imas:nameKanaにスペース入れる是非ってどんなもんなんでしょうかね。(「ちーむ すてら」vs「ちーむすてら」)
- 発音的な読み仮名としてみる → スペースがあるとリズム的に休符的な意味が表わせる(かも)
- 平仮名からschema:nameへの変換関係としてみる → スペースは変換を発動することが多そうなので，単語の読みとしてスペースを入れるのは適切ではなさそう
わたしは変換関係を利用しているので，そっち側のユースケースで気になりますが，それはそれで回避としてスペース除去すれば変換関係になるというのは満たせるし，単にスペース休符アリ読み仮名と捉えるのも一理あるので，どっちもアリになりうるかなと思いました
There was a problem hiding this comment.
個人的にはスペースなしの方が読み仮名としては自然なのかなと思うのですが、既に登録されている C.FIRST の読み仮名が くらす ふぁーすと となっていたので、それに合わせた感じです。
imasparql/RDFs/Units/SideM.rdf
Line 178 in 7bacbb0
banjun
approved these changes
Mar 15, 2022
crssnky
approved these changes
Mar 26, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.