Demetrius S. Reporter
	Description • 10 years ago

User Agent: Mozilla/5.0 (Linux; Android 5.0.1; HTC One_M8 Build/LRX22C.H5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.109 Mobile Safari/537.36

Steps to reproduce:

1. Navigated to reddit.com
2. Went into settings
3. Changed the text size to extra large
4. Refreshed reddit.com


Actual results:

There was no change in the size of the text.


Expected results:

The text size should have become larger.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 1 • 10 years ago

I think this is intentional. Make lots of noise in bug 1127441 if you don't like it and maybe the Powers That Be will reconsider. You can also enable the pref manually (see that bug) to get it working locally.

	Kevin Brosnan [Ex-Mozilla]
	Comment 2 • 10 years ago

This works as expected on Galaxy S3 and a Droid Razr but does not work as expected on a Galaxy S5. Our intention was to disable it by default but still allow the settings to work.  :-/

	Demetrius S. Reporter
	Comment 3 • 10 years ago

pref("font.size.inflation.minTwips", 120) Is that the pref that is supposed to enable it? On my HTC One M8 it doesn't work either.

	Kevin Brosnan [Ex-Mozilla]
	Comment 4 • 10 years ago

Yes, suspect that this is display resolution dependent.

	Kevin Brosnan [Ex-Mozilla]
	Comment 5 • 9 years ago

Any thoughts on how to get the desired behavior here? From my limited testing it seems like high DPI devices  and/or devices with desktop like resolutions setting font.size.inflation.minTwips to 0 as the default seems to completely disable font inflation. The plan was to disable by default and allow users to use settings to re-enable.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 6 • 9 years ago

Nope, no idea why this would be DPI-specific. I also looked around in the code and don't see why the settings pane would not update the pref. Did you try changing the pref via about:config vs the settings pane to see if there was any difference?

	Demetrius S. Reporter
	Comment 7 • 9 years ago

Kats, I think the config setting is not working as intended; changing the pref to pref("font.size.inflation.minTwips", 120) is supposed to enable the font inflation, but it appears on high dpi screens it does not work as intended.

	Kartikaya Gupta (email:kats@mozilla.staktrace.com)
	Comment 8 • 9 years ago

Maybe it was something else that regressed this then. Can we get a regression window?

	Bogdan Surd, QA [:bsurd, NI]
	Comment 10 • 8 years ago

Hello,

I've performed a regression up until 2013-02-28 and the issue was still reproducible on a Prestigio (Android 4.4.2) on higher android versions the app would just stop running sooner. Any other earlier build would just start and instantly crash. I do not have any other device on which to continue with my regression window. Hope this helps in any way.

	Oana Horvath [:ohorvath]
	Comment 11 • 7 years ago

This is not a valid issue anymore on Nightly 56 and Beta 55, due to the changes made in Bug 1328868 to the font size setting. All pages are now resized, mobile or non-mobile friendly, including reddit.com. Closing as invalid.