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

Composer window bug (after startup of app) - missing cursor incorrect input recognition #383

Closed
itisafrickinhighlander opened this issue Mar 8, 2021 · 9 comments

Comments

@itisafrickinhighlander
Copy link

When I start EM and go to reply/compose an email, there is no visible cursor in the text field. When I start typing it is fine until I hit Enter, it doesn't create a new line, instead it moves the text one line down. Delete would work just fine, backspace would be deleting those new lines, formatting wouldn't work either.

My fix is to click on DB view or just the account tile in the EM sidebar, after that cursor will visually appear again and it will work as it should. Also, I think it goes away eventually, but I have no idea why or how.

Here is a short showcase of this
https://user-images.githubusercontent.com/54138438/110292602-c7250180-7fed-11eb-8e96-1a41ad106130.mp4

@vladimiry
Copy link
Owner

Does the same scenario work well in a regular browser (chrome / chromium)?

@itisafrickinhighlander
Copy link
Author

A good thought.
I have just tried a Chromium and this bug doesn't manifest there at all, editor works just fine. Same applies to FF, of course.
It seems like yet another Electron's specialty. Just as a note, I am using the current release v4.10.2.

@vladimiry
Copy link
Owner

Kind of OS and app package used?

@itisafrickinhighlander
Copy link
Author

OS: Win10 and also Debian 10.
EM versions: I tried all starting from the latest down to 4.9.2, all of these showed up this bug.

It will show up right after the startup of app until you, e.g. go to PM settings and back, click on the DB view in EM or just click the tile of the account in EM's control bar.

@vladimiry
Copy link
Owner

Can confirm that it's reproducible on my side too which should help with the fixing process.

@vladimiry
Copy link
Owner

vladimiry commented Mar 23, 2021

Fix has been implemented in df58f0c. The change also improves focusing in terms of using the protonmail hotkeys. For example the up/down arrows keys navigation should be working without a need to manually focus the protonmail's mail list view.

@vladimiry
Copy link
Owner

@itisafrickinhighlander can you verify issue resolving in this build?

@itisafrickinhighlander
Copy link
Author

Done, it is fixed in the provided build.

@vladimiry
Copy link
Owner

vladimiry commented Apr 10, 2021

One originally enabled change, related to improving initial focus in @ProtonMail UI, has been lost in v4.12.0 release. The change is not critical but recommended for better UX. For concerned users, the fixed build uploaded here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants