Fix highlighting of range selections that span till the end of the score

cbjeukendrup · cbjeukendrup · commit a5b8c284135f · 2025-02-11T15:42:50.000+01:00
diff --git a/src/notation/internal/notationselectionrange.cpp b/src/notation/internal/notationselectionrange.cpp
@@ -167,12 +167,12 @@ mu::engraving::Segment* NotationSelectionRange::rangeEndSegment() const
 {
     mu::engraving::Segment* endSegment = score()->selection().endSegment();
 
-    if (!endSegment) {
-        return nullptr;
+    if (endSegment && !endSegment->enabled()) {
+        endSegment = endSegment->next1MMenabled();
     }
 
-    if (!endSegment->enabled()) {
-        endSegment = endSegment->next1MMenabled();
+    if (!endSegment) {
+        endSegment = score()->lastSegmentMM();
     }
 
     return endSegment;

Original file line number	Diff line number	Diff line change
`@@ -167,12 +167,12 @@ mu::engraving::Segment* NotationSelectionRange::rangeEndSegment() const`
`167`	`167`	`{`
`168`	`168`	`mu::engraving::Segment* endSegment = score()->selection().endSegment();`
`169`	`169`
`170`		`- if (!endSegment) {`
`171`		`- return nullptr;`
	`170`	`+ if (endSegment && !endSegment->enabled()) {`
	`171`	`+ endSegment = endSegment->next1MMenabled();`
`172`	`172`	`}`
`173`	`173`
`174`		`- if (!endSegment->enabled()) {`
`175`		`- endSegment = endSegment->next1MMenabled();`
	`174`	`+ if (!endSegment) {`
	`175`	`+ endSegment = score()->lastSegmentMM();`
`176`	`176`	`}`
`177`	`177`
`178`	`178`	`return endSegment;`