Skip to content
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

Increase NavBar preview size on high DPI screens #318

Open
byzod opened this issue Sep 2, 2022 · 13 comments
Open

Increase NavBar preview size on high DPI screens #318

byzod opened this issue Sep 2, 2022 · 13 comments

Comments

@byzod
Copy link

byzod commented Sep 2, 2022

ComparePlus_1.0.0 and npp 8.4.4 (x64)
Original compare plugin works fine (2.0.2.0)

1

btw is it possible to set the color style of navbar individually? I use a grey background, it works fine with the editor but change indicators are barely visible in navbar
2

@pnedev
Copy link
Owner

pnedev commented Sep 3, 2022

@byzod ,
Could you please paste here the Notepad++ debug info?

is it possible to set the color style of navbar individually?

It is currently not possible.
You could try modifying your diff colors making them more contrasting related to your background color.

@pnedev pnedev added the question label Sep 3, 2022
@byzod
Copy link
Author

byzod commented Sep 4, 2022

@byzod , Could you please paste here the Notepad++ debug info?

Notepad++ v8.4.4   (64-bit)
Build time : Jul 15 2022 - 17:54:42
Path : C:\App\Something\Notepad++\notepad++.exe
Command Line : 
Admin mode : OFF
Local Conf mode : ON
Cloud Config : OFF
OS Name : Windows 10 Home China (64-bit) 
OS Version : 21H2
OS Build : 19044.1889
Current ANSI codepage : 936
Plugins : 
	BetterMultiSelection (1.5)
	ComparePlus (1)
	CSharpRegexTools4Npp (1.1.2)
	EmmetNPP (1.0.2)
	HexEditor (0.9.12)
	JSMinNPP (1.2006)
	mimeTools (2.8)
	nppAutoDetectIndent (2.3)
	NppConverter (4.4)
	NppExport (0.4)
	NppTextViz (0.4.2)
	PythonScript (2)

It is currently not possible. You could try modifying your diff colors making them more contrasting related to your background color.

OK that's an option, but I choose light grey as background is all about for a smaller contrast...but considering that compare result is not always shown, high contrast is acceptable

What's more, is it possible to change the relative size of navbar lines? Resize the navbar winodw didn't help

It looks really small in a 4k monitor, I use 200% dpi scale already. I need to be really close to the screen to tell where those lines are....look how large the pointer is compares to those lines...
1

@pnedev
Copy link
Owner

pnedev commented Sep 5, 2022

@byzod ,

I use 200% dpi scale ...

I think that explains the bad markers. It seems they are not resized properly, I'll check that.

You are right about the NavBar on large monitors. I will consider fixing it but I don't know when I'll have the time to do it so it might take a while.

@byzod
Copy link
Author

byzod commented Sep 5, 2022

@byzod ,

I use 200% dpi scale ...

I think that explains the bad markers. It seems they are not resized properly, I'll check that.

You are right about the NavBar on large monitors. I will consider fixing it but I don't know when I'll have the time to do it so it might take a while.

Take your time, it's just a minor visual problem :D

@Yaron10
Copy link

Yaron10 commented Sep 5, 2022

Hello Pavel & @byzod,

Original compare plugin works fine (2.0.2.0).

Pavel,

Any idea what might cause the difference?

Thank you.

@pnedev
Copy link
Owner

pnedev commented Sep 5, 2022

Hello @byzod ,

Please try this build - it will fix the symbols view but unfortunately the images are not resized so they will appear rather small on your 4k display.
Could you try and confirm please?
Thanks.

Hello @Yaron10 ,

The problem comes from the change of the marker symbols. They used to be XMP and now are RGBA and I forgot to set the RGBA images dimensions. Unfortunately I did not manage to resize those and on monitors with very large DPI they will appear small. If I manage to resize those I will write back.

The NavBar will be fixed sometime in the future.

BR

@Yaron10
Copy link

Yaron10 commented Sep 5, 2022

Hello Pavel,

Thank you for the explanation.

Can't NPP's DPIManager be used?
https://github.com/notepad-plus-plus/notepad-plus-plus/blob/6d36d9d398497a6914c64a5804f0400b8bcd6fbd/PowerEditor/src/Parameters.h#L1687

BR

@byzod
Copy link
Author

byzod commented Sep 6, 2022

Hello @byzod ,

Please try this build - it will fix the symbols view but unfortunately the images are not resized so they will appear rather small on your 4k display. Could you try and confirm please? Thanks.

I don't know how do download from that site

@pnedev
Copy link
Owner

pnedev commented Sep 6, 2022

I don't know how do download from that site

Sorry about that.
This is the 32-bit build.
And this is the 64-bit one.

@byzod
Copy link
Author

byzod commented Sep 7, 2022

It's working :D
I don't quite understand what those icon means though, there're so many new ones :D
1

@pnedev
Copy link
Owner

pnedev commented Sep 7, 2022

@byzod ,
Good.

The icons are basically the same. The new info is:
If you see a small dot on the changed, added or removed symbol it means that the same line also exist somewhere else in the file. If there is no dot then the line is unique in the file.
About the moved symbol.... well... what you have attached means that the two lines are actually whole moved block so those two lines appear as a block in the other file. The symbol of the Return line means "start of moved block" while the symbol on the next line means "end of moved block". If the moved block had other lines between those two they would have had different moved symbol - with two vertical hollow dots meaning "part of moved block".
A single moved line will have a symbol without the vertical dots.

I'm currently unable to upload an image, sorry about that.

Hello @Yaron10 ,

Just to let you know that I'm already doing in CP DPI scaling of toolbar images which is basically what Notepad++ does in its DPI manager but it doesn't help with margin symbols - those are in hex and need to be re-scaled by Scintilla itself but the corresponding API (SCI_RGBAIMAGESETSCALE) is not working as I expected.
Thanks.

BR

@byzod
Copy link
Author

byzod commented Sep 8, 2022

@byzod , Good.

The icons are basically the same. The new info is: If you see a small dot on the changed, added or removed symbol it means that the same line also exist somewhere else in the file. If there is no dot then the line is unique in the file. About the moved symbol.... well... what you have attached means that the two lines are actually whole moved block so those two lines appear as a block in the other file. The symbol of the Return line means "start of moved block" while the symbol on the next line means "end of moved block". If the moved block had other lines between those two they would have had different moved symbol - with two vertical hollow dots meaning "part of moved block". A single moved line will have a symbol without the vertical dots.

Cool

@Yaron10
Copy link

Yaron10 commented Sep 8, 2022

Hello Pavel,

👍
Thank you for the explanation.

BR

@pnedev pnedev changed the title Icon of highlighted lines is broken Increase NavBar preview size on high DPI screens Oct 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants