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

Editor - Fixed on scroll - Or option to do so #8489

Closed
thenexus00 opened this issue Jul 28, 2020 · 3 comments
Closed

Editor - Fixed on scroll - Or option to do so #8489

thenexus00 opened this issue Jul 28, 2020 · 3 comments

Comments

@thenexus00
Copy link

thenexus00 commented Jul 28, 2020

When clients are writing in content into an editor property and not using the grid and that text is more then say 30 lines and part of a Document type with other fields, especially with 8 and the loss of tabs to format they have to keep scrolling up and down to access the editor buttons.

It should have an option so on scroll it remains fixed at the top.
Something like this:

Screenshot 2020-07-28 10 03 55

@nul800sebastiaan
Copy link
Member

I am not sure what the problem is? Do you have an illustration of the problem please?

@nul800sebastiaan nul800sebastiaan added the state/needs-more-info We don't have enough information to give a good reply label Jul 29, 2020
@thenexus00
Copy link
Author

I have informed the clients in question who ask to hit the full screen option for now. Here when you have large content you got the full screen and the editor controls stay fixed.

I thought it would be good as you scroll that this did that as well in normal view without having to go there and the mock up shows that.
There are some challenges, with multiple editors when the next one comes into view you overlay that one but even Angular supports that scroll fix effect in it already.

@abjerner
Copy link
Contributor

I just stumbled upon this issue. This seems to be fixed in 8.10 due to this pull request: #9316

As a result of this, I'm closing the issue. But feel free to comment again if it should not work after upgrading 😉

@umbrabot umbrabot removed the state/needs-more-info We don't have enough information to give a good reply label Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants