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

CodeFactorで検出されたCSSの問題を修正する #1443

Merged
merged 6 commits into from
Oct 28, 2020

Conversation

berryzplus
Copy link
Contributor

PR の目的

CodeFactorで検出されたCSSの問題を修正します。

カテゴリ

  • リファクタリング

PR の背景

この警告に対応するための修正です。
https://www.codefactor.io/repository/github/sakura-editor/sakura/issues?lang=25
https://www.codefactor.io/repository/github/sakura-editor/sakura/issues?lang=7

PR のメリット

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

仕様・動作説明

テスト内容

PR の影響範囲

関連 issue, PR

参考資料

@AppVeyorBot
Copy link

@ghost
Copy link

ghost commented Oct 27, 2020

CSSの他にPowerShellスクリプトの変更が含まれていませんか?

@berryzplus
Copy link
Contributor Author

CSSの他にPowerShellスクリプトの変更が含まれていませんか?

検証不能で出さないつもりのコミットが混じってました。revertします。

@berryzplus
Copy link
Contributor Author

CSSセレクタの定義が分かれているのを苦tっ付ける

コミットメッセージにtypo発見したので修正します。

@berryzplus berryzplus marked this pull request as draft October 28, 2020 03:41
@berryzplus
Copy link
Contributor Author

CSSの他にPowerShellスクリプトの変更が含まれていませんか?

検証不能で出さないつもりのコミットが混じってました。revertします。

別のコミットと間違って入れていたようです。
ややこしくなるので新規コミットとして修正を入れ直しました。
修正されるissueは全部で34件です。

@berryzplus berryzplus marked this pull request as ready for review October 28, 2020 03:47
@AppVeyorBot
Copy link

@AppVeyorBot
Copy link

@sanomari
Copy link
Contributor

sanomari commented Oct 28, 2020

修正されるissueは全部で34件です。

32FIXEDと出ているみたいです。

@AppVeyorBot
Copy link

@ghost
Copy link

ghost commented Oct 28, 2020

修正されるissueは全部で34件です。

32FIXEDと出ているみたいです。

CSSの34件中31件と、HTMLの1件で合計32件ですね。
HTMLのissueも内容はCSS関係なので同時対応してよいと思います。

Copy link
Contributor

@sanomari sanomari left a comment

Choose a reason for hiding this comment

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

修正した数と修正された数が合わないのが気になりますが、内容は問題なさそうに見えます。

@berryzplus
Copy link
Contributor Author

修正した数と修正された数が合わないのが気になります

なんでだろう・・・。
自分のブランチで試した限りでは HTML/CSS のすべての警告が解消されていました。

あれ?ということは 34 + 1 で 35 件解消されるということに・・・。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。

@berryzplus berryzplus merged commit e9d5b06 into sakura-editor:master Oct 28, 2020
@berryzplus berryzplus deleted the feature/fix_css_issues branch October 28, 2020 10:05
@beru beru added the refactoring リファクタリング 【ChangeLog除外】 label Mar 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring リファクタリング 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants