Skip to content

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

Open
ykws wants to merge 2 commits into
mainfrom
feature/27-input-name
Open

name を入力できるように#28
ykws wants to merge 2 commits into
mainfrom
feature/27-input-name

Conversation

@ykws
Copy link
Copy Markdown
Owner

@ykws ykws commented Nov 28, 2021 •
edited
Loading

  • 対応する 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

@ykws ykws self-assigned this Nov 28, 2021
This was linked to issues Nov 28, 2021
設定画面への遷移をナビゲーションに変更する #8
Open
不要ファイルが追加されている #14
Open
name を入力できるようにする #27
Open
@ykws ykws requested a review from NMai-source November 29, 2021 09:29
NMai-source
NMai-source reviewed Nov 30, 2021
View reviewed changes
Comment thread OneTimePasswordExample/SettingController.m
Comment thread OneTimePasswordExample/Base.lproj/Main.storyboard
<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
Copy Markdown
Collaborator

@NMai-source NMai-source Nov 30, 2021

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Owner Author

@ykws ykws Dec 2, 2021

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
Copy Markdown
Collaborator

@NMai-source NMai-source Dec 2, 2021

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
Copy Markdown
Owner Author

@ykws ykws Dec 4, 2021

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
Copy Markdown
Collaborator

@NMai-source NMai-source Dec 6, 2021

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
@ykws
name を入力できるように
968115e 
- Static Cells を利用する
  - 不特定な要素を扱わないので、利用用途が限定されているため Static で良い
  - TextField を扱いやすくなる
- NavigationController を追加
  - ライフサイクルの管理を楽にするため
  - viewWillAppear で更新処理ができるようになる

fixes #8 #27
@ykws
不要になったファイルを削除
2d1ae9a 
fixes #14
@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

Reviewers

@NMai-source NMai-source NMai-source left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@ykws ykws

Labels

question Further information is requested

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

name を入力できるようにする 不要ファイルが追加されている 設定画面への遷移をナビゲーションに変更する

2 participants

@ykws @NMai-source