File type descriptions for non-HTML files shouldn't be "Firefox HTML Document"
Categories
(Firefox :: Installer, defect, P3)
Tracking
()
People
(Reporter: mkaply, Unassigned)
References
Details
When you try to save a webp image using our Save As dialog, the type is "Firefox HTML Document".
It should be WEBP image, similar to JPG/PNG.
Go to this image and try to save it:
Comment 1•4 years ago
|
||
On macOS the dropdown says "Document" but the file extension says webp. What OS are you testing on and what, exactly, doesn't work?
Reporter | ||
Comment 2•4 years ago
|
||
Windows.
It gets the correct extension when we save, so this is only about the text in the dialog. I'll update the bug.
Comment 3•4 years ago
|
||
Molly, is this something left over after bug 1570477 ? Can we change the description of the other filetypes we handle in the registry so we don't pretend they're all HTML docs?
Comment 4•4 years ago
|
||
That string comes from the file type registration (which in this case was created by our installer) unless the application that opened the dialog overrides it using a filter string (of the kind that we generated in this function for saving web pages). We probably shouldn't generated our own filter strings for every file type we try to save, so that brings us to changing what the installer puts in the registry.
Right now (as you can guess from this bug) we have one description for every file type, and it would be a major pain to start having separate ones (we'd have to make a bunch of new handler entries, one for each type, which also means migration code, and new uninstall logic, and ...), but even having just one string there's no reason it should say "HTML" in it now that we're using it for other things.
So I'm moving this bug to Installer and making it about changing that description string.
Comment 5•4 years ago
|
||
tbf I'd also be OK with having the mime info code overwrite some of these descriptions with generic ones. As it is, if you have chrome installed and that registered the filetype first, we'll say "Chrome HTML document" which of course is no better...
Still, because other apps will also use this description, updating our own to not use "HTML" for things that, well, aren't, is probably a good idea, so I'll file a separate bug.
Updated•4 years ago
|
Updated•4 years ago
|
Description
•