Merge lp://qastaging/~midori/midori/hidpiTabs into lp://qastaging/midori

Proposed by Cris Dywan
Status: Merged
Approved by: gue5t gue5t
Approved revision: 7019
Merged at revision: 7105
Proposed branch: lp://qastaging/~midori/midori/hidpiTabs
Merge into: lp://qastaging/midori
Diff against target: 153 lines (+31/-37)
3 files modified
katze/katze-item.c (+1/-1)
midori/midori-notebook.vala (+16/-2)
midori/midori-view.c (+14/-34)
To merge this branch: bzr merge lp://qastaging/~midori/midori/hidpiTabs
Reviewer Review Type Date Requested Status
gue5t gue5t Approve
Review via email: mp+267467@code.qastaging.launchpad.net

Commit message

Scale tab icons taking scale factor into account

Use GLib.Icon all the way.

To post a comment you must log in.
7018. By Cris Dywan

No need to deal with icon size in _midori_web_view_load_icon

Revision history for this message
gue5t gue5t (gue5t) wrote :

Pages like <https://developer.gnome.org/gtk3/unstable/GtkApplication.html> still show a small icon (which makes tabs different sizes) with wk1; locally it looks like changing the "> icon_width" condition to "!= icon_width" (and same for height) is enough to allow upscaling to make sure we get the right size.

Also, the "Tabs" menubar menu as well as bookmarks and history still have blurry icons that we should fix, but I blame Katze infrastructure for that. It might be better to address that stuff in a separate branch.

review: Needs Fixing
7019. By gue5t <email address hidden>

Scale both larger- and smaller-than-expected tab icons

Revision history for this message
gue5t gue5t (gue5t) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: