search

Morph21 / MercuryTrade-Community-Fork

An overlay application for Path of Exile(Tracking, Trading, Chat, etc)
MIT License
140 stars 23 forks source link

[BUG] - Commas now break stash searching, click-to-search no longer works #137

Open SyrupBuccaneer opened 2 years ago

SyrupBuccaneer commented 2 years ago

As seen here https://i.imgur.com/RQprdyK.jpeg this was the item to be sold, however it no longer becomes focused in stash when a comma is added.

This functionality is slightly outdated given a recent update on their end, however in a quad-stash it's not always easy to parse a small highlight vs. the search's functionality of only highlighting that one item. But only searching by name carries with it a potential issue as rares share names regardless of base.

Maybe a regex solution?

Morph21 commented 2 years ago

Does it work if you remove comma from search box?

SyrupBuccaneer commented 2 years ago

Does it work if you remove comma from search box?

screenshot-0020

Yeah that seems completely fine

Reodia commented 2 months ago

Good afternoon. Can you please tell me if it is possible to fix this function? As I understand, it is enough to supplement the function with automatic deletion of 1 character “,” to make it work correctly. Thank you in advance for your answer.

Morph21 commented 2 months ago

Yeah, it should be possible. I will check it out when I'm back from vacation