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

一部の表記修正と日本語対応 #674

Merged
merged 5 commits into from
Mar 28, 2024
Merged

一部の表記修正と日本語対応 #674

merged 5 commits into from
Mar 28, 2024

Conversation

Sayamame-beans
Copy link
Collaborator

resolve: #619
fix: #620
partially fix (not closable): #648
fix: #649

"description for show prerelease packages"でbrタグを使用したいため、追加で許可する必要があります。(未対応)
ただ、そもそもこの部分、小見出し的な位置なのに説明文のみ載せているのが違和感があります。
何か別途小見出しを用意するか、説明の表示箇所をチェックボックスの下に移動するなどが望ましいのでしょうか…?

@Sayamame-beans Sayamame-beans added vrc-get-gui Issues for vrc-get-gui localization Improvements or additions to localization labels Mar 27, 2024
@anatawa12
Copy link
Member

description for show prerelease packages、見出しに適切なテキストが思いつかなかったためdescriptionのみにしたという経緯があるので分ける適切なテキストがあれば分けたいです

@anatawa12
Copy link
Member

brタグが使える話はTransコンポーネントの仕様のようですね
許可されたタグの一覧的に問題なさそうなのでそのままにします。 https://react.i18next.com/latest/trans-component#usage-with-simple-html-elements-like-less-than-br-greater-than-and-others-v10.4.0

vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Show resolved Hide resolved
vrc-get-gui/locales/ja.json5 Outdated Show resolved Hide resolved
@Sayamame-beans
Copy link
Collaborator Author

なんか、エラー出ますね…(翻訳作業中のデフォルトjaからデフォルトenに戻したら発生するようになった…?)
VPMリポジトリ ページを開いたら

Unhandled Runtime Error
Error: Text content does not match server-rendered HTML.

Warning: Text content did not match. Server: "Community Repositories" Client: "コミュニティリポジトリ"

See more info here: https://nextjs.org/docs/messages/react-hydration-error

Component Stack
p
eval
./node_modules/@material-tailwind/react/components/Typography/index.js (1:3639)
div
nav
eval
./node_modules/@material-tailwind/react/components/Navbar/index.js (1:3018)
HNavBar
./components/layout.tsx (27:11)
div
VStack
./components/layout.tsx (15:11)
PageBody
./app/repositories/page.tsx (47:80)
Suspense
Page
Call Stack
checkForUnmatchedText
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (32655:0)
diffHydratedProperties
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (34908:0)
hydrateInstance
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (35973:0)
prepareToHydrateHostInstance
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (7299:0)
completeWork
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (19712:0)
completeUnitOfWork
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25843:0)
performUnitOfWork
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25648:0)
workLoopConcurrent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25623:0)
renderRootConcurrent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25579:0)
performConcurrentWorkOnRoot
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (24432:0)
workLoop
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (256:0)
flushWork
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (225:0)
MessagePort.performWorkUntilDeadline
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (534:0)

Unhandled Runtime Error
Error: There was an error while hydrating this Suspense boundary. Switched to client rendering.

Call Stack
updateDehydratedSuspenseComponent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (17511:0)
updateSuspenseComponent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (17110:0)
beginWork$1
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (18433:0)
beginWork
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (26791:0)
performUnitOfWork
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25637:0)
workLoopConcurrent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25623:0)
renderRootConcurrent
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (25579:0)
performConcurrentWorkOnRoot
(app-pages-browser)\node_modules\next\dist\compiled\react-dom\cjs\react-dom.development.js (24432:0)
workLoop
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (256:0)
flushWork
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (225:0)
MessagePort.performWorkUntilDeadline
(app-pages-browser)\node_modules\next\dist\compiled\scheduler\cjs\scheduler.development.js (534:0)

@anatawa12
Copy link
Member

Error: Text content does not match server-rendered HTML.

は直し方がわかってない問題で、サーバー側と言語設定が違うことが原因なのでデフォルト言語を変えればよく、また無視して問題ないです。

@anatawa12
Copy link
Member

#648 は何が残っていますかね

@Sayamame-beans
Copy link
Collaborator Author

既知はLicensesのみのはずです
点検サボったので他にないかは不明

@Sayamame-beans
Copy link
Collaborator Author

"description for show prerelease packages"でbrタグを使用したいため、追加で許可する必要があります。(未対応)
ただ、そもそもこの部分、小見出し的な位置なのに説明文のみ載せているのが違和感があります。
何か別途小見出しを用意するか、説明の表示箇所をチェックボックスの下に移動するなどが望ましいのでしょうか…?

これは一旦スルーしちゃって大丈夫な感じです? (マージ後対処予定?)

@anatawa12
Copy link
Member

何か別途小見出しを用意するか、説明の表示箇所をチェックボックスの下に移動するなどが望ましいのでしょうか…?

ができそうならそうしたいけど、そうじゃないなら一旦スルーしてt()呼び出しを全体的に書き換えるつもり

@Sayamame-beans
Copy link
Collaborator Author

なるほど…まあ英語も共通なのでとりあえずmergeしますね
(次のパッチリリース直前に再点検したい気もしますが、どうせbetaだからってことでそのままリリースもありですね…)

@Sayamame-beans Sayamame-beans merged commit f067872 into master Mar 28, 2024
17 checks passed
@Sayamame-beans Sayamame-beans deleted the localize_ja branch March 28, 2024 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localization Improvements or additions to localization vrc-get-gui Issues for vrc-get-gui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

add addになっている migrateのキャンセルボタンに誤字 日本語翻訳
2 participants