Neop / mudmap2

A mapping tool for text-based games, like text adventures, MUDs and MUSHs
42 stars 12 forks source link

Suggestion: Add Search Functionality #12

Closed avatard closed 6 years ago

avatard commented 6 years ago

v2.4.5

Initially, just to get something out:

Ideally:

My Thoughts:

Neop commented 6 years ago
avatard commented 6 years ago

Sorry, I hadn't touched MUD Map for several months until today.

I didn't see the search option at the bottom right. My bad.

I would probably use an ordered list for search results though? I ran into a situation at work a while back where unordered search results caused some serious confusion during review/change boards.

I noticed that when I'm mapping, I'm working on a particular "Map" and adding "Places." (obviously, yes) If I create a Place in MUDMap, then go NW in my particular game, I'll first search my current MUDMap Map to ensure that the Place does not already exist. I frequently run into scenarios in my game where going a cardinal direction and then the reverse cardinal direction doesn't necessarily take you back where you started. Does that make sense?

Neop commented 6 years ago

Yes, that makes sense to me. The maps and places lists are in fact ordered lists (not sure why I said otherwise in the previous comment), but I noticed that upper and lower case are not ignored (in sorting). Say you have three places "A", "b" and "C" they will be ordered "A", "C", "b". Can you confirm that this is the issue you ran into or is it something else?

avatard commented 6 years ago

I didn't notice the case sensitivity/ACb issue, though I have also done that at work too. LOL!

The existing search works OK. It's just a little un-intuitive. Instead of spending time on a new dialog, I might do something like: 1) CTRL-F to focus on the search box, and 2) ESC to clear the search box/results and return focus to Map/Place.

Complex search/regex like I mentioned before is probably overkill.

Edit: I'd probably prefer defaulting to a Place search instead of a Map search.

avatard commented 6 years ago

Just to clarify, my original issue was that I didn't know there was an existing search option. Sorry about that

Neop commented 6 years ago

Don't worry, it made me find the case sensitivity issue. I've fixed that and added the keyboard shortcuts (ESC only if the search box is focused since it's also used to reset selections on the map). I'll publish a new release in a few days.

On default place search / map search.. While place search is probably used more often than map search I think the map side bar is more useful as an overview when you just opened a world file (especially for first time users). It's just one click to switch anyway.