-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Fixes Page Scroll Issue #87
Fixes Page Scroll Issue #87
Conversation
Deploy preview for text-to-handwriting processing. Building with commit 9ce4928 https://app.netlify.com/sites/text-to-handwriting/deploys/5f4526fc88cd110007b7bc70 |
Thank you for the PR @CaptainDredge. I haven't checked PR properly but we can't paste content inside page. It is throwing some error in console. Can you check? Also, is there any reason why it is using jQuery and not JavaScript. It is better to avoid additional dependencies as it will make code complicated to maintain. Can it be done without jQuery? |
Use of Jquery adds an additional prototype in getting clipboard data from event Please enter the commit message for your changes. Lines starting
@saurabhdaware I've corrected the paste event handler. Also, Jquery is necessary for event delegation in dynamically added pages. There's neither an easy nor an elegant solution of this in pure javascript. |
@saurabhdaware Have you looked into the PR now? |
Hi there, I am on a break right now so I will try to check it by the next week. I hope it is fine with you. Thanks! |
Hi, I just checked in the preview. It is not working when we have text larger than the area. Normally it is suppose to generate multiple images when a user pastes large text in the textarea. |
No according to updated functionality user can paste as much text as he wants but page will contain only the text it can accommodate on one page. Then you've to add another page to paste the remaining text. And that's how even actual pages in notebook works. Adding the functionality of accommodating all the text and moving it to the next pages according to current status of page capacity itself is a pretty complicated task and that'll essentially make it like MS word. |
No, I meant pasting large text at once. It was one of the reasons to have a scrollbar so that they can paste in one box. There were lots of people initially who wanted a way to generate multiple images in a single paste so the usecase is too common to ignore |
Added the functionality of automatically creating pages when the content becomes more than the current page capacity
@saurabhdaware Added that functionality although I had to do a lot of thinking behind implementing it 😅 |
This looks amazing! Few things are not working as expected:
Let me know if it is possible to fix this otherwise maybe I can take this code to separate branch and see if I can do something. Thanks and not sure if adding page below was intended but it actually looks great! maybe we can have a scrollbar like we had before but have multiple pages in the scroll instead a single section. That would look great |
I think its happening because of the event handlers attached to key down. But the keys pressed here like |
@saurabhdaware have checked on the first issue? I had already resolved the other two |
not yet. I'll merge the PR in separate branch and will check what I can do. Thank youu! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much! I'll see what we can do. Thank you.
So will I be added to the contributor list or not? |
Yes you will be added to contributor list. The list on the website comes from GitHub API so the name will be reflect there after I merge this new branch into master. I will have to make some changes before that. |
@saurabhdaware can you let me know what are the issues here? |
Fixes #81 Fixes #82
In this PR I've tried to solve issue #81 and #82 by making the whole layout of pages discrete rather than continuous ones. It has several advantages in terms of performance as well as user interaction.