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 644247 - Remove UI for minimum font size setting
Remove UI for minimum font size setting
Status: RESOLVED FIXED
Product: epiphany
Classification: Core
Component: General
git master
Other Linux
: Normal normal
: ---
Assigned To: Epiphany Maintainers
Epiphany Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-03-08 22:06 UTC by Xan Lopez
Modified: 2011-03-09 20:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
minfontsizeui.diff (4.78 KB, patch)
2011-03-08 22:08 UTC, Xan Lopez
none Details | Review
gmail broken with 0 min font size (23.97 KB, image/png)
2011-03-09 07:45 UTC, Diego Escalante Urrelo (not reading bugmail)
  Details

Description Xan Lopez 2011-03-08 22:06:35 UTC
Our current understanding of the whole font size mess is that this preference does more harm than good, since it very frequently breaks the layout of web pages. We should set it 0, remove it from the UI, and let the user control the text with the new font size preferences and/or zooming. For those that prefer big fonts with broken layouts, we leave the gsetting where it is, so it can always be changed through dconf.
Comment 1 Xan Lopez 2011-03-08 22:08:55 UTC
Created attachment 182886 [details] [review]
minfontsizeui.diff
Comment 2 Diego Escalante Urrelo (not reading bugmail) 2011-03-09 07:45:34 UTC
Created attachment 182932 [details]
gmail broken with 0 min font size

I agree, but there are some ugly bugs if we do this. Try gmail with 0 or 1 as minimum font size (attached)
Comment 3 Diego Escalante Urrelo (not reading bugmail) 2011-03-09 07:47:06 UTC
(Notice the Spam item, should be bold)
Comment 4 Xan Lopez 2011-03-09 20:47:54 UTC
What Diego is seeing is related to his font size pref, so we'll address that separatedly. Pushed the change to master, will be in the next release.