タスク選択画面のTableView実装

yasasii-team / Stepippo-iOS

初めての一歩を応援するアプリのiOS版

21 stars 5 forks source link

Closed Koki-Nakamura closed 5 years ago

Koki-Nakamura commented 5 years ago

Issue #145

Addボタンをタップすると一時的に保存されるTableViewを実装

AddボタンをタップするとTableViewにTextField内の文字が追加される

Koki-Nakamura commented 5 years ago

結構、抜け落ちてる所ありますが、ご容赦を...

Koki-Nakamura commented 5 years ago

修正＆追記しましたので再レビューお願いします。

Nextinnovation-Koki commented 5 years ago

Sketchのイメージだとセルの中に赤丸のマイナスボタン（一度セットしたものを削除するためのボタン？）のようなものがあったので、てっきりその実装のためにisCircleを追加されたのかと思っていました。この丸のボタンは無しにしたのでしょうか？？

なしです。削除する必要がある場合は別PRで対応しようと思います。

Sab-swiftlin commented 5 years ago

なしです。削除する必要がある場合は別PRで対応しようと思います。

承知しました！そうなるとカスタムセル作らなくても良いかもしれませんね。（将来追加対応濃厚であれば作っておいても問題ないと思います）現状AddTaskCell.swift内でオーバーライドしているメソッドも中で特に処理していないので、記載不要な気がしました。

Koki-Nakamura commented 5 years ago

修正しました。追加でセルをスワイプした時に削除出来るようにしておきました。再レビューお願いします。

Sab-swiftlin commented 5 years ago

@Koki-Nakamura ご対応ありがとうございます！修正いただいた部分良いと思います！スワイプ削除もいいですね！！

一点だけすいません、cellのselectionはnoneにした方が良いかなと思いました。よろしくお願いします。

Koki-Nakamura commented 5 years ago

こちらこそ何度もすいません... Noneに修正しておきました！

Sab-swiftlin commented 5 years ago

マージ前にコンフリクトの解消だけお願い致します:bow:

Koki-Nakamura commented 5 years ago

コンフリクト修正しました！！！

Sab-swiftlin commented 5 years ago

@Koki-Nakamura すいません、見た感じまだ完全には解消されていないように思われます。。。

Sab-swiftlin commented 5 years ago

あ、いけそうですね。失礼致しました。マージしちゃってくださいー。