Skip to content

synced-tab-dialog: Fix named icons

Alice Mikhaylenko requested to merge wip/exalm/synced-tabs-icons into master

The computer icon was still dark in dark variant. The placeholder icon was just missing because we've moved the icon in resources at some point. Since that's not what we use for tabs anyway, fix that as well.

While we're here, fix a leak - we were passing ephy_pixbuf_get_from_surface_scaled() results into the tree store and forgetting about them.


Before

Screenshot_from_2022-02-23_16-03-39 Screenshot_from_2022-02-23_16-03-41

After

Screenshot_from_2022-02-23_16-05-41 Screenshot_from_2022-02-23_16-05-46

Probably makes sense to get into 42 as well given everything.

Edited by Alice Mikhaylenko

Merge request reports