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

保存食一覧のメニューを追加 #60

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

Conversation

yamanetaisei
Copy link
Collaborator

Issueの番号 / Title

#9

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

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

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

メニューを実装するためのbranchのPRです。

@yamanetaisei yamanetaisei changed the title Create menu preserved food list taisei [WIP] 保存食一覧のメニューを追加 Feb 2, 2020
@yamanetaisei
Copy link
Collaborator Author

Screenshot_1580903011

@yamanetaisei yamanetaisei changed the title [WIP] 保存食一覧のメニューを追加 保存食一覧のメニューを追加 Feb 6, 2020
import androidx.lifecycle.ViewModelProviders
import com.example.kanazawaapp_2019.R

class SlideshowFragment : Fragment() {
Copy link
Owner

Choose a reason for hiding this comment

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

名前を変更して欲しい

Comment on lines +1 to +9
<vector xmlns:android="http://schemas.android.com/apk/res/android"
android:width="24dp"
android:height="24dp"
android:viewportWidth="24.0"
android:viewportHeight="24.0">
<path
android:fillColor="#FF000000"
android:pathData="M4,6H2v14c0,1.1 0.9,2 2,2h14v-2H4V6zm16,-4H8c-1.1,0 -2,0.9 -2,2v12c0,1.1 0.9,2 2,2h12c1.1,0 2,-0.9 2,-2V4c0,-1.1 -0.9,-2 -2,-2zm-8,12.5v-9l6,4.5 -6,4.5z" />
</vector>
Copy link
Owner

Choose a reason for hiding this comment

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

使わないなら消して欲しい

import androidx.lifecycle.MutableLiveData
import androidx.lifecycle.ViewModel

class GalleryViewModel : ViewModel() {
Copy link
Owner

Choose a reason for hiding this comment

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

いらないです

@kirikirisu
Copy link
Owner

いらない差分が多いので、いるやつだけあげて欲しい

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

Successfully merging this pull request may close these issues.

2 participants