Skip to content

Update syntax.{txt,jax} #749

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

Merged
merged 7 commits into from
Jan 5, 2020
Merged

Update syntax.{txt,jax} #749

merged 7 commits into from
Jan 5, 2020

Conversation

tennashi
Copy link
Contributor

@tennashi tennashi commented Dec 15, 2019

  • TOhtmlProgress ハイライトグループについての説明
  • コピー抑制エリアの作り方についての変更
    • g:html_use_input_for_pc
  • C 構文ファイルの修正
  • ReST 構文ファイルの修正
  • (原文のみ) 行末の余計な空白文字の削除
  • (翻訳のみ) typo の修正 // 4662 行目
  • (翻訳のみ) ブラウザからブラウザーへ修正
  • Rego の構文ファイル追加

@tennashi
Copy link
Contributor Author

あ、寝惚けてんのか、逆だった...

@tennashi
Copy link
Contributor Author

確認をおねがいしますー

doc/syntax.jax Outdated
*hl-TOhtmlProgress* *TOhtml-progress-color*
プログレスバーは HTML 変換が進むにつれ、ステータスラインに沿って色付きの四角で
表示される。デフォルトでは背景色は現在の "DiffDelete" ハイライトグループの設定
が適用される。もし "DiffDelete" と "StatusLine" の背景色が同じ場合、TOhtml は
Copy link
Member

Choose a reason for hiding this comment

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

もしは不要だと思いました。(Ifは、~の場合に訳されていると捉えています)

doc/syntax.jax Outdated
プログレスバーは HTML 変換が進むにつれ、ステータスラインに沿って色付きの四角で
表示される。デフォルトでは背景色は現在の "DiffDelete" ハイライトグループの設定
が適用される。もし "DiffDelete" と "StatusLine" の背景色が同じ場合、TOhtml は
自動で色を調整する。もし自動で色を選択したくない場合は、自身でプログレスバーの
Copy link
Member

Choose a reason for hiding this comment

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

同上。

doc/syntax.jax Outdated
注意が必要である。<input> 要素がペーストされないようにするのは困難だ。
0 なら、もし |g:html_prevent_copy| が空でなく |g:html_use_input_for_pc| も
"none" でない場合、コピー抑制エリアのための <input> エレメントに不正な属性が意
図的に挿入される。これによって、いくつかのアプリケーションで <input> 要素が
Copy link
Member

Choose a reason for hiding this comment

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

改行位置が違います。
参照: #743 (comment)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

こちらは autofmt を入れて gq しても変化がありませんでした

Copy link
Contributor

Choose a reason for hiding this comment

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

78越えになっちゃってる所があるのは確かっぽいので、手動で直す必要がありそうですね

Copy link
Contributor Author

Choose a reason for hiding this comment

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

すいません、formatoptions の設定を忘れてました...
#54 (comment)
このコメントだと set formatoptions+=mM 推奨とのことでしたが

|g: html_use_input_for_pc| も "none" でない場合

の部分で の前後の空白の解釈ゆれで format 結果が変わってしまったので set formatoptions+=mB でやりました

doc/syntax.jax Outdated
@@ -929,8 +960,9 @@ Cには幾つかの補助的なハイライト方法がある。それらを有
をエラーとして表示しない。
デフォルトでエラーとして表示される。欠落した
")" を見つけられないため。
*c_curly_error* 対応する { がない } をハイライトする。これを有効にする
と、ファイルの先頭から同期が始まるため、遅くなる
*c_curly_error* 全てのペアを検証することにより対応する { がない } をハイ
Copy link
Member

Choose a reason for hiding this comment

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

改行位置が違います。

@h-east
Copy link
Member

h-east commented Dec 20, 2019

PR後に .txt 更新されましたね。追加7行(うち、翻訳対象3行)
このPRで対応してもいいですし、しなくてもいいですし、お任せします。

@tennashi
Copy link
Contributor Author

追加分も対応しますー

@tennashi
Copy link
Contributor Author

指摘いただいた部分修正しました
再度確認をおねがいしますー

doc/syntax.jax Outdated
REGO *rego.vim* *ft-rego-syntax*

Rego は Styra が開発したクエリ言語である。主に kubernetes のポリシー言語として
使用される。次の拡張子のファイルが rego ファイルとして認識される: .rego
Copy link
Member

Choose a reason for hiding this comment

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

, but can be applied to almost anything. が訳されていないです。
こんな感じでしょうか?
「主に kubernetes のポリシー言語として使用されるが、ほぼすべての言語に適用できる。」
(の言語は補足的に足してみましたがあってます?)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

https://www.openpolicyagent.org/https://www.openpolicyagent.org/docs/latest/integration/ を読んでいると、どちらかといえば "ほぼ全てのアプリケーションで" のほうが近いようです

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ライブラリが提供されているのも Go だけで、その他はサイドカーとしてエージェントを用意して、REST API を叩いて使うようです

@tennashi
Copy link
Contributor Author

というわけで再度修正しました!

Copy link
Member

@h-east h-east left a comment

Choose a reason for hiding this comment

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

最後の修正後に本家の更新が入っていますが、あんまり長引くと疲れると思うので、一旦mergeしてもいいかなと思っています。

@k-takata k-takata merged commit bce3925 into vim-jp:master Jan 5, 2020
@k-takata
Copy link
Member

k-takata commented Jan 5, 2020

Thanks.

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.

4 participants