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

Fix HUD position after screen rotation #32

Closed
wants to merge 4 commits into from
Closed

Fix HUD position after screen rotation #32

wants to merge 4 commits into from

Conversation

nappannda
Copy link

English

Problem

  • Unnatural HUD Position after screen rotation

Solution

  • Set the frame of hudViewController to the frame of UIWindow when resetLayout is called
  • Since the centerPosition was not changed after rotating the screen
    To change centerPosition when resetLayout is called

ScreenShot

portraitposition

landscapeposition

日本語

問題

  • 画面回転後のHUD表示位置が不自然だった（スクリーンショットを参照してください）

解決方法

  • hudViewControllerのframeが画面回転後に正しく設定されていなかったのでresetLayout時に設定するように変更
  • 画面回転してもcenterPositionが変更されていなかったので反映されるように変更

@nappannda
Fix HUD position after screen rotation
bc57687 
# ==== Emojis ====
# 🐛  🐛 バグ修正
# 👍  👍 機能改善
# ✨  ✨ 部分的な機能追加
# 🎉  🎉 盛大に祝うべき大きな機能追加
# ♻️  ♻️ リファクタリング
# 🚿  🚿 不要な機能・使われなくなった機能の削除
# 💚  💚 テストやCIの修正・改善
# 👕  👕 Lintエラーの修正やコードスタイルの修正
# 🚀  🚀 パフォーマンス改善
# 🆙  🆙 依存パッケージなどのアップデート
# 🔒  🔒 新機能の公開範囲の制限
# 👮  👮 セキュリティ関連の改善

# ==== Format ====
# :emoji: Subject
#
# Commit body...

# ==== The Seven Rules ====
# 1. Separate subject from body with a blank line
# 2. Limit the subject line to 50 characters
# 3. Capitalize the subject line
# 4. Do not end the subject line with a period
# 5. Use the imperative mood in the subject line
# 6. Wrap the body at 72 characters
# 7. Use the body to explain what and why vs. how
#
# How to Write a Git Commit Message http://chris.beams.io/posts/git-commit/

# for http://memo.goodpatch.co/2016/07/beautiful-commits-with-emojis/
@nappannda
Remove appearance viewCenterPosition test
a2f5d6a
@nappannda
Fix value center position test
f52ee1b
@nappannda
Fix test window size
72aef10 
# ==== Emojis ====
# 🐛  🐛 バグ修正
# 👍  👍 機能改善
# ✨  ✨ 部分的な機能追加
# 🎉  🎉 盛大に祝うべき大きな機能追加
# ♻️  ♻️ リファクタリング
# 🚿  🚿 不要な機能・使われなくなった機能の削除
# 💚  💚 テストやCIの修正・改善
# 👕  👕 Lintエラーの修正やコードスタイルの修正
# 🚀  🚀 パフォーマンス改善
# 🆙  🆙 依存パッケージなどのアップデート
# 🔒  🔒 新機能の公開範囲の制限
# 👮  👮 セキュリティ関連の改善

# ==== Format ====
# :emoji: Subject
#
# Commit body...

# ==== The Seven Rules ====
# 1. Separate subject from body with a blank line
# 2. Limit the subject line to 50 characters
# 3. Capitalize the subject line
# 4. Do not end the subject line with a period
# 5. Use the imperative mood in the subject line
# 6. Wrap the body at 72 characters
# 7. Use the body to explain what and why vs. how
#
# How to Write a Git Commit Message http://chris.beams.io/posts/git-commit/

# for http://memo.goodpatch.co/2016/07/beautiful-commits-with-emojis/
@krimpedance
Copy link
Owner

@nappannda
バグ報告，修正ありがとうございます．
バグは直っているようですが，appearance()でHUDの初期位置を設定できなくなっています．

@krimpedance
Copy link
Owner

また，現在レイアウト周りの調整を簡潔にするため，AutoLayoutによる配置を検討しています．
よって，今回のPRは見送らせていただきます．

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.

None yet
2 participants
@nappannda @krimpedance