Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

After page refreshes in background whilst editing a draft, the space bar stops working #860

Closed
jdaviescoates opened this issue Jan 17, 2023 · 19 comments
Labels
needs feedback Further information is requested

Comments

@jdaviescoates
Copy link

Describe the bug
This is an odd one.

Very often, whilst editing a draft, the space bar just randomly stops working.

But it happen twice just now and I noticed it is happening after SnappyMail refreshes in the background (I guess checking for new mail or something). I think perhaps something to do with losing focus or something?

To Reproduce
Steps to reproduce the behaviour:

  1. Go to edit a draft
  2. Notice the page refresh behind (I guess it's checking for new mail or something)
  3. Realise the space bar doesn't work anymore
  4. Save the draft, close the window, start editing again, wait for this to happen again, repeat this over and over until I've finished editing my draft
  5. Get rather frustrated

Expected behavior
I expect the space bar to continue working.

Please complete the following information:

  • Browser: Latest Firefox on latest Ubuntu
  • IMAP daemon: dovecot
  • PHP version: not sure, likely 8.1 (that is what the underlying Nextcloud is on)
  • SnappyMail Version: Latest Nextcloud app
@jdaviescoates
Copy link
Author

I just noticed it is also the letter U that stops working when this happens. And perhaps it is when the message is auto-saving as opposed to SnapyyMail checking for new message in the background.

@jdaviescoates
Copy link
Author

jdaviescoates commented Jan 17, 2023

This was getting so annoying that I stopped editing my long email using the Nextcloud app and went to compose it in the standalone app and I can confirm this doesn't happen there (despite the same refreshing/ saving or whatever is happening evidently happening there too - it doesn't break things when it happens like it does on the Nextcloud app)

But when using the Nextcloud app this basically happens every minute or so.

Here's a video of it happening. At 1:05 you can see the screen refresh in the background. After that the space bar (and some others keys) no longer work

https://youtu.be/fnA4ZTx30vE

@the-djmaze
Copy link
Owner

I see your movie and i'm really baffled.

Somehow i can't reproduce this issue in standalone nor nextcloud using Firefox.

@jdaviescoates
Copy link
Author

Yeah, it's baffling. I'll see if I can reproduce it in other Nextclouds I'm running...

@the-djmaze
Copy link
Owner

This issue is similar to #690 and also weird.

  1. Go to: Admin -> Config
  2. Debug section enable javascript

Now in the browser developer console certain messages are logged:

Shortcuts scope set to: Compose
Shortcuts scope set to: Ask
Shortcuts scope set to: MessageList

The last Shortcuts scope is important when typing fails.

@jdaviescoates
Copy link
Author

Yep, that sounds very similar.

I'll give the above a try when I have a moment...

Thanks

@jdaviescoates
Copy link
Author

I haven't noticed this happening recently whilst I'm originally drafting a message, but it does still seem to routinely happen when editing saved drafts...

I've gone into Admin -> Config and enabled javascript but I'm not sure what to do after that. I can't find anything about Shortcuts scope when looking in the browser web developer tools console thing, but here are some warnings etc:

Screenshot from 2023-02-21 09-42-00

@jdaviescoates
Copy link
Author

I've gone into Admin -> Config and enabled javascript

Ah, but I hadn't clicked Save at the bottom and now going into a meeting... more later.

@jdaviescoates
Copy link
Author

Here a new error after enabling javascript debugging:

image

and here's a load of logs mentioning the Shortcuts scope stuff

Screenshot from 2023-02-21 23-29-33

Does that shed any light on this issue?

@the-djmaze
Copy link
Owner

Great!

The scope goes from Compose to MessageList 1 minute later.

When you see the error it:

  1. save draft
  2. reload draft folder
  3. reload message list
  4. CRASH

Can you see why it crashes?
Take a look at the developer network tab, it should give response info about the issue.

@jdaviescoates
Copy link
Author

Looks like the only things failing are to do with notifications?

image

But also says "cached" for those 🤔

@jdaviescoates
Copy link
Author

Or maybe it's something to do with these one that seemingly have no status code?

image

@jdaviescoates
Copy link
Author

Does seems related to app.js imho but I know nothing 😆

image

@jdaviescoates
Copy link
Author

This seems to be the moment it happens:

image

https://youtu.be/YuFnfNf3M-Y

@jdaviescoates
Copy link
Author

But just updated to latest Snappymail Nextcloud app and now it seems it happens here:

image

https://www.youtube.com/watch?v=75TMAUsZQz0

@jdaviescoates
Copy link
Author

Same thing still happens in standalone Snappymail too

image

https://youtu.be/1LYeBn8v3T8

@jdaviescoates
Copy link
Author

I'm really hoping all this will help you to resolve this issue as when I'm crafting a long email it really bugs me 🙏

the-djmaze pushed a commit that referenced this issue Feb 22, 2023
@the-djmaze
Copy link
Owner

I've found something. Maybe it helps in next release.

@jdaviescoates
Copy link
Author

I've found something. Maybe it helps in next release.

Yep, just updated to 2.26.4 version of the Nextcloud app and it seems to have helped! Thanks! 🙏

I'll now close this (and hopefully won't need to reopen it in the future).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs feedback Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants