nea / MarkdownViewerPlusPlus

A Notepad++ Plugin to view a Markdown file rendered on-the-fly
MIT License
1.17k stars 143 forks source link

Contents of Viewer not syncing anymore (Npp v7.7) #107

Open GerardWassink opened 5 years ago

GerardWassink commented 5 years ago

Issue description

After the upgrade to Notepad++ v7.7 (64-bit) this morning, the syncing of the contents in the preview windowrefresh did not work anymore. Tried to 'refresh', but that did not work either. I had to restart the session to see my result. Soame goes for the automatic scrolling, that broke as well.

Markdown to reproduce

  Please post the markdown code producing the described issue, if any

Screenshot

First screenshot, in the middle of an edit session: Capture1

Second screenshot, after having save the file: Capture2_after_saving

Third screen shot, after having performed a View / Refresh: Capture3_after_View_Refresh

Environment

GerardWassink commented 5 years ago

bump!

CennoxX commented 5 years ago

Can confirm the bug for 64 bit, the 32 bit version is having no problems.

GerardWassink commented 5 years ago

Can confirm the bug for 64 bit, the 32 bit version is having no problems.

Okay, thanks. Already an idea of when it's gonna be fixed please?

jg44 commented 5 years ago

Same issue here on 64 bit. Interim workaround is to close and open (by hitting Ctrl-Shift-M twice) which forces a Refresh.

csrichter commented 5 years ago

Can also confirm this bug for NP++ 7.7.1 64bit on Windows 10

medivh91 commented 5 years ago

Same problem in one of my PC,but not occurs in another one.It's strange

GerardWassink commented 5 years ago

@medivh91 could it be that the PC where it does not occur is 32bit? Or, on that PC, where it does not occur, which version of NP++ are you using?

erikblue commented 5 years ago

I have the same issue. I'm running notepad++ v7.7.1 64-bit and markdown++ v0.8.2.25564. My current work around is to just switch document tabs.

medivh91 commented 5 years ago

@medivh91 could it be that the PC where it does not occur is 32bit? Or, on that PC, where it does not occur, which version of NP++ are you using?

Version 7.6.6 is OK.When I update NP++ to 7.7.1,it has same problem

Cybot-BS commented 5 years ago

The issue is still relevant; no automatic updates in the previewer. I run Notepad++ v7.7.1 (64 bit) with MarkdownViewer++ v0.8.2.25564

jedio60 commented 5 years ago

Same here, but hint: another plugin has the same issue! So maybe it's a bug inside Notepad++, not in the plugins?

Details: Tried out in: Npp (1) v7.7.1 64-bit on Win10 (bad) and (2) Npp v7.7.1. 32-bit on Win7 (good). Tried out plugins: MarkdownViewer++ (v0.8.2.25525) and NppMarkdownPanel (v0.4.0) (both downloaded fresh 2019-08-12).

I haven't tried to check if the issue is known to the Notepad++ developers.

GerardWassink commented 5 years ago

Schermafbeelding 2019-08-14 om 18 12 47

"I may be slow to respond"...

I'll say! It has been TWO and a HALF months now. Still nothing. C'mon dude! Your move...

sonichy commented 4 years ago

Same problem. Win7 Notepad++ v7.7.1 (64-bit) MarkdownViewer++ v0.8.2.25564

CennoxX commented 4 years ago

We have enough confirmations of the issue. The issue occurs with MarkdownViewer++ v0.8.2.25564 and older versions but only under Notepad 7.7.1 64 bit, this is already known. Only add new comments if you have new informations.

GerardWassink commented 4 years ago

Any idea of an ETA for a fix?

tmalaher commented 4 years ago

Still happening after upgrade to Npp 7.8.1 64-bit MD Viewier 0.8.2.25564 Win 10 Enterprise 1803

RookTKO commented 4 years ago

Bump. Same issue here.

CorradoLanera commented 4 years ago

same issue here. NPP 7.7.1 (64 bit); MDVPP 0.8.2; win 10pro 1909

elplanenjoyer commented 4 years ago

same issue here. NPP 7.8.1(64bit) MDVPP 0.8.2.25564 Windows 10 pro 1909

wilfreddijksman commented 4 years ago

Topic that contains more info about the cause: https://community.notepad-plus-plus.org/topic/17755/notepad-7-7-breaking-change-of-scnotification-definition-in-x64-build

CoMeIncJM commented 4 years ago

Same issue notepad++ 7.8.9 (64 bit) Win 10 Pro 1909 build 18363.959

JayShoe commented 3 years ago

confirmed broken.

markus-zhang commented 3 years ago

Same issue confirmed. Thanks for the double Ctrl+Shift+M trick.

PawelWozniak commented 3 years ago

This is still an issue. I am using 7.9.2 x64 and Markdown Panel plugin. View is not refreshed.

bazsodombiandras commented 3 years ago

Still not working with Notepad++ 7.9.1 x64 and freshly downloaded Markdown++. Judging by the history of this ticket, it never will.

eduardodarocha commented 3 years ago

Still a issue with Notepad++ 7.9.1 x64 / Markdownviewer++ version: 0.8.2.25564 and NppMarkdownpanel version 0.5.0 Any new plugin to substitute these ones?

CapinWinky commented 3 years ago

As @wilfreddijksman has linked, this is a problem with Npp 64bit v7.7 and up. They changed an API to use 64bit precision for document position instead of the previous 32bit, something that would only come into play for documents approaching 1GB in size. That breaks all plugins that used that API. I don't know the relationship between Npp and Scintilla, but the change is in the Scintilla side.

Because of the ridged way Npp handles plugins (each version of Npp has a fixed list of plugins and fixed versions for each of those plugins) even if this MarkdownViewer++ plugin is changed to work with newer 64bit versions of Npp, it will remain broken in all versions from v7.7 up until the version that first points to the fixed version of this plugin.

Npp has combined a dumb, hard-coded plugin list with a pointless API change (possibly just pulled up from Scintilla) that breaks several plugins in that list. It is ridiculous and indefensible, if anything, Npp should have rolled back the Scintilla version as soon as this disaster came to light. You can rebuild this plugin's .dll with the changes to the Scintilla code related to positions and import it as an off-book plugin to make it work with 64bit or install the 32bit version of Npp instead. You could also step away from Npp altogether in search of a better managed project.

kenlyon commented 3 years ago

Npp has combined a dumb, hard-coded plugin list with a pointless API change (possibly just pulled up from Scintilla) that breaks several plugins in that list. It is ridiculous and indefensible, if anything, Npp should have rolled back the Scintilla version as soon as this disaster came to light.

@CapinWinky Let's keep it constructive. It's also a free product and all things considered, I think it's pretty good.