cwtch.im
/
cwtch-ui
10
11
Fork 17
Code Issues 88 Pull Requests 2 Projects 2 Releases 28 Wiki Activity

Implement View Replies #507

Merged
dan merged 4 commits from viewreplies into trunk 2022-07-07 22:13:11 +00:00
Conversation 3 Commits 4 Files Changed 18 +299 -19
sarah commented 2022-07-07 19:36:10 +00:00
Owner
Copy Link
No description provided.
dan was assigned by sarah 2022-07-07 19:36:11 +00:00
sarah force-pushed viewreplies from a96be8cb3d to 337f6dc5d9 2022-07-07 19:58:55 +00:00 Compare
buildbot commented 2022-07-07 20:25:15 +00:00
Member
Copy Link

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/cwtch-ui/879

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch-ui/879
sarah added 1 commit 2022-07-07 20:40:36 +00:00
continuous-integration/drone/pr Build is passing Details
ad52f2e0c8 Fixup Padding and add message about no replies
buildbot commented 2022-07-07 20:51:34 +00:00
Member
Copy Link

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/cwtch-ui/880

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch-ui/880
sarah added 1 commit 2022-07-07 20:52:18 +00:00
continuous-integration/drone/pr Build is passing Details
7540aed701 Handle viewing replies to images correctly
dan reviewed 2022-07-07 20:57:06 +00:00
lib/models/message.dart
@ -204,0 +208,4 @@
MessageInfo original = cache.cache[messageIdentifier]!;
String hash = original.metadata.contenthash;
cache.cache.forEach((key, messageInfo) {
dan commented 2022-07-07 20:54:26 +00:00
Owner
Copy Link

we're still iterating over ALL messages

we could possibly pass in index and then

for i = index; i< cache.cacheByIndex.len; i++ {
	messageInfo = cache.getByIndex(i)
we're still iterating over ALL messages we could possibly pass in index and then ``` for i = index; i< cache.cacheByIndex.len; i++ { messageInfo = cache.getByIndex(i) ```
lib/widgets/messagerow.dart
@ -93,0 +100,4 @@
var cache = Provider.of<ContactInfoState>(context).messageCache;
Widget wdgSeeReplies = Platform.isAndroid
? SizedBox.shrink()
dan commented 2022-07-07 20:48:49 +00:00
Owner
Copy Link

~does this show as nothing on android?~
aaah on android we long press the whole thing?

~does this show as nothing on android?~ aaah on android we long press the whole thing?
buildbot commented 2022-07-07 21:06:35 +00:00
Member
Copy Link

Drone Build Status: success

https://build.openprivacy.ca/cwtch.im/cwtch-ui/881

Drone Build Status: success https://build.openprivacy.ca/cwtch.im/cwtch-ui/881
dan merged commit 3a752b7397 into trunk 2022-07-07 22:13:11 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
android

   
arch

   
backlog

   
blocked-on-external

This issue requires effort from an external organization to move forward

  
bug

Something is not working

  
bugbash

   
component/bindings

   
component/bine

   
component/connectivity

   
component/cwtch

   
component/tapir

   
component/ui

   
cwtch-1.14

   
cwtch-1.15

   
cwtch-beta-1.1

   
cwtch-beta-1.10

Changes Planned for Cwtch Beta 1.10

  
cwtch-beta-1.11

   
cwtch-beta-1.12

   
cwtch-beta-1.13

   
cwtch-beta-1.2

   
cwtch-beta-1.3

   
cwtch-beta-1.4

   
cwtch-beta-1.5

   
cwtch-beta-1.5.x

point release vug fixes for 1.5

  
cwtch-beta-1.6

   
cwtch-beta-1.7

   
cwtch-beta-1.8

   
cwtch-beta-1.9

   
design

   
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
flutter

   
funding-needed

   
help wanted

Need some help

  
hybrid-groups

   
in-nightly

   
in-progress

   
invalid

Something is wrong

  
ios

   
linux

   
mac

   
need-replication-or-investigation

   
ops

   
packaging

   
post-stable

These issues won't be considered until after Cwtch Stable ships.

  
question

More information is needed

  
questionable

there is an open question as to whether this is an issue at all / if we can even fix this

  
requires-more-effort-than-we-can-spare

The amount of work involved to solve this issue would occupy our entire development team for a significant period of time and/or provide little benefit to the rest of the project

  
rust

   
scheduled

   
stable-blocker

   
tails

   
testing-needed

   
tests

   
tor

   
waiting-on-fix-confirmation

   
waiting-on-new-flutter-feature

The cause of this issue is a bug in the underlying flutter framework.

  
whonix

   
windows

   
wontfix

This won't be fixed

No Label
android
arch
backlog
blocked-on-external
bug
bugbash
component/bindings
component/bine
component/connectivity
component/cwtch
component/tapir
component/ui
cwtch-1.14
cwtch-1.15
cwtch-beta-1.1
cwtch-beta-1.10
cwtch-beta-1.11
cwtch-beta-1.12
cwtch-beta-1.13
cwtch-beta-1.2
cwtch-beta-1.3
cwtch-beta-1.4
cwtch-beta-1.5
cwtch-beta-1.5.x
cwtch-beta-1.6
cwtch-beta-1.7
cwtch-beta-1.8
cwtch-beta-1.9
design
duplicate
enhancement
flutter
funding-needed
help wanted
hybrid-groups
in-nightly
in-progress
invalid
ios
linux
mac
need-replication-or-investigation
ops
packaging
post-stable
question
questionable
requires-more-effort-than-we-can-spare
rust
scheduled
stable-blocker
tails
testing-needed
tests
tor
waiting-on-fix-confirmation
waiting-on-new-flutter-feature
whonix
windows
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
dan
3 Participants
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: cwtch.im/cwtch-ui#507
No description provided.
Delete Branch "viewreplies"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?