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

GrepとGrep置換ダイアログの微調整 #1242

Merged
merged 2 commits into from
Apr 26, 2020

Conversation

7-rate
Copy link
Contributor

@7-rate 7-rate commented Apr 24, 2020

PR の目的

GrepダイアログとGrep置換ダイアログの「結果出力」が除外フォルダの入力ボックスに近くて見づらいという問題があります。
それを解消します。

カテゴリ

  • その他
    UIの微調整のみです。

PR の背景

目的に書いた通り。

PR のメリット

GrepダイアログとGrep置換ダイアログが見やすくなります

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

特になし

PR の影響範囲

UIの微調整のみであるため無し

関連チケット

特になし

参考資料

変更前後の画像を貼ります。

Grepダイアログ-日本語

image

Grep置換ダイアログ-日本語

image

Grepダイアログ-英語

image

Grep置換ダイアログ-英語

image

@AppVeyorBot
Copy link

Build sakura 1.0.2711 completed (commit 7c3f874409 by @7-rate)

beru
beru previously approved these changes Apr 24, 2020
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

問題無いと思います。

上フォルダボタンも除外フォルダのエディットに近いですね。

英語版リソースの更新(が必要な場合)は別PRで良いと思います。

@m-tmatma
Copy link
Member

英語版リソースの更新(が必要な場合)は別PRで良いと思います。

忘れそうなので一緒に直したいですね。

@7-rate
Copy link
Contributor Author

7-rate commented Apr 25, 2020

英語版も上フォルダボタン等も一緒に直してしまうのは同意ですが、英語版の動作確認方法が分からないです。どこかに書いてありますか?

@7-rate
Copy link
Contributor Author

7-rate commented Apr 25, 2020

自己解決しました。

英語版:除外ファイル、結果出力のUI位置調整
@7-rate
Copy link
Contributor Author

7-rate commented Apr 25, 2020

指摘内容以外に除外ファイルの位置も微調整しました。
PRの内容のキャプチャを更新したのでそこで確認してもらえれば。

@7-rate 7-rate requested a review from beru April 25, 2020 15:04
@AppVeyorBot
Copy link

Build sakura 1.0.2719 completed (commit 858f64f8ec by @7-rate)

Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

問題無いと思います。

ローカルで動作確認はしていませんが画面キャプチャを見て判断しました。

@m-tmatma m-tmatma merged commit 8339035 into sakura-editor:master Apr 26, 2020
Copy link
Contributor

@beru beru left a comment

Choose a reason for hiding this comment

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

PRありがとうございます。merge します。

と思ったら既に merge されてました。🎉

@7-rate 7-rate deleted the feature/GREPUI branch April 26, 2020 12:24
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 16, 2020
GrepとGrep置換ダイアログの微調整
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