Ignore blank lines containing indentation

[trypsynth]

Some programmers put blank lines between their functions, but with whitespace on the lines. I have no idea why, but nonetheless it's extremely annoying to go through with indent nav. It would be awesome if you could make it so blank lines containing only indentation are ignored the same way standard blank lines are.

[mltony]

Blank lines are indeed ignored. Please provide step-by-step instructions on how to reproduce with a concrete example.

…

On 6/3/2022 1:28 PM, Quin wrote: Some programmers put blank lines between their functions, but with whitespace on the lines. I have no idea why, but nonetheless it's extremely annoying to go through with indent nav. It would be awesome if you could make it so blank lines containing only indentation are ignored the same way standard blank lines are. — Reply to this email directly, view it on GitHub <#6>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJRDHFFLBNH2MJ5PK5S52LVNJTH7ANCNFSM5X2CRUWQ>. You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[trypsynth]

After doing a bit of testing, it only seems to happen if a blank line contains a tab. Download https://www.dropbox.com/s/0tulmvd8ix9ox9e/test.py?dl=1. Attempt to navigate between functions with indent nav.

[mltony]

Ah, right, it doesn't work well with tabs. I took a note - will try to fix that whenever I have time.

…

On 6/3/2022 2:49 PM, Quin wrote: After doing a bit of testing, it only seems to happen if a blank line contains a tab. Download https://www.dropbox.com/s/0tulmvd8ix9ox9e/test.py?dl=1. Attempt to navigate between functions with indent nav. — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJRDHBXTL4PDUBJR4PZIV3VNJ4VNANCNFSM5X2CRUWQ>. You are receiving this because you commented.Message ID: ***@***.***>

[trypsynth]

If nvaccess/nvda#14102 gets merged, this issue should be closed. That PR makes speech.isBlank also detect \t as a blank character.

[trypsynth]

That's a no, but when I have a free weekend I'll tackle this issue. Shouldn't be too difficult.

[mltony]

Sure, feel free to send PR for this.

…

On 9/5/2022 7:31 AM, Quin wrote: That's a no, but when I have a free weekend I'll tackle this issue. Shouldn't be too difficult. — Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AIJRDHCITKYDML3NVUMXRU3V4X72NANCNFSM5X2CRUWQ>. You are receiving this because you commented.Message ID: ***@***.***>