Merge lp:~vvort/slingshot/tooltip_fix_2 into lp:~elementary-pantheon/slingshot/trunk

Proposed by Vort
Status: Merged
Approved by: Danielle Foré
Approved revision: 644
Merged at revision: 644
Proposed branch: lp:~vvort/slingshot/tooltip_fix_2
Merge into: lp:~elementary-pantheon/slingshot/trunk
Diff against target: 33 lines (+6/-1)
2 files modified
src/Widgets/Grid.vala (+4/-1)
src/Widgets/SearchItem.vala (+2/-0)
To merge this branch: bzr merge lp:~vvort/slingshot/tooltip_fix_2
Reviewer Review Type Date Requested Status
elementary Pantheon team Pending
Review via email: mp+292575@code.launchpad.net

Commit message

Show tooltips on hover

Description of the change

Fix for lp:1444396

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Widgets/Grid.vala'
--- src/Widgets/Grid.vala 2015-12-23 18:03:42 +0000
+++ src/Widgets/Grid.vala 2016-04-21 18:49:23 +0000
@@ -83,12 +83,15 @@
83 stack.add_titled (current_grid, page.number.to_string (), page.number.to_string ());83 stack.add_titled (current_grid, page.number.to_string (), page.number.to_string ());
8484
85 // Fake grids in case there are not enough apps to fill the grid85 // Fake grids in case there are not enough apps to fill the grid
86 current_grid.attach (new Gtk.Grid (), 0, 0, (int)page.columns, (int)page.rows);86 for (var row = 0; row < page.rows; row++)
87 for (var column = 0; column < page.columns; column++)
88 current_grid.attach (new Gtk.Grid (), column, row, 1, 1);
87 }89 }
8890
89 public void append (Gtk.Widget widget) {91 public void append (Gtk.Widget widget) {
90 update_position ();92 update_position ();
9193
94 current_grid.get_child_at ((int)current_col, (int)current_row).destroy ();
92 current_grid.attach (widget, (int)current_col, (int)current_row, 1, 1);95 current_grid.attach (widget, (int)current_col, (int)current_row, 1, 1);
93 current_col++;96 current_col++;
94 current_grid.show ();97 current_grid.show ();
9598
=== modified file 'src/Widgets/SearchItem.vala'
--- src/Widgets/SearchItem.vala 2016-02-20 17:11:51 +0000
+++ src/Widgets/SearchItem.vala 2016-04-21 18:49:23 +0000
@@ -64,6 +64,8 @@
64 icon.gicon = app.icon;64 icon.gicon = app.icon;
65 icon.pixel_size = ICON_SIZE;65 icon.pixel_size = ICON_SIZE;
6666
67 tooltip_text = app.description;
68
67 // load a favicon if we're an internet page69 // load a favicon if we're an internet page
68 var uri_match = app.match as Synapse.UriMatch;70 var uri_match = app.match as Synapse.UriMatch;
69 if (uri_match != null && uri_match.uri.has_prefix ("http")) {71 if (uri_match != null && uri_match.uri.has_prefix ("http")) {

Subscribers

People subscribed via source and target branches