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

検索ボックスの表示位置を調整する #1345

Merged
merged 2 commits into from
Jul 12, 2020

Conversation

berryzplus
Copy link
Contributor

PR の目的

ツールバーの項目「検索(ボックス)」の表示位置を調整して、#1343 ツールバーに設定された「検索(ボックス)」の表示場所がおかしい で報告された不具合を修正します。

カテゴリ

  • 不具合修正

PR の背景

#1343 を参照。

PR のメリット

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

仕様・動作説明

ツールバーの項目「検索(ボックス)」に表示するコンボボックスの表示位置を調整します。

項目 仕様説明
left(左端) 指定された左端 ⇒ 指定された左端 + ボーダー幅
top(上端) コンボボックスを指定された矩形高さの上下中央に配置するために必要なオフセット高さ ⇒ コンボボックスを指定された矩形の上下中央に配置するための上端
width(幅) 変更しない ⇒ 元の幅 - ボーダー幅 × 2
height(高さ) 変更しない ⇒ 元の高さ

テスト内容

改修効果確認のテストを実施しました。

テスト1

手順

  1. このPRをチェックアウトしてビルドしてsakura.exeを起動します。
  2. 設定 - 共通設定 のツールバータブで以下の設定を行います。
    ツールバー折り返しの後に「検索(ボックス)」を選択することがポイントです。
    image
  3. 以下のように、「検索(ボックス)」が2行目に表示されることが確認できたらOKです。image

PR の影響範囲

関連 issue, PR

fixes #1343

参考資料

@AppVeyorBot
Copy link

@AppVeyorBot
Copy link

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.

動作確認して不具合が解消された事を確認しました。問題無いと思います。

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。マージしちゃいます。
何か問題が見つかったら別PRで対処していきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ツールバーに設定された「検索(ボックス)」の表示場所がおかしい
3 participants