Merge lp:~julien-spautz/granite/1134672 into lp:~elementary-pantheon/granite/granite

Proposed by Julien Spautz
Status: Merged
Approved by: David Gomes
Approved revision: 540
Merged at revision: 540
Proposed branch: lp:~julien-spautz/granite/1134672
Merge into: lp:~elementary-pantheon/granite/granite
Diff against target: 39 lines (+10/-8)
2 files modified
lib/Widgets/DecoratedWindow.vala (+9/-0)
lib/Widgets/LightWindow.vala (+1/-8)
To merge this branch: bzr merge lp:~julien-spautz/granite/1134672
Reviewer Review Type Date Requested Status
David Gomes (community) Approve
Review via email: mp+153880@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
David Gomes (davidgomes) :
review: Approve
Revision history for this message
David Gomes (davidgomes) wrote :

Good code, fixed the bug perfectly.

Good job!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/Widgets/DecoratedWindow.vala'
2--- lib/Widgets/DecoratedWindow.vala 2013-01-14 20:26:07 +0000
3+++ lib/Widgets/DecoratedWindow.vala 2013-03-18 17:30:37 +0000
4@@ -235,6 +235,15 @@
5
6 return on_close_button;
7 }
8+
9+ public override bool key_press_event (Gdk.EventKey event) {
10+ if (event.keyval == Gdk.Key.Escape) {
11+ this.delete_event (((Gdk.Event*) (&event))->any);
12+ return true;
13+ }
14+
15+ return base.key_press_event (event);
16+ }
17
18 private bool coords_over_close_button (double x, double y) {
19 return this.deletable &&
20
21=== modified file 'lib/Widgets/LightWindow.vala'
22--- lib/Widgets/LightWindow.vala 2013-01-01 23:46:39 +0000
23+++ lib/Widgets/LightWindow.vala 2013-03-18 17:30:37 +0000
24@@ -35,14 +35,7 @@
25 public LightWindow (string title = "") {
26 base (title, StyleClass.CONTENT_VIEW_WINDOW, StyleClass.CONTENT_VIEW);
27 }
28-
29- public override bool key_press_event (Gdk.EventKey event) {
30- if (event.keyval == Gdk.Key.Escape) {
31- this.delete_event (((Gdk.Event*) (&event))->any);
32- return true;
33- }
34- return base.key_press_event (event);
35- }
36+
37 }
38
39 }

Subscribers

People subscribed via source and target branches