Skip to content

設定画面のテキストを全て表示する対応#12

Merged
NMai-source merged 2 commits into
ykws:mainfrom
NMai-source:feature-#9
Nov 12, 2021
Merged

設定画面のテキストを全て表示する対応#12
NMai-source merged 2 commits into
ykws:mainfrom
NMai-source:feature-#9

Conversation

@NMai-source
Copy link
Copy Markdown
Collaborator

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

対応内容

設定画面のテキストを全て表示する対応
トークン表示画面のテキスト表示調節対応

スクリーンショット

before after
simulator_screenshot_994C0A30-ACD8-4988-8ACD-B70E401415DF simulator_screenshot_3A5C01FC-2C22-4EAB-B301-7286B17CDEEA
------- ------
simulator_screenshot_3DE4B6FF-D818-43EE-99C4-3D822AD35968 simulator_screenshot_C6522044-980B-4029-87DA-16846A314B3A

Closes #9

Copy link
Copy Markdown
Owner

@ykws ykws left a comment

Choose a reason for hiding this comment

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

must-badge

内容はばっちりです!
ただし、こちらも他の変更が含まれているため、 main から branch を作成して、この課題の変更のみで PR を作成してください。

@ykws ykws added the bug Something isn't working label Oct 30, 2021
@NMai-source
Copy link
Copy Markdown
Collaborator Author

branchを作り直しました。
ご確認よろしくお願いいたします。

@ykws
Copy link
Copy Markdown
Owner

ykws commented Nov 2, 2021

@NMai-source 赤枠のコミットだけがこの課題の変更に該当すると思うので、この PR のコミットはこれだけになっているのが望ましいです。

Screen Shot 2021-11-03 at 8 42 48

今回のケースだと、一例ですが、 cherry-pick を利用すると実現しやすそうです。

git switch -c feature-#9-new upstream:main
git cherry-pick 33ffe00ffde4a92b8c0253abc16070c8e86dd203

参考

@ykws ykws mentioned this pull request Nov 2, 2021
1 task
@NMai-source
Copy link
Copy Markdown
Collaborator Author

@ykws
すみません、対象のコミットを勘違いしていました。
cherrypickについて参考記事ありがとうございます。
参考にしてコミットを整理します。

@NMai-source
Copy link
Copy Markdown
Collaborator Author

@ykws
修正しました。
何度もすみませんが、ご確認よろしくお願いいたします。

Copy link
Copy Markdown
Owner

@ykws ykws left a comment

Choose a reason for hiding this comment

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

気になる箇所があったのでコメントしました。確認をお願いします。

どこで SettingsTableViewCell を定義しているのか迷ったので、
不要ファイルは別途削除しておきたいです。 #14

<rect key="frame" x="30" y="11" width="170" height="20"/>
<constraints>
<constraint firstAttribute="width" constant="170" id="J4w-Qf-Jlq"/>
<constraint firstAttribute="width" relation="lessThanOrEqual" constant="170" id="fvy-zA-QbD"/>
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

must-badge

直前の制約と重なる部分があるので見直しをお願いします。
以下、制約の警告が確認できるので、この警告には対応するようにお願いします。

赤矢印の指す、黄色の矢印のアイコンをクリックすると、制約の警告を確認できます。

Screen Shot 2021-11-03 at 23 44 26

クリックした後の警告表示

Screen Shot 2021-11-03 at 23 44 06

Copy link
Copy Markdown
Owner

@ykws ykws left a comment

Choose a reason for hiding this comment

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

全て表示できるようになっていることを確認できました!

スクリーンショットも見やすくて良いです!

最新の main を rebase してこの PR をマージしてみてください。

@ykws
Copy link
Copy Markdown
Owner

ykws commented Nov 11, 2021

最新の main を rebase してこの PR をマージしてみてください。

⬇️ GitHub 上で「Update branch」をクリックしても良いですが、マージコミットが作成されてしまう気がします。

Screen Shot 2021-11-11 at 21 17 28

@NMai-source
Copy link
Copy Markdown
Collaborator Author

NMai-source commented Nov 11, 2021

@ykws
mainをrebaseでマージしました。
ご確認よろしくお願い致します。

@ykws
Copy link
Copy Markdown
Owner

ykws commented Nov 12, 2021

@NMai-source ばっちりです! GitHub 上でマージできるようになっているのでそのままマージして大丈夫です!

Screen Shot 2021-11-12 at 9 45 02

@NMai-source NMai-source merged commit 3a71511 into ykws:main Nov 12, 2021
@NMai-source
Copy link
Copy Markdown
Collaborator Author

@ykws
すみません、マージまでしたつもりになっていました。
マージ完了しました。

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.

設定画面のテキストを全て表示する

2 participants