aboutsummaryrefslogtreecommitdiff
path: root/fiv-browser.c
diff options
context:
space:
mode:
authorPřemysl Eric Janouch <p@janouch.name>2023-06-01 18:17:20 +0200
committerPřemysl Eric Janouch <p@janouch.name>2023-06-01 19:11:20 +0200
commit4c8df56193b2e43ea8651b731d31380103f99fe4 (patch)
tree00dbc03633df8ef7ac280c8b1b4c4ba287f97e99 /fiv-browser.c
parent200485246bdfda2c77689408dcd407d106639e60 (diff)
downloadfiv-4c8df56193b2e43ea8651b731d31380103f99fe4.tar.gz
fiv-4c8df56193b2e43ea8651b731d31380103f99fe4.tar.xz
fiv-4c8df56193b2e43ea8651b731d31380103f99fe4.zip
Distinguish removed files more prettily
It's still somewhat bad, but at least no longer ridiculous.
Diffstat (limited to 'fiv-browser.c')
-rw-r--r--fiv-browser.c34
1 files changed, 26 insertions, 8 deletions
diff --git a/fiv-browser.c b/fiv-browser.c
index 8c79ca6..4201232 100644
--- a/fiv-browser.c
+++ b/fiv-browser.c
@@ -402,7 +402,7 @@ draw_row(FivBrowser *self, cairo_t *cr, const Row *row)
// Performance optimization--specifically targeting the checkerboard.
if (cairo_image_surface_get_format(item->entry->thumbnail) !=
- CAIRO_FORMAT_RGB24) {
+ CAIRO_FORMAT_RGB24 || item->entry->removed) {
gtk_render_background(style, cr, border.left, border.top,
extents.width, extents.height);
}
@@ -418,21 +418,39 @@ draw_row(FivBrowser *self, cairo_t *cr, const Row *row)
cairo_mask_surface(
cr, item->entry->thumbnail, border.left, border.top);
} else {
+ // Distinguish removed items by rendering them only faintly.
+ if (item->entry->removed)
+ cairo_push_group(cr);
+
cairo_set_source_surface(
cr, item->entry->thumbnail, border.left, border.top);
cairo_paint(cr);
- // Here, we could consider multiplying
+ // Here, we could also consider multiplying
// the whole rectangle with the selection color.
+ if (item->entry->removed) {
+ cairo_pop_group_to_source(cr);
+ cairo_paint_with_alpha(cr, 0.25);
+ }
}
- // TODO(p): Come up with a better rendition.
+ // This rendition is about the best I could come up with.
+ // It might be possible to use more such emblems with entries,
+ // though they would deserve some kind of a blur-glow.
if (item->entry->removed) {
- cairo_move_to(cr, 0, border.top + extents.height + border.bottom);
- cairo_line_to(cr, border.left + extents.width + border.right, 0);
- cairo_set_source_rgb(cr, 0.5, 0.5, 0.5);
- cairo_set_line_width(cr, 5);
- cairo_stroke(cr);
+ int size = 32;
+ cairo_surface_t *cross = gtk_icon_theme_load_surface(
+ gtk_icon_theme_get_default(), "cross-large-symbolic",
+ size, gtk_widget_get_scale_factor(GTK_WIDGET(self)),
+ gtk_widget_get_window(GTK_WIDGET(self)),
+ GTK_ICON_LOOKUP_FORCE_SYMBOLIC, NULL);
+ if (cross) {
+ cairo_set_source_rgb(cr, 1, 0, 0);
+ cairo_mask_surface(cr, cross,
+ border.left + extents.width - size - size / 4,
+ border.top + extents.height - size - size / 4);
+ cairo_surface_destroy(cross);
+ }
}
if (self->show_labels) {