xterm256 support in colour trigger pattern

Bug #1408515 reported by Vadim Peretokin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mudlet
Opinion
Wishlist
Unassigned

Bug Description

We should add xterm256 support for matching triggers with. Has been requested by a user: http://forums.mudlet.org/viewtopic.php?f=5&t=4684#p22271 and doesn't make sense to leave it out since we do support xterm256.

description: updated
description: updated
Stephen Lyons (slysven)
description: updated
Revision history for this message
Stephen Lyons (slysven) wrote :

It will need a new dialogue as the current selection of a colour is from a pop-up dialogue (dlgColorTrigger class) with 16 buttons that is called from dlgTriggerEditor::slot_color_trigger_fg() and dlgTriggerEditor::slot_color_trigger_fg(). I think I did play with this a while back so I will have to dig around and see how far I got.

I think I got bogged down with the fact that those 16 colours used on the existing selection dialog are the user redefine-able 16-ANSI colours and I was unclear where to get the other (256 - 16) colour definitions from - also there needs to be a way to choose the "default" foreground or background colour (depending on the caller slot method)...

Things may also be complicated as I think there are duplicate colours in the 256 colour set where there are the same RGB values for the 16 ANSI colours, the 6^3 other colours and the 24 grey-scale - for colour triggers do we detect on the colour RGB values or on the ANIS colour codes?

Revision history for this message
Vadim Peretokin (vperetokin) wrote :

Migrating issues to Github, please follow the new discussion here: https://github.com/Mudlet/Mudlet/issues/703

This issue needs to be closed and there is no appropriate status, so will set it to "Opinion" just for migration purposes.

Changed in mudlet:
status: New → Opinion
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.