Browse should set DPI for XO screen

[manuq]

The DPI should be changed for the screen in use. At least in the XO device, it should be changed to 134.

What we used to set in hulahop: ​http://kb.mozillazine.org/Layout.css.dpi

There is an attribute in webkitgtk+, device_dpi gobject property in viewport attributes, it is read-write, but changing it does have no effect in the viewer. Attaching test script.

Here is a blog post about what Safari does: ​http://www.webkit.org/blog/55/high-dpi-web-sites/

[manuq]

Chatted a bit in #webkitgtk+ . So, the device-dpi is only used for pages with the viewport metatag.

There is a ScreenHorizontalDPI and screenVerticalDPI in WebKit, but is not implemented in WebKitGTK+:

​http://trac.webkit.org/browser/trunk/Source/WebCore/platform/gtk/PlatformScreenGtk.cpp#L63

They told me that most things in WebKitGTK+ stack hardcode 96dpi.

[manuq]

Not really setting the DPI in webkit, but changing the font sizes proved to give a very similar look. Pushed as 21b490da36fef775836de15f909717393b92f231 .

[tonyforster]

Installed V135, still getting wrong font sizes on OS7

[manuq]

Replying to tonyforster:

Installed V135, still getting wrong font sizes on OS7

With the new font size the pages will look similar as in previous Browse, but not exactly the same.

Do you see something like this in Wikipedia, for example?

​http://dev.laptop.org/~manuq/browse_dpi/browse-8pt.png

Previous Browse looked like this in the same page:

​http://dev.laptop.org/~manuq/browse_dpi/browse-11.3.1.png

Am I missing something? Or is there a page that looks too much different?

Thanks for testing!

[tonyforster]

The page looks a lot different, I am getting a much larger font

[manuq]

Replying to tonyforster:

The page looks a lot different, I am getting a much larger font

Can you run this in a Python shell and paste the output?

from gi.repository import Gtk
gtk_settings = Gtk.Settings.get_default()
gtk_font_name = gtk_settings.get_property('gtk-font-name')
print gtk_font_name

Also a screenshot that shows the issue you see can be convenient. Thanks!

[tonyforster]

OS7 XO-1

Sorry, Browse behaviour has changed, now it crashes while starting or shortly after. The font size looks OK and the python script gives Sans 10

If I get a bad font size again, I will attempt to capture a screenshot and run the script

The crash during start is error dbus proxies:Introspect error
The crash shortly after is pallettewindows.py ln 499, none type object has no attribute set_invoker

Sorry I cannot attach proper crash logs because os7 will not mount a usb and has no working browser, the Gnome browser failed too

[erikos]

Replying to tonyforster:

OS7 XO-1

Sorry, Browse behaviour has changed, now it crashes while starting or shortly after. The font size looks OK and the python script gives Sans 10

If I get a bad font size again, I will attempt to capture a screenshot and run the script

The crash during start is error dbus proxies:Introspect error
The crash shortly after is pallettewindows.py ln 499, none type object has no attribute set_invoker

Sorry I cannot attach proper crash logs because os7 will not mount a usb and has no working browser, the Gnome browser failed too

Hi Tony, you likely see #3401 on the XO-1. Will be fixed in the next build.

[greenfeld]

Browse for the Spanish OLPC page looks like the attached screenshot in 12.1.0 os10/Browse-137.

There are other things such as both Wikipedia activities' homepage/list of articles which look noticeably different; but I don't know what we can do about that.

[dnarvaez]

Milestone 0.96 deleted