This repository has been archived by the owner on Dec 22, 2023. It is now read-only.

Use the correct upper bound when clamping the display line #480

Open

toehead2001 wants to merge 1 commit into jacobslusser:master from toehead2001:DocLineFromVisible-fix

Contributor

toehead2001 commented May 9, 2020 •

edited

Loading

Currently, Lines.Count is used for the upper bound when clamping the displayLine in DocLineFromVisible().

That can be incorrect in the following:

It can be too low if Word Wrap is enabled.
It can be too high if lines are collapsed with Code Folding.

Instead, the actual visible line count should be calculated.


          Use the correct upper bound when clamping display line

f3de3c6

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet