mIRC "reverse" formatting code is not implemented
I searched for any issue that might discuss this, but I can't find one, so I'm reporting it, although I know the developers probably know if it already. Just to keep track of (I presume?) TODO stuff.
The mIRC "reverse" formatting code is not actually implemented in Quassel currently. "%R" seems to be designated as a FormatCode for "Reverse" in /src/uisupport/uistyle.cpp , but it doesn't play well with mIRC's implementation of reversed colors, if it is implemented at all (sorry, I was not able to immediately find where in the source formatting is actually handled). Attached are two screenshots of the same line in mIRC and in Quassel, showing the problem.
For reference, mIRC's control code for Reverse is 0x16, which is accounted for in /src/uisupport/uistyle.cpp , so I guess the problem is just that displaying it is not implemented.
Marking this as "High priority" because the UI gives no indication that mIRC colors support is incomplete, and reversed text is arguably part of mIRC colors.
#1 Updated by flamingspinach about 13 years ago
- File 1104278827.png 1104278827.png added
- File 1104278122.png 1104278122.png added
By the way, it seems that X-Chat implements reverse formatting differently from mIRC. mIRC seems to define "reverse" formatting as having default foreground color set as background color, and vice versa. X-Chat seems to define it as having whatever foreground and background colors are currently defined, if any, swapped. If the "reverse" FormatCode is implemented in Quassel, devs will have to choose one of these interpretations, I guess. Attached are screenshots of both. Incidentally I prefer X-Chat's interpretation since it seems to make more sense, but I guess if you want better accuracy to mIRC's "standards"...
#2 Updated by Sputnick almost 13 years ago
- Status changed from New to Confirmed
- Assignee set to Sputnick
- Priority changed from High to Normal
- Version changed from 0.6-pre to 0.3.0.x
Indeed, we never implemented reverse formatting. IIRC mostly because it's slightly annoying to teach that to the layout engine... So far noone has complained :)
I'll have a look into that at some point, I guess.