TakeV
/
Tooth
mirror of https://github.com/TakeV-Lambda/Tooth.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

feat(attachment): ALT dialog 

having alt text as a tooltip is not that useful considering its size and inablity to select it
this commit creates a dialog with a selectable label for it as well as set it as alternative text on the Gtk.Picture for screen readers

fix: #64
This commit is contained in:
Evangelos Paterakis 2023-01-25 11:37:35 +02:00
parent 4e53d0e4b5
commit 78a183fdae
No known key found for this signature in database
GPG Key ID: FE5185F095BFC8C9
3 changed files with 67 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
data/style.css
View File

@ -165,3 +165,18 @@ headerbar.flat.no-title .title {
.ttl-status-line-height-large {
line-height: 1.4;
}
.ttl-status-badge {
background-color: rgba(0,0,0, .8);
border-radius: 6px;
margin-left: 8px;
margin-bottom: 8px;
}
.ttl-status-badge > * {
padding: 6px 12px;
}
.ttl-status-badge > button > label {
padding: 6px 12px;
}

1
src/Widgets/Attachment/Image.vala
View File

@ -17,6 +17,7 @@ public class Tooth.Widgets.Attachment.Image : Widgets.Attachment.Item {
protected override void on_rebind () {
base.on_rebind ();
pic.alternative_text = entity == null ? null : entity.description;
image_cache.request_paintable (entity.preview_url, on_cache_response);
}

55
src/Widgets/Attachment/Item.vala
View File

@ -15,6 +15,7 @@ public class Tooth.Widgets.Attachment.Item : Adw.Bin {
protected Overlay overlay;
protected Button button;
protected Button alt_btn;
protected Label badge;
private void copy_url () {
@ -86,16 +87,30 @@ public class Tooth.Widgets.Attachment.Item : Adw.Bin {
gesture_click_controller.pressed.connect(on_secondary_click);
gesture_lp_controller.pressed.connect(on_secondary_click);
badge = new Label ("") {
var badge_box = new Gtk.Box(Orientation.HORIZONTAL, 1) {
valign = Align.END,
halign = Align.START
};
badge.add_css_class ("osd");
badge = new Label ("");
badge.add_css_class ("heading");
alt_btn = new Button.with_label("ALT");
alt_btn.add_css_class ("heading");
alt_btn.add_css_class ("flat");
alt_btn.clicked.connect(() => {
if (entity != null && entity.description != null)
create_alt_text_window(entity.description).show();
});
badge_box.append(badge);
badge_box.append(alt_btn);
badge_box.add_css_class ("linked");
badge_box.add_css_class ("ttl-status-badge");
overlay = new Overlay ();
overlay.child = button;
overlay.add_overlay (badge);
overlay.add_overlay (badge_box);
child = overlay;
child.add_css_class ("attachment");
@ -105,6 +120,38 @@ public class Tooth.Widgets.Attachment.Item : Adw.Bin {
context_menu.unparent ();
}
protected Adw.Window create_alt_text_window (string alt_text) {
var clamp = new Adw.Clamp () {
child = new Label(alt_text) {
selectable = true,
wrap = true
},
tightening_threshold = 100,
valign = Align.START
};
var scrolledwindow = new ScrolledWindow() {
child = clamp,
vexpand = true,
hexpand = true
};
var box = new Gtk.Box(Orientation.VERTICAL, 6);
var headerbar = new Adw.HeaderBar();
box.append(headerbar);
box.append(scrolledwindow);
return new Adw.Window() {
modal = true,
title = @"Alternative text for $(badge.label)",
transient_for = app.main_window,
content = box,
default_width = 400,
default_height = 300
};
}
protected void create_context_menu() {
var menu_model = new GLib.Menu ();
menu_model.append (_("Open in Browser"), "attachment.open-in-browser");
@ -116,7 +163,7 @@ public class Tooth.Widgets.Attachment.Item : Adw.Bin {
}
protected virtual void on_rebind () {
button.tooltip_text = entity == null ? null : entity.description;
alt_btn.visible = entity != null && entity.description != null && entity.description != "";
badge.label = entity == null ? "" : entity.kind.up();
}