seaglass icon indicating copy to clipboard operation
seaglass copied to clipboard

Accept and decline buttons are grayed out after being kicked from a room and reinvited

Open aaronraimist opened this issue 7 years ago • 16 comments

Specifications

  • Seaglass Version: 0.0.224-c536529 (actually 0.0.354?)
  • macOS Version: 10.13.6

Expected Behavior

Accept and decline buttons on room invite should always work

Actual Behavior

Accept and decline buttons on room invite are grayed out

Steps to Reproduce the Problem

  1. Create a new room from account 1
  2. Invite account 2
  3. Accept the room invite on account 2
  4. Kick account 2 from the room
  5. Invite account 2 to the room again

Workaround: quit Seaglass and launch again (actually it crashed but I'm assuming quitting and relaunching would work) and the buttons will be enabled again

aaronraimist avatar Sep 08 '18 20:09 aaronraimist

This should have been fixed in 1324d4fdffbdf206b3a083615b05ffbf57574a15 - can you please retest with the latest build?

neilalexander avatar Sep 12 '18 21:09 neilalexander

Kicks don't work properly in 1324d4f, you continue see the room after the kick. It just says "Aaron Raimist has left the room".

Then if you attempt to send a message you get this crash: screen shot 2018-09-12 at 9 03 46 pm

aaronraimist avatar Sep 13 '18 02:09 aaronraimist

Hopefully that's fixed in d2701e78cbdf31100d902a524d01b61b422b6fc9 now.

If the invite buttons look good then we should be OK to close the issue.

neilalexander avatar Sep 13 '18 10:09 neilalexander

Almost all good now, invite buttons work properly but the emoji button doesn't get reenabled when you rejoin the room after being kicked and then reinvented.

Also I'd be nice to see the placeholder text change in the input field change or something. First I thought it had crashed when I couldn't select the input field but then I realized you just disabled it. Is it not possible for you to detect that it was a kick vs a leave?

aaronraimist avatar Sep 13 '18 13:09 aaronraimist

Have made some further changes to this behaviour in 05c23828a6264a08e35a123934caabd299f27b48 - the button enabling/disabling for all types of events is now handled by a single function which seems to be a bit more robust.

The label seems to be a bit more difficult - not sure why that's not rendering the right text. Can you retest the rest please?

neilalexander avatar Sep 13 '18 15:09 neilalexander

Looks good but invites don't show up until I relaunch Seaglass

aaronraimist avatar Sep 13 '18 16:09 aaronraimist

Actually I guess the invite does show up, just not at the top anymore and the input field and buttons didn't come back.

aaronraimist avatar Sep 13 '18 16:09 aaronraimist

Yeah I'm having a hard time with the controls displaying reliably - I'm not sure if I'm just going cross-eyed from having been looking at the code for so long but from what I can tell this should work.

neilalexander avatar Sep 13 '18 16:09 neilalexander

49796ca might help.

neilalexander avatar Sep 13 '18 16:09 neilalexander

Unfortunately not.

It might be worth me clearing the cache b/c I sometimes see the same room with an old name. There isn't an easy button hooked up to clear cache yet right?

Anyway /me -> work so I'll test any fixes you come up with later today.

aaronraimist avatar Sep 13 '18 16:09 aaronraimist

Any luck with this on newer commits?

neilalexander avatar Sep 17 '18 10:09 neilalexander

So my raim.ist server is down at the moment so I found another unrelated bug which is preventing me from testing this. See #75.

aaronraimist avatar Sep 17 '18 16:09 aaronraimist

That should be fixed now.

neilalexander avatar Sep 17 '18 20:09 neilalexander

#75 is fixed but this issue is still present. After being kicked, accept/decline buttons still don't reappear until Seaglass is relaunched

aaronraimist avatar Sep 17 '18 20:09 aaronraimist

I'm having a hard time reproducing what you're seeing - for me this works perfectly. The only reason that the code would not show you the buttons again is if the room you are selecting is not actually an invite.

Can you describe exactly the steps you are taking and whether the invite you are using to reproduce this is a fresh invite?

neilalexander avatar Sep 17 '18 21:09 neilalexander

Sent a video in #seaglass since GitHub doesn't support .mov https://matrix.to/#/!qvCRFpZsGETXtHJSnJ:matrix.org/$1537222180115120qCQAQ:matrix.org

aaronraimist avatar Sep 17 '18 22:09 aaronraimist