After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 734381 - Remove encoding preference from preferences dialog
Remove encoding preference from preferences dialog
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
git master
Other All
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks: 755292
 
 
Reported: 2014-08-06 20:46 UTC by Michael Catanzaro
Modified: 2016-01-15 16:54 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Remove encoding preference from preferences dialog (9.42 KB, patch)
2014-08-06 20:46 UTC, Michael Catanzaro
none Details | Review
Remove encoding preference from preferences dialog (9.00 KB, patch)
2014-08-06 21:49 UTC, Michael Catanzaro
committed Details | Review

Description Michael Catanzaro 2014-08-06 20:46:31 UTC
How about removing the ability to change the default page encoding from the preferences dialog?
Comment 1 Michael Catanzaro 2014-08-06 20:46:33 UTC
Created attachment 282740 [details] [review]
Remove encoding preference from preferences dialog

I can't see any benefit to this setting, besides to confuse and break
pages.

This patch does not remove the encodings dialog that allows changing the
encoding on the current page, and it doesn't remove encodings from the
window menu. We should consider these as well. Users don't need to know
anything about encodings and Firefox doesn't allow changing encoding at
all anymore.
Comment 2 Michael Catanzaro 2014-08-06 21:49:09 UTC
Created attachment 282746 [details] [review]
Remove encoding preference from preferences dialog

I can't see any benefit to this setting, besides to confuse and break
pages.

This patch does not remove the encodings dialog that allows changing the
encoding on the current page, and it doesn't remove encodings from the
window menu. We should consider these as well. Users don't need to know
anything about encodings and Firefox doesn't allow changing encoding at
all anymore.
Comment 3 Michael Catanzaro 2014-08-27 18:23:23 UTC
We've discovered that Firefox does allow changing the default encoding (Preferences -> Content -> Fonts and Colors -> Advanced) but does not allow overriding the encoding on a per-page basis.  I don't think that means we need this setting, though.
Comment 4 Reinout van Schouwen 2014-08-27 20:50:59 UTC
(In reply to comment #3)
> We've discovered that Firefox does allow changing the default encoding
> (Preferences -> Content -> Fonts and Colors -> Advanced) but does not allow
> overriding the encoding on a per-page basis.  I don't think that means we need
> this setting, though.

I use it sometimes on Gnome changelogs, for example http://ftp.gnome.org/pub/gnome/sources/epiphany/3.12/epiphany-3.12.1.changes . It's served in the wrong encoding (try looking up Matej Urbančič in that changelog).
Comment 5 Michael Catanzaro 2014-08-27 23:21:51 UTC
Hm, I hadn't considered non-HTML documents... maybe we should default to UTF-8 for these, or see how gedit detects the document's encoding....

Interestingly enough, Firefox gets it wrong too.
Comment 6 Michael Catanzaro 2014-08-28 00:44:45 UTC
Let's discuss that further in bug #735559 and leave this bug solely for the preferences dialog option.
Comment 7 Michael Catanzaro 2015-06-24 21:50:31 UTC
Still want to push this. We should not allow changing the _default_ encoding since that breaks the web. If individual documents are broken, the existing encoding menu can be used on a per-document basis.
Comment 8 Michael Catanzaro 2015-08-30 14:00:46 UTC
Pinging reviewers
Comment 9 Michael Catanzaro 2015-12-07 11:20:15 UTC
Ping reviewers again.

We seem to all agree that this setting should never be changed. There is only some concern as to the proper default, which is a WebKit issue.
Comment 10 Carlos Garcia Campos 2016-01-07 07:08:10 UTC
Comment on attachment 282746 [details] [review]
Remove encoding preference from preferences dialog

Ok, we can always bring it back if needed.