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.

Sign up for GitHub

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

Jump to bottom

Experiment/use percentage in zooming #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Experiment/use percentage in zooming #3

wants to merge 3 commits into from

Conversation

suconbu
Copy link
Owner

@suconbu suconbu commented Jan 27, 2021

PR の目的

カテゴリ

  • 機能追加
  • 仕様変更
  • 不具合修正
  • 速度向上
  • リファクタリング
  • ドキュメント修正
  • プログラムの動作上の問題
    • 正式リリース版
    • Azure Pipelines ビルド版
    • AppVeyor ビルド版
    • GitHub Actions ビルド版
    • ローカルビルド版
  • ビルド関連
    • ビルド手順
    • Azure Pipelines
    • AppVeyor
    • GitHub Actions
    • ローカルビルド
  • ドキュメントの問題
  • GitHub 関連の問題
  • 実験 (master へのマージを目的としない)
  • その他の問題

PR の背景

PR のメリット

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

仕様・動作説明

PR の影響範囲

テスト内容

テスト1

手順

関連 issue, PR

参考資料

@suconbu
マウスホイールでのフォントサイズ変更をパーセンテージに基づいて行うようにする
717ce6c
@suconbu
レビュー指摘対応
bfad7fe 
* ZoomSettingのコンストラクタの引数見直し
* ZoomSettingのメンバ変数をconst/publicに変更(合わせてクラスから構造体に変更)
* ZoomSettingの正当性判定をコンストラクタから削除
@suconbu
起動テストのマクロからSetFontSizeを呼び出す(テストカバレッジ確保)
5a98c8b
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

suconbu pushed a commit that referenced this pull request Feb 8, 2021
@EldersJavas
Merge pull request #3 from sakura-editor/master
22fc397 
2021-01-02 09:32:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@suconbu