search

blueprint-freespeech / ricochet-refresh

Anonymous peer-to-peer instant messaging
https://www.ricochetrefresh.net
Other
162 stars 27 forks source link

Bad UX after removing a contact #72

Closed morganava closed 3 years ago

morganava commented 3 years ago

Whenremoving a contact from the contact list, the pop-up informs the user that the contact will be notified. We could not confirm the notification part on the side of the removed contact. This likely also leads to the following minor issue: Re-adding a previously deleted contact leads to several bugs in the UI where a) unread message notifications are shown, but not the messages themselves and b) no messages can be sent anymore because the UI is locked. Below is a flow chart with the actions required to trigger the issue and see the resulting command line errors.

image

qt.qpa.xcb: QXcbConnection: XCB error: 3 (BadWindow), sequence: 2041, resource id: 8501173, major code: 40 (TranslateCoords), minor code: 0
qrc:/ui/ChatPage.qml:22: TypeError: Cannot call method 'resetUnreadCount' of null
qrc:/ui/ChatPage.qml:154: TypeError: Cannot call method 'sendMessage' of null
QTextCursor::setPosition: Position '25' out of range
morganava commented 3 years ago

Fixed in e59de1a48c4b46ca86d69c8ae0d8ce12a7cb4536