Bug #323
Partial self triggered highlights
| Status: | Closed | Start: | 01/28/2010 | |
| Priority: | Normal | Due date: | ||
| Assigned to: |  Mirco Bauer |
% Done: | 100% |
|
| Category: | Engine IRC | |||
| Target version: | 0.7.1 | |||
| Complexity: | Low |
Found in Version: | ||
| Votes: | 0 |
Description
You usually can't trigger self triggered highlights (if you say your own nick for example), but if combined with other formattings you will get a partial highlight.
This triggers the issue:
This triggers the issue:
Hi $your_nick, _this_ triggers partial self highlights!
Associated revisions
Revision 6215995285e74fba4fe0c4147d6296c1405475e9
Fixed partial self triggered highlights (closes: #323)
History
Updated by Mirco Bauer 5441 days ago
- Status changed from New to Closed
- % Done changed from 0 to 100
Applied in changeset 6215995285e74fba4fe0c4147d6296c1405475e9.
Updated by Mirco Bauer 5435 days ago
- Status changed from Closed to Assigned
- Target version changed from 0.7.1 to 0.7.2
Reopened, seems like this isn't completely fixed.
Half of this message for example was orange and the rest (behind "*is*") not:
Half of this message for example was orange and the rest (behind "*is*") not:
20:49:06 <directhex> meebey, thanks for your help on this btw. i want to get it as right as possible now, despite mad upstream, so that a) when it *is* debianable, it's already correct in binary terms, and b) make it so there's no delta between ubuntu/unofficial packages and eventual debian packages
Updated by Mirco Bauer 5431 days ago
Actually I think this partial highlight is wanted. The idea was to only highlight the word that triggered it, but splitting the message down into smaller pieces would add overhead to the message size as the object tree grows. So the fix would be to implement that "only highlight words" instead of the message "chunk" that contains the highlight word.
Updated by Mirco Bauer 5309 days ago
- Status changed from Assigned to Closed
- Target version changed from 0.7.2 to 0.7.1
This issue was already solved but didn't address the inconsistent highlight issue as reported in #344, thus closing this now.