Find province command

Hepps

Couldn't the search line simply ignore the special characters? Instead looking for a match in simply spelling?

So "Lhûn" could then be searched as Lhun?

or "Lands of the Éothéod" could be searched as Lands of the Eotheod?

redrum

@hepps You'd need a way to map the special û character to u and é to e. Essentially a list of supported character mappings.

Cernel

@hepps Yeah, of course, the program doesn't see the characters looking similar to some basic ones, unless of course a full list telling it so; and anyways either the special characters may look or ever be not just a variant of a basic word, for example you can have Ælf, and you cannot say that is an A or an E, since it became an E in "elf", but it became an A in "Alfred", or you can have words that are multiple, like the German ß, that would be an ss or sz (also the English W is not a word of the actual Latin alphabet, and started as a way to mean VV, but, of course it's now not "special", but just because it is English (if the Germans would have decided what is standard, probably the ß would be on all the world's keyboards as well)) or you can have words that look like some, but eventually became something totally different, like the Polish Ł, that is practically the same sound as the English "W" or Italian "U".

Hepps

Cool, was just making an uniformed suggestion.

Hepps

Could a new properties file be added for games that use special characters?

Say... a "Search Properties" File

Where the map maker defines the plain English character for any and all special characters used within a game?

ssoloff

Just wanted to leave this here so people don't spend a lot of time thinking how to convert accented characters to unaccented characters for this feature... In most programming languages (including Java) one can find a library to "normalize" Unicode accented characters to pairs of the corresponding unaccented characters and characters representing the accent marks themselves. Therefore, with some creativity, it should be possible to match the unaccented text a user types to the accented territory names specified in the map.

I was planning on taking a look at this feature, in general, this weekend, but not sure if I'll get to this specific aspect of it.

Cernel

@ssoloff Does it support also alternative characters, like seeing Ƿ as W? And how does that work for stuff like Æ or ß?

ssoloff

@cernel Normalization typically only handles diacritics, and so would not modify the character examples you provided.

If those characters are typically used in maps, and we have to handle them, Java does provide a way to customize how strings are collated so that, for example, "ss" would be considered equal to "ß". However, that's a bit trickier to get right in all scenarios.

Are there examples of such characters in maps today? Or are you simply envisioning them as a possibility in the future?

Cernel

@ssoloff said in Find province command:

Are there examples of such characters in maps today?

Don't think so. Actually, "Battle for Arda" is the only map I can think of with any special characters at all.

ssoloff

@cernel Then I'd suggest we just focus on accented characters for now and deal with other issues, such as ligatures, if and when they arise. It will keep the code simpler.

Frostion

Before adding support for strange and very rarely used characters, maybe we should think about if players/people are actually able to chat-type the characters, know what they mean or even pronounce them. Maybe, apart from using the characters for the displaying of territory names visually correct (like in LotR), it might not be smart to implement weird (and unknown to many) characters that people can't use or read correctly.

Most languages have italic style stand-ins for their special characters for practical reason. Maybe that is actually smarter than presenting strange unrecognised characters.

I'm totally for the support of new characters, but I would never use any that might not be obvious and easily recognized when it came to the reading of the characters.

ssoloff

There is a pending PR for the first iteration of this feature: https://github.com/triplea-game/triplea/pull/4115

A question that has come up is where would most users expect to find the new Find Territory... command on the TripleA main menu. Typically, such "find" commands would appear on an application's Edit menu. As TripleA has no such menu, this leaves choosing an appropriate location for the new command a bit less straightforward.

As proposed in the above PR, it is at the bottom of the Game menu. Other proposed locations have included:

Somewhere else on the Game menu (e.g. at the top).
Somewhere on the View menu.
Somewhere on the Help menu.
As the sole item on a new Search menu (which could appear, for example, between the File and View menus).

Any other ideas beyond those proposed above are welcome, as well.

zlefin

I wonder if, in addition to wherever else it is placed; there should be an instance of it in the "territory" tab on the right side of the screen, i.e. when you have the territory tab selected. (the place that also has notes, actions, players, etc)

ssoloff

@zlefin I failed to mention above that, as part of the aforementioned PR, the hotkey Ctrl+F will also bring up this feature.

If a sufficient number of users feel having an additional Find Territory button on the Territory tab, presumably under the Battle Calculator button, is beneficial, we can certainly add it. If such a button were added, my guess is whatever territory is currently selected in the Territory tab would be used to seed the Find Territory dialog. That, in and of itself, is probably useless since the user already knows where the selected Territory is because they selected it. But another access point for the dialog couldn't hurt.

Hepps

I would think the View menu would make the mot sense.

prastle

@hepps I agree with @Hepps and I read your pr on git and agree with you as well @ssoloff

ssoloff

The first iteration of this feature is now available in pre-release 12238 or later. In this version, one can only select the territory to "find" from a drop-down list. I will be submitting a new PR shortly that will also give the user the option of typing the territory name (with auto-complete).

EDIT: BTW, I did move the command to the View menu, as recommended above. Ctrl+F continues to work unchanged.

ssoloff

Pre-release 12242 now allows users to type a territory name in the Find Territory dialog with auto-complete. No support yet for normalizing accented characters, however.

I'd really like to hear from players if the way auto-complete behaves meets your expectations. I thought it was a bit strange at first, but quickly got used to it. However, I'm biased.

ssoloff

Pre-release 12254 now briefly animates the territory border after the territory is centered on the map. This should make it easier to identify territories that may not have a name on the map (e.g. sea zones in Classic)

Hepps

@ssoloff Smart