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

ヘルプ中のスタイルシートでセミコロンの指定の誤りによる Code Factor の警告を修正する #1111

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Dec 7, 2019

PR の目的

ヘルプ中のスタイルシートでセミコロンの指定の誤りによる Code Factor の警告を修正する

カテゴリ

  • ドキュメント修正

PR の背景

sakura-editor/sakura-editor.github.io#22 の誤りを修正するために
sakura-editor/sakura-editor.github.io#52 を投げたが、
Code Factor の警告が検出された。

PR のメリット

Code Factor の警告が修正される。

PR のデメリット (トレードオフとかあれば)

なし

PR の影響範囲

関連チケット

sakura-editor/sakura-editor.github.io#52

参考資料

https://www.codefactor.io/repository/github/sakura-editor/sakura-editor.github.io/pull/51

@m-tmatma
Copy link
Member Author

m-tmatma commented Dec 7, 2019

https://www.codefactor.io/repository/github/sakura-editor/sakura/pull/fixed/1111
を見ると警告が修正されているのを確認できる。

@AppVeyorBot
Copy link

Build sakura 1.0.2415 completed (commit 2803812811 by @m-tmatma)

@m-tmatma m-tmatma merged commit ee281f7 into sakura-editor:master Dec 7, 2019
@m-tmatma m-tmatma deleted the feature/fix-Unexpected-extra-semicolon branch December 7, 2019 06:37
@berryzplus
Copy link
Contributor

パッと見で問題なさげ(文句もnothing)だったので approve にしてしまいました。

「背景」のところを後で見て、Webサイト側は修正済み、ということを知りました。

「.(ドット)」が見辛いので 「MS ゴシック(Windowsで最も一般的な等幅ゴシックフォント)」にするとコメントされてるのが少し気になりました。ドットが見やすいフォントは他にもあるよね?的な意味で・・・。

@KENCHjp KENCHjp added the document ドキュメント label Jan 7, 2020
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
…ected-extra-semicolon

ヘルプ中のスタイルシートでセミコロンの指定の誤りによる Code Factor の警告を修正する
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
document ドキュメント
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants