Open g4borg opened 6 years ago
I am not going to test the plugin against each protocol plugin out there, sorry. ;-)
However, from a first glance, I probably should've used purple_conversation_get_title()
(or purple_chat_get_name()
) instead of conv->name
in all outputs of conversation_handler.c.
From #11 I see that you're building the plugin from source, so if you want to give it a shot on your own, you can reset the plugin by removing all hide-chat-state
nodes from ~/.purple/blist.xml
while Pidgin is not running. Otherwise I'd built a patch for you to try as soon as I can find the time.
thank you for the hint. i might update it, however i wanted to report it, since e.g .for matrix its quite a usability hassle, as matrix rooms have the true name of !SoM3cr4zYhash:domain.tld - while they do have a definite title aswell. in my case, i picked the two rooms i wanted to see each time, and disabled all others, but my usecase is a very small scale one. :)
so i just thought, maybe that would be a good improvement. i understand completely you dont want to test it everywhere :)
The plugin ask if it should show or hide different chatrooms, it should maybe try to include another field (some kind of title or caption) for the rooms - because for matrix rooms it shows nasty hashtag ids.
I am not sure if this is possible.
basicly reproducible by using purple matrix plugin.