list selection mode and deleting published items #26

Merged
akinwale merged 2 commits from delete-published-files into master 2019-08-16 19:26:14 +02:00
akinwale commented 2019-08-16 10:06:15 +02:00 (Migrated from github.com)
No description provided.
neb-b (Migrated from github.com) reviewed 2019-08-16 10:06:15 +02:00
kauffj (Migrated from github.com) reviewed 2019-08-16 16:20:13 +02:00
kauffj (Migrated from github.com) left a comment

Code only review. Would like to review UI/UX at our 2pm meeting today (or potentially on standup)

Code only review. Would like to review UI/UX at our 2pm meeting today (or potentially on standup)
kauffj (Migrated from github.com) commented 2019-08-16 16:18:05 +02:00

i18n for the text above and below

i18n for the text above and below
kauffj (Migrated from github.com) commented 2019-08-16 16:18:41 +02:00

this could be written as uris.forEach

this could be written as `uris.forEach`
@ -39,0 +85,4 @@
abandonClaim(txid, nout);
});
this.onExitSelectionMode();
},
kauffj (Migrated from github.com) commented 2019-08-16 16:17:48 +02:00

It looks like handleItemLongPress and handleSelectItem don't actually need to exist, but if you like them for clarity that's okay.

It looks like `handleItemLongPress` and `handleSelectItem` don't actually need to exist, but if you like them for clarity that's okay.
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-react-native#26
No description provided.