Skip to content

Editor search matches Marker names as Property names#14656

Merged
uckelman merged 2 commits into
vassalengine:release-3.7from
rlament:13052-marker-search
Jun 10, 2026
Merged

Editor search matches Marker names as Property names#14656
uckelman merged 2 commits into
vassalengine:release-3.7from
rlament:13052-marker-search

Conversation

@rlament

@rlament rlament commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

An Editor search advanced filters change. Changes the filter selection for searching for Marker properties to be consistent with other properties.

On the Marker edit pane, the name field is labelled "Property name". It seems logical to search for a Marker in the Property name search category. With this change, Marker search becomes consistent with searches on Calculated Property and Dynamic Property.

Closes #13052

@rlament rlament changed the base branch from master to release-3.7 April 18, 2026 14:45
@uckelman uckelman added this to the 3.7.22 milestone Apr 19, 2026
@uckelman uckelman removed this from the 3.7.22 milestone Apr 27, 2026
Comment thread vassal-app/src/main/java/VASSAL/counters/Marker.java
@uckelman uckelman added this to the 3.7.24 milestone Jun 9, 2026
@uckelman uckelman merged commit 78685c2 into vassalengine:release-3.7 Jun 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editor Search - consider matching Marker names as Property Names

2 participants