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

[BUG] Zoom breaks breaking column render #390

Open
NPLN138 opened this issue Feb 17, 2025 · 4 comments
Open

[BUG] Zoom breaks breaking column render #390

NPLN138 opened this issue Feb 17, 2025 · 4 comments
Assignees
Labels
bug Something isn't working ready for release

Comments

@NPLN138
Copy link

NPLN138 commented Feb 17, 2025

Image
Somehow zoom creates empty space which does not fill.
In fact, the entire code will fold instead of just setting a UI zoom level.

@SpartanJ
Copy link
Owner

SpartanJ commented Feb 17, 2025

Hi @NPLN138!
I'm not sure if I follow.
So you zoomed in with ctrl+wheel up and the line wrap was not updated?
Can you please tell me your:
"Wrap Mode"
"Wrap Against..."
how did you zoom
and if resizing the side panel or the window will update the line wrap correctly

Also: are you using the latest version? Because there was a fix for something related and your screenshot doesn't seem to be from the latest version since the latest highlights an extra char block on new line selection.

@SpartanJ SpartanJ self-assigned this Feb 17, 2025
@SpartanJ SpartanJ added the bug Something isn't working label Feb 17, 2025
@NPLN138
Copy link
Author

NPLN138 commented Feb 17, 2025

Ecode is 0.7.0.
Wrap mode is set to character-wrap against line breaking column.
It happens when zooming either through mouse wheel or when using the
buttons in the menu.
Indeed, resizing the side panel or window will update it correctly.
Can the side panel size be reset?

@SpartanJ
Copy link
Owner

wrap against line breaking column

Ohh, the problem is there, it's not updating the wrap on zoom when wrap against line breaking column is set, I'll send a fix in a couple of minutes. Thanks for reporting it!

SpartanJ added a commit to SpartanJ/eepp that referenced this issue Feb 17, 2025
@SpartanJ
Copy link
Owner

A fix has been pushed. Fix will be available in the nithgly build after ~15 minutes this message.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for release
Projects
None yet
Development

No branches or pull requests

2 participants