Closed Bug 40636 Opened 25 years ago Closed 25 years ago

add pref to delete original text when smileys are inserted.

Categories

(Other Applications Graveyard :: ChatZilla, defect, P3)

Tracking

(Not tracked)

VERIFIED FIXED
Future

People

(Reporter: R6Agent, Assigned: rginda)

References

Details

From Bugzilla Helper: User-Agent: Mozilla/4.0 (compatible; MSIE 5.5; Windows 98) BuildID: 2000052508 In chatzilla, typing :-), :-(, or :-| and pressing enter in a chatroom (i tried in #mozillazine) will output the text and a picture of a smiley face. Reproducible: Always Steps to Reproduce: 1. join a room in chatzilla 2. type in :-) 3. press enter Actual Results: The ouput looks like the line below :-)<yellow face> Expected Results: only a yellow face should be printed not the text
not my code (chatzilla uses it's own)
Actually this should be a preference. Some people might not want a smiley face, just text. And other people might want the smiley face instead of the text. I don't see much point to having both though.
OS: Windows 98 → All
Hardware: PC → All
The point in having both (and yes, this is on purpose) is that the match may be incorrect. If you /msg mozbot stocks, for example, he may come back with "AOL at 57 (+3 7/8)" where the trailing 8) would look like a smiley. If 8) were deleted information would be lost. Anyway, I'll add a pref to delete the text for smileys. Also, you'll be able to turn off individual munger rules at some point, which will make it possible to turn off smiley insertion without disrupting auto-linkifying.
Status: NEW → ASSIGNED
Summary: typing faces draws text and picture in chatzilla → add pref to delete original text when smileys are inserted.
I didn't even think of that rginda. That was a really good example. Maybe we should just delete the text if it is a common smiley, like :) :-) :( :-(, etc. because those would be unlikely to be used in normal conversation except as smilies.
Blocks: 23265
mass marking chatzilla bugs future
Target Milestone: --- → Future
You *should* use the same code as mail/news for detecting smileys. (And so should Navigator, when viewing text/plain files.)
ben, do chatzilla and mailnews share smiley code?
------- Additional Comments From Ben Bucksch 2000-05-25 17:12 ------- not my code (chatzilla uses it's own)
*MASS SPAM* Changing QA contact on all open or unverified ChatZilla bugs to me, David Krause, as I am now the QA contact for this component.
QA Contact: rginda → David
Depends on: 57633
fixed
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Verified fixed with 2000120808 linux build
Status: RESOLVED → VERIFIED
No longer blocks: 23265
Product: Core → Other Applications
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.