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

footerの実装 #43

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

footerの実装 #43

wants to merge 13 commits into from

Conversation

sato-na
Copy link
Collaborator

@sato-na sato-na commented Jan 11, 2020

Issueの番号 / Title

#8

どのように解決したか / How

tabLayoutでfooterを作成した

なぜそのような解決策を取ったのか / Why

tabLayoutでfooterを作成した

その他 / スクリーンショットなど

ui作成のプルリクから変更してません

@sato-na sato-na self-assigned this Jan 12, 2020
shoppingListTabLayout.addOnTabSelectedListener(object: TabLayout.OnTabSelectedListener {
override fun onTabSelected(tab: TabLayout.Tab) {
var position = shoppingListTabLayout.selectedTabPosition
if (position == 0) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ここif文よりもwhenの方がいいかも

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ちなみにif文よりもwhen文のほうがいい理由って何ですか?


override fun onTabReselected(tab: TabLayout.Tab) {
var position = shoppingListTabLayout.selectedTabPosition
if (position == 0) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

ここも

@sato-na
Copy link
Collaborator Author

sato-na commented Jan 21, 2020

↑「if文からwhen文に変更」のミスです

@sato-na sato-na changed the base branch from master to createScreen-shoppingList-shinbo January 25, 2020 03:57
@maresuke3
Copy link
Collaborator

マージしたいのでコンフリクト直しておいてください。

@sato-na sato-na changed the base branch from createScreen-shoppingList-shinbo to master February 3, 2020 05:25
@sato-na
Copy link
Collaborator Author

sato-na commented Feb 3, 2020

baseBranchを変更したらコンフリクトが治りました。
おそらくmasterとはコンフリクトが起きてないと思います。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants