JohnMcLear / ep_page_view

Etherpad Page View plugin
https://npmjs.org/package/ep_page_view
14 stars 17 forks source link

Automatic PageBreaks disappear #20

Open deepsaggas opened 9 years ago

deepsaggas commented 9 years ago

While editing the line just before 'computedPageBreak', page break disappears. How to reproduce: 1) Open a pad with enough text such that it has a pagebreak (automatic ones not manual). 2) Now edit a line just before computedPageBreak(line which contains div with class='pageBreakComputed'). 3) The computedPageBreak disappears

Possible reason for failure: redrawPageBreaks() is not called from aceEditEvent hook because callstack type is 1) idleWorkTimer with callstack.callstack.docTextChanged = true or 2) handleKeyEvent with callstack.callstack.docTextChanged = false

JohnMcLear commented 9 years ago

Is this proving to be a major issue for you? I feel like by fixing it we will lose a lot of optimizations!

deepsaggas commented 9 years ago

It is an issue that I think should be fixed but on the other hand if it makes editing too slow, then there is no point in fixing it as page breaks are secondary, they will be used less than typing text obviously. Were you able to isolate the issue?

deepsaggas commented 9 years ago

closed by mistake

monkey-pawan commented 6 years ago

Hi, any update here? I would like to know the fix, please. Even if you can't commit the fix to repository due to optimization issue or whatnot.

JohnMcLear commented 6 years ago

I'm not working on it due to not having time to work on it, you should hire a freelancer to work on it.

monkey-pawan commented 6 years ago

It's very unlikely to remember what you did about 2 years ago, but if by any chance you remember anything and give me a hint or point me to something useful, I would really appreciate it.

I tried calling reDrawPageBreaks() at aceEditEvent, it stopped disappearing but created other issues like generating new <div>s on Up Arrow. Thank you for your time, even if you don't remember. 😅 Awesome work you did in all those years. 👍

JohnMcLear commented 6 years ago

sorry


From: monkey-pawan notifications@github.com Sent: 26 Oct 2017 1:50 pm To: ether/ep_page_view Cc: John McLear; Assign Subject: Re: [ether/ep_page_view] Automatic PageBreaks disappear (#20)

It's very unlikely to remember what you did about 2 years ago, but if by any chance you remember anything and give me a hint or point me to something useful, I would really appreciate it.

I tried calling reDrawPageBreaks() at aceEditEventhttps://github.com/ether/ep_page_view/blob/master/static/js/page_view.js#L359, it stopped disappearing but created other issues like generating new

s on Up Arrow. Thank you for your time, even if you don't remember. ? Awesome work you did in all those years. ?

- You are receiving this because you were assigned. Reply to this email directly, view it on GitHubhttps://github.com/ether/ep_page_view/issues/20#issuecomment-339655089, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AANewMLylh0ZvtFcRrHrf7nzKyzjl3cSks5swH_6gaJpZM4DgCej.

monkey-pawan commented 6 years ago

Np.