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アイコン」と「アウトライン解析アイコン」の間に区切り線を入れるのはどうでしょうか? #772

Closed
arigayas opened this issue Jan 19, 2019 · 7 comments
Milestone

Comments

@arigayas
Copy link

要望機能

検索関連とアウトライン解析の機能って別のような気がするので
間に区切り線を入れるのはいかがでしょうか?

スクリーンショット

image

カスタマイズしたスクリーンショット
image

@m-tmatma
Copy link
Member

#773 で実装してみました。

@arigayas
Copy link
Author

実装が速過ぎて笑いましたww

@berryzplus
Copy link
Contributor

「ダメだぁっ!」とは思ってないんですが要りますか?

ここは設定でどうとでもなる部分なので、勝手に変えたらええやん、と思う部分と、
設定ファイルの値も使いやすくなるように少しずつ見直していったほうがいいのかな、と思う部分があって「やりたい人がやったらええんちゃう?(他人任せ」な感じになってます。

PRを見たら分かるかも知れないんですが、ツールバーに区切り線を入れるときの決まりみたいなものがあって、それをペペッとコピペしただけの内容です。実装的に問題がないであろうことはほぼ間違いありません。

いまんとこ、この変更が要るのかどうかが落ちてなくてとどまってる感じです。
もうちょい考えたいです。

@m-tmatma
Copy link
Member

設定で変えられる部分でも、デフォルトで対応してると
わかりやすいと、思います。

@arigayas
Copy link
Author

設定で変えられる部分でも、デフォルトで対応してると
わかりやすいと、思います。

同意です。

区切り線があると「グループが違う」って視覚的にわかるので、
デフォルトで明示されているのが良いと思い提案しました。

@m-tmatma
Copy link
Member

m-tmatma commented Feb 3, 2019

#773 で実装してみました。

マージしました。

@arigayas
Copy link
Author

arigayas commented Feb 3, 2019

ローカルにてビルドした物が要望通りになりました。
作業していただきありがとうございました。

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

No branches or pull requests

3 participants