Merge lp:~codygarver/granite/fix-1293284 into lp:~elementary-pantheon/granite/granite

Proposed by Cody Garver
Status: Merged
Approved by: Danielle Foré
Approved revision: 717
Merged at revision: 729
Proposed branch: lp:~codygarver/granite/fix-1293284
Merge into: lp:~elementary-pantheon/granite/granite
Diff against target: 23 lines (+2/-0)
2 files modified
lib/Widgets/DecoratedWindow.vala (+1/-0)
lib/Widgets/LightWindow.vala (+1/-0)
To merge this branch: bzr merge lp:~codygarver/granite/fix-1293284
Reviewer Review Type Date Requested Status
Sergey "Shnatsel" Davidoff (community) Abstain
Victor Martinez (community) Approve
Review via email: mp+214877@code.launchpad.net

Commit message

Deprecate DecoratedWindow and LightWindow to fix bug #1293284

To post a comment you must log in.
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) wrote :

Corentin already complained about something we deprecated "because it's implemented upstream" while the upstream implementation was not sufficient for our needs: https://plus.google.com/u/0/107345637861329011169/posts/5iVNoV3BouZ

So I'd rather not deprecate anything until half the apps using this widget actually migrate to Gtk.Dialog.

review: Needs Information
Revision history for this message
Victor Martinez (victored) wrote :

Hmm, DecoratedWindow and LightWindow don't do anything that cannot be accomplished using Gtk.Dialog + client-side decorations.

The only thing we'll miss is the overlay close button in the top-left corner.

review: Approve
Revision history for this message
Sergey "Shnatsel" Davidoff (shnatsel) :
review: Abstain

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-11-16 20:24:25 +0000
3+++ lib/Widgets/DecoratedWindow.vala 2014-04-09 05:29:33 +0000
4@@ -24,6 +24,7 @@
5 /**
6 * This class is a standard decorated window.
7 */
8+ [Deprecated (replacement = "Gtk.Dialog", since = "0.3")]
9 public class DecoratedWindow : CompositedWindow {
10
11 const string DECORATED_WINDOW_FALLBACK_STYLESHEET = """
12
13=== modified file 'lib/Widgets/LightWindow.vala'
14--- lib/Widgets/LightWindow.vala 2014-04-05 11:07:27 +0000
15+++ lib/Widgets/LightWindow.vala 2014-04-09 05:29:33 +0000
16@@ -26,6 +26,7 @@
17 *
18 * {{../../doc/images/LightWindow.png}}
19 */
20+ [Deprecated (replacement="Gtk.Dialog", since = "0.3")]
21 public class LightWindow : DecoratedWindow {
22
23 /**

Subscribers

People subscribed via source and target branches