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

name を入力できるように #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

name を入力できるように #28

wants to merge 2 commits into from

Conversation

ykws
Copy link
Owner

@ykws ykws commented Nov 28, 2021

  • 対応する Issue が Linked Issues に設定されていることを確認してください

対応内容

スクリーンショット

  • 画面の変更があるのでスクリーンショットを用意する
before after
storyboard (省略) Screen Shot 2021-11-28 at 17 57 29
settings (省略) Simulator Screen Shot - iPhone 8 - 2021-11-28 at 17 54 19

OneTimePasswordExample/SettingController.m Show resolved Hide resolved
<rect key="frame" x="0.0" y="0.0" width="414" height="43.5"/>
<autoresizingMask key="autoresizingMask"/>
<subviews>
<label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Period" textAlignment="natural" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="d1t-YT-Wmb">
Copy link
Collaborator

Choose a reason for hiding this comment

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

@ykws
シミュレータ上でperiodのドラムロールが操作できませんでした。

Copy link
Owner Author

Choose a reason for hiding this comment

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

@NMai-source うーん、手元だと動きますね。
Simulator で ⌘+Rで録画できるので、再現を録画した動画ファイルを添付してもらっても良いでしょうか?

Simulator.Screen.Recording.-.iPhone.8.-.2021-12-02.at.22.47.26.mp4

Copy link
Collaborator

Choose a reason for hiding this comment

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

@ykws
録画しました。
分かりづらいですが、上から順にクリックしてperiodのドラムロールを表示しようとしています。
ご確認お願い致します。

Simulator.Screen.Recording.-.iPod.touch.7th.generation.-.2021-12-02.at.23.26.11.mp4

Copy link
Owner Author

Choose a reason for hiding this comment

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

@NMai-source なるほど、 Algorithm と Digits はタップするセルの色が変わりますが、 Period は変わっていないので、タップしても反応しない、という認識であってますか?

Copy link
Collaborator

Choose a reason for hiding this comment

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

@ykws
はい、タップしても反応しません。

ykws added 2 commits December 2, 2021 22:38
- Static Cells を利用する
  - 不特定な要素を扱わないので、利用用途が限定されているため Static で良い
  - TextField を扱いやすくなる
- NavigationController を追加
  - ライフサイクルの管理を楽にするため
  - viewWillAppear で更新処理ができるようになる

fixes #8 #27
@ykws ykws force-pushed the feature/27-input-name branch from 73cb4bf to 2d1ae9a Compare December 2, 2021 13:38
@ykws ykws added the question Further information is requested label Dec 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
2 participants