backgroundcolor left pane does not match GTK Theme color

I have reproduced this. To repro change the background window color of your theme. Most windows take this, however, the shotwell window does not appear to be honoring the change.

Thanks for the bug report. I can reproduce this too. I thought we had fixed this (see http://trac.yorba.org/ticket/2483), but that fix seems to be incomplete. I've reopened that ticket and we'll readdress this for Shotwell 0.8.

shotwell 0.8.1 is now in Natty

Given the simplicity of this fix, can we not easilly get it pulled back for Lucid/Maverick?

if someone could point to a specific commit we could certainly backport it for maverick

The fix was in this commit

http://trac.yorba.org/changeset/2370

Hey adam would this patch do it for shotwell 0.7.2?

Hi Omer,

I'm an engineer at Yorba on the Shotwell team and the patch looks good to me. I'll ask Jim Nelson (the Shotwell project technical lead) to take a look at it and give you a final thumbs up.

Lucas

The code looks fine to patch 0.7.2. I've not tested it, but that was the patch that solved this problem.

OK - our engineers have looked at the patch and it looks OK. I'll emphasize that we haven't built or tested it, however - we do lots of testing on our official releases but if you guys want to cherry pick individual bug fixes then you should do some testing yourselves. I doubt this is a particularly high-risk change, but testing is still always a good idea when pushing to millions of users! :)

i'll prepare a branch with this fix. On the testing part we usually dont push updates right away, we have -proposed repository for those people who wish to help us with testing the bug fixes so when we are certain there there were no regressions with the update they move to the -updates repository and yes I just picked the change from the upstream commit and made it suitable for 0.7.2

also, is there a shotwell 0.7.3 release expected?

Jeremy, could you test the shotwell package from this ppa https://launchpad.net/~om26er/+archive/test/ and see if that solves the issue (I have hard time reproducing the issue).

Omar: We are not planning an 0.7.3 release at this time. (Our development efforts are currently focused on 0.9, which we plan to release in time for Ubuntu 11.04 this spring.)

Omer: oops - I spelled your name wrong in my previous comment. Sorry for the typo! :)

I think this is suitable for an SRU to maverick, and the branch looks good to upload, but we should first provide more details for testers to reproduce.

@m038: can you please comment with the theme you are using to reproduce this bug?

@Ken VanDine
The repo in ubuntu 10.10 still has 0.7.2