Merge lp://staging/~sinzui/bzr-gtk/get_style_context into lp://staging/bzr-gtk
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 795 | ||||
Proposed branch: | lp://staging/~sinzui/bzr-gtk/get_style_context | ||||
Merge into: | lp://staging/bzr-gtk | ||||
Diff against target: |
57 lines (+13/-7) 2 files modified
annotate/gannotate.py (+11/-6) branchview/graphcell.py (+2/-1) |
||||
To merge this branch: | bzr merge lp://staging/~sinzui/bzr-gtk/get_style_context | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jelmer Vernooij (community) | Approve | ||
Review via email: mp+122411@code.staging.launchpad.net |
Description of the change
bzr gannotate fails to start using Gtk 3.4 as provided by quantal. The get_style() method and the GtkStyle object are deprecated. Most of the calls were fixed earlier this year, but a few remain and now must be removed. Per the previous fixes, get_style() becomes get_style_
While this branch did fix most of the test suite and all of the issue I see running bzr-gtk tip, there are two test failures that relate to fonts. Fonts in quantal are broken at the moment, there are warnings shown on the command line for all gtk apps that this moment. There are two errors and I am confident that one will be fixed with quantal is fixed. There is an error about setting the monospace font in the DiffView is confusing. The test says the font was not set, but I can see it was. I tried re writing both the test and how the font was set, but there is no change. We may want to consider deleting the assert if the issue is not resolved for quantal.
Thanks Curtis.