Added background for ViewSource's ImageBox

[ezequielpereira]

• Added a grid background for ViewSource's ImageBox, so they are visible even when they are completely white
• This is a GCI 2015 task: https://codein.withgoogle.com/tasks/5721811266306048/
• Depends on: https://github.com/sugarlabs/sugar/pull/641
• Bug ticket 4909: https://bugs.sugarlabs.org/ticket/4909

[quozl]

Forgot to mention bug 4909, which contains an easy test case.

[ezequielpereira]

Added to the description.

[quozl]

May the image be png instead of gif? We have no gif in sugar-artwork now, and gif has had an unfortunate history.

[ezequielpereira]

Updated.

[ezequielpereira]

Now the background image is in SVG format.

[quozl]

Thanks. This and the related PRs,

Reviewed-by: James Cameron <quozl@laptop.org>

[i5o]

Testing.

[i5o]

Not working for me, for some reason? Maybe I build in wrong way or applied bad the patch?

http://paste.fedoraproject.org/304158/8305331/

[samdroid-apps]

There is a toolkit patch as well (or maybe a shell patch?)

[i5o]

Applied sugar-artwork and sugar one, that was my result

[ezequielpereira]

@i5o On sugar-build the new files aren't added in the build folder, maybe that's the problem.

[i5o]

i'm doing a osbuild clean & pull, and then will apply your patchs & build

[i5o]

Nop, didn't worked..

[ezequielpereira]

Check build/out/install/share/themes/sugar-72/gtk-3.0/assets/viewsource-imageBox-bg.svg, it should be there (or not).

[quozl]

@163gal, you did not change gtk3/theme/assets/Makefile.am?

[i5o]

And here is the why: You forgot to add "viewsource-imageBox-bg.svg" to Makefile.am .

From me it's a -1 for design. Sorry I don't feel comfortable seeing this..

[ezequielpereira]

Let me say I don't know how sugar-build works...

[quozl]

+1 for design, -1 for build system implementation. easily fixed. i don't know how sugar-build works either, but the same problem would have appeared with "make install", therefore rpmbuild or dpkg-buildpackage.

[i5o]

Do you really feel comfortable with that?.. See it for few seconds and move around..

[ezequielpereira]

I fixed the problem in Makefile.am. @i5o How do you want it?

[i5o]

@163gal I think the black is too black? How about with the toolbars color?

[i5o]

Tested with #282828, I think the problem is that the squares are too small :P

[quozl]

@i5o, what screen DPI and resolution?

[i5o]

@quozl

 [ignacio@localhost sugar-build]$ xdpyinfo | grep -B2 dots
screen #0:
  dimensions:    1440x838 pixels (381x222 millimeters)
  resolution:    96x96 dots per inch
[ignacio@localhost sugar-build]$ 

[quozl]

Thanks. A fairly low DPI. Have you a 200 DPI screen you can check with?

[samdroid-apps]

Maybe add another 0 to each number in the background grid? Does that fix the size?

Also, why do you have a grid of circles? Why not rects? https://developer.mozilla.org/en-US/docs/Web/SVG/Element/rect

[ezequielpereira]

@samdroid-apps it is a grid of circles because of the convert command.

[samdroid-apps]

Ok, drawing lots of svg circles can't be good for performance. Maybe hand crafting the svg is better here?

[ezequielpereira]

@samdroid-apps Done.

[samdroid-apps]

Tested on 13" 1080p screen (so high dpi) with sugar-100 theme.

• The squares are wayyyyyy to small. It feels like some optical illusion. Maybe make them 40x40?
  • Depending on the difference between a high/low dpi screen, you may need to scale based on sugar-scaling (so have a -72 and -100 version of the bg image, and use a variable to choose between it, or something to the same effect)
• Black and white is the wrong color. It distracts the user from the image. Maybe white and a very light grey?

[samdroid-apps]

Image:

[ezequielpereira]

I updated the commit, the sugar-100 tiles are now bigger.

[quozl]

Thanks. Cherry-picked for OLPC builds. Don't see why this isn't merged by Sugar Labs.

[i5o]

Oops. I probably forgot, but last time we discussed about the tile size.

I will test it asap

[i5o]

Testing this and https://github.com/sugarlabs/sugar/pull/641 ATM.

@samdroid-apps can you test in high-dpi setup?

[i5o]

Just tested, looks really good for me. You can even notice some part of white icons

[i5o]

Will re-test and then merge.

[quozl]

Yay.

[i5o]

Okay it does not work on fedora 25, maybe due to Gtk changes, but I'll merge it anyway.

@quozl reviewed, and it worked for me in older versions (screenshots are attached in few messages ago ) of fedora/gtk.