NppFTP icon indicating copy to clipboard operation
NppFTP copied to clipboard

NppFTP doesn't use Npp Style configuration

Open ramiy opened this issue 9 years ago • 15 comments
trafficstars

NPP allows to change theme from Settings > Style Configuration screen.

Seems like NppFTP uses its own styles, it doesn't use the theme style.

screenshot 2016-06-19 18 24 16

It should be fixed.

ramiy avatar Jun 19 '16 15:06 ramiy

On the other hand, I really like this white background. (left side in my case, so that I don't have the code near to the edge of my screen)

I don't use the recent Workspace feature only because of that. :joy:

vlakoff avatar Jun 25 '16 14:06 vlakoff

@vlakoff if you like the white background, choose white style. Others like black background, they should see black backgrounds. Green background lovers should see green backgrounds.

The users should choose what they loves. The software needs to comply to user chooses.

ramiy avatar Jun 25 '16 17:06 ramiy

I do use a Notepad++ style with dark background. But all my plugin panels (Light Explorer, NppFTP, SourceCookifier) have white backgrounds, which clearly distinguishes them from the code view.

Ideally, plugin background should be customisable: "inherit" (use Notepad++ background color) or a specified custom color.

vlakoff avatar Jun 26 '16 03:06 vlakoff

+1 for this

@ashkulz @chcg Are you planning to support this functionality? It'd be a great UX improvement if we were able make the background color of the NppFTP plugin match that of the text editor background. Even if it doesn't read from the primary Notepad++ style configuration, just being able to set the NppFTP plugin colors manually would be a huge improvement.

ElliotNB avatar Jul 17 '18 05:07 ElliotNB

Currently I'm not aware how to take over the background colour from the current active theme. Would need some further investigation or some hint\example how to do it. So I think there will be no shortterm fix on this topic.

chcg avatar Jul 17 '18 21:07 chcg

@donho Would you be able to advise on how the NppFTP plugin might take over the background color from the current active theme? NppFTP is one of the most popular plugins for Notepad++ and it'd really help the overall UX experience if it were possible to set the NppFTP background color to match whatever background color has been set to the main text editing window.

ElliotNB avatar Jul 18 '18 03:07 ElliotNB

I'm in total agreement of this feature for better UI uniformity/consistency, less eye strain.

drewwesley avatar Feb 22 '19 00:02 drewwesley

@ashkulz @chcg Just checking back in -- any luck finding time to investigate a possible solution for this? I know a lot of us use dark themes with Notepad++ and it would be wonderful to have the NppFTP dark as well.

@donho Any chance that you could help point @chcg in the right direction?

ElliotNB avatar Oct 21 '20 01:10 ElliotNB

@chcg @ElliotNB Here are 2 needed API:

	#define NPPM_GETEDITORDEFAULTFOREGROUNDCOLOR    (NPPMSG + 90)
	// INT NPPM_GETEDITORDEFAULTFOREGROUNDCOLOR(0, 0)
	// Return: current editor default foreground color. You should convert the returned value in COLORREF

	#define NPPM_GETEDITORDEFAULTBACKGROUNDCOLOR    (NPPMSG + 91)
	// INT NPPM_GETEDITORDEFAULTBACKGROUNDCOLOR(0, 0)
	// Return: current editor default background color. You should convert the returned value in COLORREF

And search setBackgroundColor in Notepad++ project your will see how different panels set their background color.

donho avatar Oct 21 '20 02:10 donho

@donho Thank you!

ElliotNB avatar Oct 21 '20 02:10 ElliotNB

@ashkulz Any chance you'd be able to take a look at this with the tips provided by donho? Thank you!

ElliotNB avatar Nov 25 '20 19:11 ElliotNB

@ElliotNB I haven't used Windows in ~5 years, @chcg is the maintainer :slightly_smiling_face:

That said, I'm pretty sure a PR would be reviewed/merged more quickly :+1:

ashkulz avatar Nov 26 '20 05:11 ashkulz

See https://github.com/funap/npp-explorer-plugin/blob/515e1e5923479c1fb347e4a458e609406fa4573d/Explorer/src/MISC/NppInterface.cpp#L67-L75 on how to implement the retrieval of the foreground/background colours.

They need to be used at:

https://github.com/ashkulz/NppFTP/blob/master/src/Windows/OutputWindow.cpp#L302-L316

for the output window.

chcg avatar Dec 14 '20 19:12 chcg

Please make it customizable. Its a shame that this plugin doesn't follow the theme settings of his master.

MIOsystems avatar Jul 23 '22 20:07 MIOsystems

EDIT: I AM AN IDIOT. I didn't realize nppftp already follows styling of the NPP window its attached to, it just doesn't update. When I was testing my build that possibility did not enter my mind. Sorry for the false alert. What happened was I'd noticed nppftp inherit styling from a window. so if i made a new window, changed the style on the new window, and only after that open NppFTP, it would grab the style from that window. of course it doesn't update. Please see it in action, it might at least be useful to some person here who needs a quick fix to style the ftp window. I'll try again this weekend. notepad++_QZHIvkMWCL

zitot avatar Feb 09 '24 04:02 zitot