Skip to content

broken view after NPP 8.4.3 #45

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

Closed
Deadooshka opened this issue Jul 17, 2022 · 23 comments · Fixed by #48
Closed

broken view after NPP 8.4.3 #45

Deadooshka opened this issue Jul 17, 2022 · 23 comments · Fixed by #48
Labels

Comments

@Deadooshka
Copy link

Wrong sub-window style and scroll bugs.

@ffes
Copy link
Owner

ffes commented Jul 18, 2022

image

Works for me

@Deadooshka
Copy link
Author

snippet_broken
Dark mode seems ok, but normal mode is displayed with wrong background color and distorted scrolling.

@ffes ffes added bug and removed wait-for-user labels Jul 19, 2022
@ffes
Copy link
Owner

ffes commented Jul 19, 2022

That is very strange indeed. Not sure if it is my plugin or if it a bug in n++ itself. I need to investigate it further.

@Deadooshka Deadooshka changed the title broken view after NPP 8.4.4 broken view after NPP 8.4.3 Jul 29, 2022
@emulite
Copy link

emulite commented Jul 29, 2022

Having the same issue. When change from editing php to js and the other way, it seems to 'overprint',
snippets error after changing language

Also in normal mode the drop-down menu has a black background so cannot see options (black on black) untill you move cursor down them
snippets drop down
Using NPP 8.4.4 on windows 10

@kiwichick888
Copy link

kiwichick888 commented Jul 30, 2022

I am experiencing the same problem with the black menu and the 'overprint'. Currently using the latest Npp v8.4.4. Even with a clean install and only Snippets added this is present so it doesn't seem to be related to any non-default plugins.

@ffes
Copy link
Owner

ffes commented Jul 30, 2022

I am AFK now, using that app on my phone to react.

I have looked at it shortly before I left and saw that there was a change in N++ 8.4.3 to make plugins automatically adopt the dark theme. There is no mention that plugins devs need to change their plugins in any way to support this. And it works fine in my tags plugin. But that is a tree and this plugin uses a list which might be exactly the problem.

Therefore I think (without having the possibility to really investigate) it is an upstream problem. Something in N++ had changed and it broke this plugin in the light theme because it uses a docked list.

I think this won't be fixable for me. Someone has to reports this upstream, and I can't do that in the coming weeks.

@kiwichick888
Copy link

a change in N++ 8.4.3 to make plugins automatically adopt the dark theme

According to information given on the Npp community site, v8.4.1 provided a new API call so that plugins can find out whether Notepad++ is in Dark Mode. It didn't say the new API would make plugins automatically adopt Dark Mode. So that may make a difference to how you approach a potential fix. If not, hopefully someone will report it upstream. I absolutely love Snippets and would hate to have to do without it.

@kiwichick888
Copy link

I am AFK now, using that app on my phone to react.

I have looked at it shortly before I left and saw that there was a change in N++ 8.4.3 to make plugins automatically adopt the dark theme. There is no mention that plugins devs need to change their plugins in any way to support this. And it works fine in my tags plugin. But that is a tree and this plugin uses a list which might be exactly the problem.

Therefore I think (without having the possibility to really investigate) it is an upstream problem. Something in N++ had changed and it broke this plugin in the light theme because it uses a docked list.

I think this won't be fixable for me. Someone has to reports this upstream, and I can't do that in the coming weeks.

I have noticed that not all my Snippets libraries have the overprint problem. Some load fine, some overprint. It's very strange. However, I really hope you find a solution for this because Snippets is my absolute favourite Npp plugin and I use it every day.

@ffes
Copy link
Owner

ffes commented Aug 11, 2022

I just reported it upstream: notepad-plus-plus/notepad-plus-plus#12007

@kiwichick888
Copy link

I just reported it upstream: notepad-plus-plus/notepad-plus-plus#12007

Excellent! Hopefully they can provide a solution.

@kiwichick888
Copy link

I just reported it upstream: notepad-plus-plus/notepad-plus-plus#12007

Hey there, I've just checked that link and there's been a workaround suggested. Have you tried it?

@ffes
Copy link
Owner

ffes commented Sep 8, 2022

Yes, I've tried both, but so far it didn't help.

But to test it properly I need to update ALL the N++ header files and for that I need to create a new toolbar icon because the call to add the icon to the toolbar has changed a while ago as well to support dark mode icons. So that is a bit more work and life is quite busy at the moment.

@kiwichick888
Copy link

All good, just checking you'd seen it. Whenever you find the time to deal with it will be great. Thanks.

@Deadooshka
Copy link
Author

as a workaround we can copy stylers.xml into themes\DarkModeDefault.xml and set customized colors in the preferences/dark mode.

@ffes ffes pinned this issue Sep 12, 2022
@warbirdphotog
Copy link

I use Snippets almost daily, such a great tool. But I've been having the same issues as others on here, when I scroll it blackens out the entire list of tags. On mouse over or minimizing to taskbar and then bringing Notepad++ back up it restores them, but can scrolling will turn it back to black. I'm running the latest version of both Notepad++ and Snippets.

@ffes
Copy link
Owner

ffes commented Oct 4, 2022

Today I release v1.7.1 which fixes this issue
https://www.fesevur.com/2022/10/04/nppsnippets-171

If you don't want to wait for the plugin manager of N++ to get this version, download the zip you need, extract NppSnippets.dll and replace the one on your system.

@kiwichick888
Copy link

kiwichick888 commented Oct 6, 2022 via email

@kiwichick888
Copy link

kiwichick888 commented Oct 11, 2022 via email

@ffes
Copy link
Owner

ffes commented Oct 11, 2022

In this new version the 'Show Toolbox Icon' doesn't appear to be working. The button doesn't appear on the toolbar.

Haven't tested it myself yet (always hide the icon anyway), but could you please file a separate issue for this. This is not directly related to this issue, which is about problems redraw problems with the list.

@kiwichick888
Copy link

kiwichick888 commented Oct 11, 2022 via email

@ffes ffes unpinned this issue Nov 9, 2022
@ffes
Copy link
Owner

ffes commented Nov 9, 2022

With the release of N++ 8.4.7, the new version of the plugin is available in the Plugin Admin

@Deadooshka
Copy link
Author

Still white colored gui controls in N++ with dark mode. Appears in Win10 but not Win7.

@ffes
Copy link
Owner

ffes commented Dec 28, 2022

The problem you reported was that it didn't work in light mode. To fix that we had to disable the (broken) mode in N++ itself that automatically adjusts plugins to follow the theme. As a result of that the dark mode doesn't look that great anymore. As long as notepad-plus-plus/notepad-plus-plus#12007 is not fixed the layout of this plugin will remain as it is. There is not much I can do about it. This is an upstream problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants