Because right now they appear as (text/plain) which makes them indistiguishable from other text files. Bugzilla does keep track of whether it's a patch or not (it's a separate radio button when you're uploading the attachment), so why not display it?
True, I didn't notice that. But in createattachment.cgi it says "text/plain, diffs", we should use the same wording in each place (but I'm not sure which one is better). The initial comment is a bit terse :) - that's why I was asking.
How about just showing (patch)?
I agree they should be just patch - a patch should never have another MIME type to my knowledge.
Patches are now distinctly identified on the patch list. Timeless, if you concurr, please mark this fixed.
->New (old by now?) product Bugzilla
Timeless has nothing to say, and I agree with Dave, so marking fixed.