-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
Full screen mode when editing only makes window taller not wider #11942
Comments
There is an ideal page width for reading text. Reading very wide text is cognitively harder to the human eye as you have to scan left to right. Also in the world of 140 characters, we should not encourage long emails. No one reads emails longer than 5 sentences anyways. https://lifehacker.com/280193/limit-your-email-messages-to-5-sentences |
I appreciate your explanation Rushabh But something that you haven't taken into account is when you have a long email conversation with customers. The usability gets greatly degraded as you have to scroll up and down the screen much more than you should with just a very narrow modal view ... |
That modal is surely small sized for an embedded text editor, forget normal. Lol. |
Using demo.erpnext.com
ERPNext: v9.x.x-develop (02ac901) (develop)
Frappe Framework: v9.x.x-develop (ec33e82) (develop)
Chromium Version 62.0.3202.94
Firefox 57.0.1
To replicate
On an opportunity, Click 'New Email' (Reply to email gives the same problem too)
Wish to use full screen mode to edit an email.
Click the full screen button in the editor toolbar
This problem is also seen on the Issues.
The window lengthens, but does not widen. This used to work and provide true full screen editing capability. Perhaps it should also be in one of the unit tests ? I note that for other screens such as To do that using the full screen button does provide full screen editing. Likewise on some modal views there is a 'Edit in Full page' link which works fine too. Perhaps the emails creation view should have this too?
The text was updated successfully, but these errors were encountered: