Collapsing a func that ends in a region shows #endregion

[Janosoft]

Tested versions

• Reproducible in Godot 4.2.1

System information

Windows 11 - Godot 4.2.1 stable - Compatibility render

Issue description

Using GDScript, collapsing a function that ends in a region returns #endregion

Steps to reproduce

func test():
#region Test
print_debug("hi")
#endregion

-> then collapse the func

Minimal reproduction project (MRP)

N/A

[dalexeev]

Confirmed in master. I'm not sure if this can really be considered a bug, since there are 2 overlapping collapsing regions:

func test():          # Begin indent region.
#region Test          # Begin comment region.
    print_debug("hi") # End indent region.
#endregion            # End comment region.

СС @Paulb23

[kitbdev]

This is unrelated to regions, it happens with all comments after the expected end. (and endregion counts as a comment here)

func test():
    print_debug("hi")

# this will be folded.

[Janosoft]

It's not ohhhh WHAT A BUG MAN!
but it is somewhat annoying, and it would be nice to be able to contribute to improving Godot

[kitbdev]

related

• Folds don't work with non-indented content #15690 (comment)
• GDScript Editor Nesting bug #84820
• potential fix String and coments folding scope fix and restructuring #85016

[DinDotDout]

I have also found that something like:

#test
#region New Code Region
func _process(delta):
    pass
#endregion

and folding test will also fold the beginning of fold region.

[DinDotDout]

PR #85016 ensures that comments on adjacent lines are treated as a block, based on the indentation level of the first line. This does not impact regions. This functionality could potentially be incorporated into the PR, as a more comprehensive method for determining what should and shouldn’t be folded is already being addressed.

[DinDotDout]

Region folds may not have a clear answer as the folding the function in the following code should only close up until "var x" unless regions override the fold scope, which may result in more confusing behavior.

func _process(delta):
#region New Code Region
    pass
var x
#endregion