[Shotwell] Call for testing

[Martin Olsson]

On 06/17/2010 01:31 AM, Jim Nelson wrote:
>> * When I use "Set as Desktop Background" on a 2000x3000
>> (i.e. non-landscape orientation) photo, only a part of the
>> photo is visible on my 1680x1050 screen. It's nice that you
>> switched from the current "scale to fit" to another approach
>> where you add padding instead so that the whole photo is
>> guaranteed to be on screen (this is much better than the
>> current "scale to fit width" which will sometimes crop
>> faces halfway through etc). Maybe you can use a PNG so that
>> the padding is transparent (does that mean the bgcolor in
>> the GNOME appearence dialog is used? not sure...) Otherwise
>> black is probably fine.
>>
>
> Our thinking is that black is a better border for photos than allowing the
> theme color to come through the transparency.  (That's also how we display
> photos in full-window and fullscreen mode, for example.)  I'm not sure an
> image with a lot of transparency would look good with many background
> colors.

The note about the color of the padding was just an extra not,
the bug is about Shotwell using "scale to width" rather than adding
padding. Or did you mean that there was an existing ticket for
the "scale to width" issue? If so what was the bug no (I can't find it).

For example if you download and import this photo and use set as desktop
background, then Shotwell crops the _ball_ out of a basket ball image:
http://www.pe.com/imagesdaily/2008/04-24/lakers24dwba_300.jpg

Well, to get the buggy effect I guess you need to have "Style == Scale"
set under System::Preferences::Appearence (this is the default setting
for Ubuntu 10.04). So actually, I think what needs to happen is that
when you set the desktop background itself you also need to set the
"style" option and the "solid background color" in the GNOME Appearance
dialog as well (you can probably not get away with just adding the
padding as extra black pixels in the image because then it will look
bad if the user changes resolution so setting the solid bgcolor to
black is more versatile).


>> * Workflow issue; two persons take photos at a party/trip or
>> whatever and later share photos with each other. Since the
>> photos are from the same day, Shotwell mixes the two
>> different sets of photos into a single event. This is a problem
>> because I might want to show someone just my photos.
>> Also, if this wasn't a party but some sort of professional
>> photo shot, it's critical to know which photos
>> where taken by whom for copyright reasons etc.
>> Looking at camera EXIF is not sufficient since
>> people might very well have the same camera.
>> This can be solved by implementing "autotag with tag BLAH on import"
>> but I would personally prefer if the default was to import
>> each set of photos into a named event instead.
>>
>
> The current code should only include photos in the same event when they're
> imported at the same time.  That is, if you import 10 photos from a camera,
> Shotwell generates events for those photos.  If you then import another 10
> photos, you should get new events, and not photos mixed in with the old
> events.  If you're seeing this, please let us know, that's a bug.

Ah I accidently looked at the photos view. You're right, it's not a bug.


 >> * The print preview dialog pre-select the "Letter" paper size
>> and also happily informs me that an A4 is 8.27 inch x 11.69 inch
>> but we use "cm" here and most europeans print on A4 papers.
>> This repros even if I go into "System::Administration::Language Support",
>> switch to the "Text" TAB and choose swedish locale there
>> and finally pressing apply system-wide and restarting Shotwell.
>>
>
> Can you tell me which tab of the Print dialog you're seeing this?  We only
> have control over one tab, "Image Settings".  Or do you mean Page Setup?
> Again, that's a GTK control.  If you open Firefox (File ->  Page Setup) do
> you see the same thing?

Oh, right. If I start using:

   LANGUAGE=sv ./shotwell

...I get the right paper size etc so no bug here either.



		Martin