Merge lp:~azzar1/overlay-scrollbar/deprecated-api into lp:overlay-scrollbar
- deprecated-api
- Merge into trunk
Proposed by
Andrea Azzarone
Status: | Merged |
---|---|
Approved by: | Andrea Cimitan |
Approved revision: | 242 |
Merged at revision: | 242 |
Proposed branch: | lp:~azzar1/overlay-scrollbar/deprecated-api |
Merge into: | lp:overlay-scrollbar |
Diff against target: |
266 lines (+67/-45) 3 files modified
os/os-scrollbar.c (+56/-40) os/os-thumb.c (+10/-3) os/os-utils.h (+1/-2) |
To merge this branch: | bzr merge lp:~azzar1/overlay-scrollbar/deprecated-api |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Andrea Cimitan | Approve | ||
Review via email:
|
Commit message
Description of the change
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
1 | === modified file 'os/os-scrollbar.c' |
2 | --- os/os-scrollbar.c 2011-05-18 18:35:34 +0000 |
3 | +++ os/os-scrollbar.c 2011-05-19 14:54:29 +0000 |
4 | @@ -134,9 +134,10 @@ |
5 | * total_adjustment_range) times the trough height in pixels |
6 | */ |
7 | |
8 | - if (priv->adjustment->upper - priv->adjustment->lower != 0) |
9 | - height = ((bottom - top) * (priv->adjustment->page_size / |
10 | - (priv->adjustment->upper - priv->adjustment->lower))); |
11 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) != 0) |
12 | + height = ((bottom - top) * (gtk_adjustment_get_page_size (priv->adjustment) / |
13 | + (gtk_adjustment_get_upper (priv->adjustment) - |
14 | + gtk_adjustment_get_lower (priv->adjustment)))); |
15 | else |
16 | height = gtk_range_get_min_slider_size (GTK_RANGE (scrollbar)); |
17 | |
18 | @@ -146,9 +147,12 @@ |
19 | |
20 | y = top; |
21 | |
22 | - if (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size != 0) |
23 | - y += (bottom - top - height) * ((adjustment_value - priv->adjustment->lower) / |
24 | - (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size)); |
25 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) - |
26 | + gtk_adjustment_get_page_size (priv->adjustment) != 0) |
27 | + y += (bottom - top - height) * ((adjustment_value - gtk_adjustment_get_lower (priv->adjustment)) / |
28 | + (gtk_adjustment_get_upper (priv->adjustment) - |
29 | + gtk_adjustment_get_lower (priv->adjustment) - |
30 | + gtk_adjustment_get_page_size (priv->adjustment))); |
31 | |
32 | y = CLAMP (y, top, bottom); |
33 | |
34 | @@ -166,9 +170,10 @@ |
35 | * total_adjustment_range) times the trough width in pixels |
36 | */ |
37 | |
38 | - if (priv->adjustment->upper - priv->adjustment->lower != 0) |
39 | - width = ((right - left) * (priv->adjustment->page_size / |
40 | - (priv->adjustment->upper - priv->adjustment->lower))); |
41 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) != 0) |
42 | + width = ((right - left) * (gtk_adjustment_get_page_size (priv->adjustment) / |
43 | + (gtk_adjustment_get_upper (priv->adjustment) - |
44 | + gtk_adjustment_get_lower (priv->adjustment)))); |
45 | else |
46 | width = gtk_range_get_min_slider_size (GTK_RANGE (scrollbar)); |
47 | |
48 | @@ -178,9 +183,12 @@ |
49 | |
50 | x = left; |
51 | |
52 | - if (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size != 0) |
53 | - x += (right - left - width) * ((adjustment_value - priv->adjustment->lower) / |
54 | - (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size)); |
55 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) - |
56 | + gtk_adjustment_get_page_size (priv->adjustment) != 0) |
57 | + x += (right - left - width) * ((adjustment_value - gtk_adjustment_get_lower (priv->adjustment)) / |
58 | + (gtk_adjustment_get_upper (priv->adjustment) - |
59 | + gtk_adjustment_get_lower (priv->adjustment) - |
60 | + gtk_adjustment_get_page_size (priv->adjustment))); |
61 | |
62 | x = CLAMP (x, left, right); |
63 | |
64 | @@ -211,9 +219,12 @@ |
65 | |
66 | y = top; |
67 | |
68 | - if (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size != 0) |
69 | - y += (bottom - top - height) * ((adjustment_value - priv->adjustment->lower) / |
70 | - (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size)); |
71 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) - |
72 | + gtk_adjustment_get_page_size (priv->adjustment) != 0) |
73 | + y += (bottom - top - height) * ((adjustment_value - gtk_adjustment_get_lower (priv->adjustment)) / |
74 | + (gtk_adjustment_get_upper (priv->adjustment) - |
75 | + gtk_adjustment_get_lower (priv->adjustment) - |
76 | + gtk_adjustment_get_page_size (priv->adjustment))); |
77 | |
78 | y = CLAMP (y, top, bottom); |
79 | |
80 | @@ -233,9 +244,12 @@ |
81 | |
82 | x = left; |
83 | |
84 | - if (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size != 0) |
85 | - x += (right - left - width) * ((adjustment_value - priv->adjustment->lower) / |
86 | - (priv->adjustment->upper - priv->adjustment->lower - priv->adjustment->page_size)); |
87 | + if (gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_lower (priv->adjustment) - |
88 | + gtk_adjustment_get_page_size (priv->adjustment) != 0) |
89 | + x += (right - left - width) * ((adjustment_value - gtk_adjustment_get_lower (priv->adjustment)) / |
90 | + (gtk_adjustment_get_upper (priv->adjustment) - |
91 | + gtk_adjustment_get_lower (priv->adjustment) - |
92 | + gtk_adjustment_get_page_size (priv->adjustment))); |
93 | |
94 | x = CLAMP (x, left, right); |
95 | |
96 | @@ -571,7 +585,8 @@ |
97 | * hidden a pager that is meant to be hidden/shown. |
98 | * I don't want to see pagers reappearing because |
99 | * of a change in the adjustment of an invisible pager or viceversa. */ |
100 | - if ((adjustment->upper - adjustment->lower) > adjustment->page_size) |
101 | + if ((gtk_adjustment_get_upper (adjustment) - gtk_adjustment_get_lower (adjustment)) > |
102 | + gtk_adjustment_get_page_size (adjustment)) |
103 | { |
104 | priv->fullsize = FALSE; |
105 | if (priv->proximity != FALSE) |
106 | @@ -588,8 +603,8 @@ |
107 | } |
108 | } |
109 | |
110 | - calc_layout_pager (scrollbar, adjustment->value); |
111 | - calc_layout_slider (scrollbar, adjustment->value); |
112 | + calc_layout_pager (scrollbar, gtk_adjustment_get_value (adjustment)); |
113 | + calc_layout_slider (scrollbar, gtk_adjustment_get_value (adjustment)); |
114 | |
115 | if (!priv->motion_notify_event && !priv->enter_notify_event) |
116 | gtk_widget_hide (GTK_WIDGET (priv->thumb)); |
117 | @@ -607,8 +622,8 @@ |
118 | scrollbar = OS_SCROLLBAR (user_data); |
119 | priv = scrollbar->priv; |
120 | |
121 | - calc_layout_pager (scrollbar, adjustment->value); |
122 | - calc_layout_slider (scrollbar, adjustment->value); |
123 | + calc_layout_pager (scrollbar, gtk_adjustment_get_value (adjustment)); |
124 | + calc_layout_slider (scrollbar, gtk_adjustment_get_value (adjustment)); |
125 | |
126 | if (!priv->motion_notify_event && !priv->enter_notify_event) |
127 | gtk_widget_hide (GTK_WIDGET (priv->thumb)); |
128 | @@ -1134,11 +1149,12 @@ |
129 | frac = (MAX (0, coord - trough_start) / |
130 | (gdouble) (trough_length - slider_length)); |
131 | |
132 | - value = priv->adjustment->lower + frac * (priv->adjustment->upper - |
133 | - priv->adjustment->lower - |
134 | - priv->adjustment->page_size); |
135 | + value = gtk_adjustment_get_lower (priv->adjustment) + frac * (gtk_adjustment_get_upper (priv->adjustment) - |
136 | + gtk_adjustment_get_lower (priv->adjustment) - |
137 | + gtk_adjustment_get_page_size (priv->adjustment)); |
138 | |
139 | - value = CLAMP (value, priv->adjustment->lower, priv->adjustment->upper - priv->adjustment->page_size); |
140 | + value = CLAMP (value, gtk_adjustment_get_lower (priv->adjustment), |
141 | + gtk_adjustment_get_upper (priv->adjustment) - gtk_adjustment_get_page_size (priv->adjustment)); |
142 | |
143 | return value; |
144 | } |
145 | @@ -1215,7 +1231,7 @@ |
146 | priv->win_y + priv->overlay.y, |
147 | priv->win_y + priv->overlay.y + priv->overlay.height - priv->slider.height); |
148 | |
149 | - if (priv->adjustment->value == 0) |
150 | + if (gtk_adjustment_get_value (priv->adjustment) == 0) |
151 | { |
152 | priv->slide_initial_slider_position = 0; |
153 | priv->slide_initial_coordinate = MAX (event->y_root, priv->win_y + priv->pointer_y); |
154 | @@ -1241,7 +1257,7 @@ |
155 | priv->win_x + priv->overlay.x + priv->overlay.width - priv->slider.width); |
156 | y = priv->win_y; |
157 | |
158 | - if (priv->adjustment->value == 0) |
159 | + if (gtk_adjustment_get_value (priv->adjustment) == 0) |
160 | { |
161 | priv->slide_initial_slider_position = 0; |
162 | priv->slide_initial_coordinate = MAX (event->x_root, priv->win_x + priv->pointer_x); |
163 | @@ -1275,7 +1291,7 @@ |
164 | |
165 | priv = scrollbar->priv; |
166 | |
167 | - delta = pow (priv->adjustment->page_size, 2.0 / 3.0); |
168 | + delta = pow (gtk_adjustment_get_page_size (priv->adjustment), 2.0 / 3.0); |
169 | |
170 | if (direction == GDK_SCROLL_UP || |
171 | direction == GDK_SCROLL_LEFT) |
172 | @@ -1301,10 +1317,10 @@ |
173 | delta = get_wheel_delta (scrollbar, event->direction); |
174 | |
175 | gtk_adjustment_set_value (priv->adjustment, |
176 | - CLAMP (priv->adjustment->value + delta, |
177 | - priv->adjustment->lower, |
178 | - (priv->adjustment->upper - |
179 | - priv->adjustment->page_size))); |
180 | + CLAMP (gtk_adjustment_get_value (priv->adjustment) + delta, |
181 | + gtk_adjustment_get_lower (priv->adjustment), |
182 | + (gtk_adjustment_get_upper (priv->adjustment) - |
183 | + gtk_adjustment_get_page_size (priv->adjustment)))); |
184 | |
185 | return FALSE; |
186 | } |
187 | @@ -1410,8 +1426,8 @@ |
188 | |
189 | priv->lock_position = FALSE; |
190 | |
191 | - calc_layout_pager (scrollbar, priv->adjustment->value); |
192 | - calc_layout_slider (scrollbar, priv->adjustment->value); |
193 | + calc_layout_pager (scrollbar, gtk_adjustment_get_value (priv->adjustment)); |
194 | + calc_layout_slider (scrollbar, gtk_adjustment_get_value (priv->adjustment)); |
195 | |
196 | store_toplevel_position (scrollbar); |
197 | |
198 | @@ -1900,7 +1916,7 @@ |
199 | g_signal_connect (G_OBJECT (gtk_widget_get_toplevel (widget)), "configure-event", |
200 | G_CALLBACK (toplevel_configure_event_cb), scrollbar); |
201 | |
202 | - calc_layout_pager (scrollbar, priv->adjustment->value); |
203 | + calc_layout_pager (scrollbar, gtk_adjustment_get_value (priv->adjustment)); |
204 | |
205 | os_pager_set_parent (OS_PAGER (priv->pager), widget); |
206 | |
207 | @@ -1963,8 +1979,8 @@ |
208 | |
209 | if (priv->adjustment != NULL) |
210 | { |
211 | - calc_layout_pager (scrollbar, priv->adjustment->value); |
212 | - calc_layout_slider (scrollbar, priv->adjustment->value); |
213 | + calc_layout_pager (scrollbar, gtk_adjustment_get_value (priv->adjustment)); |
214 | + calc_layout_slider (scrollbar, gtk_adjustment_get_value (priv->adjustment)); |
215 | } |
216 | |
217 | os_pager_size_allocate (OS_PAGER (priv->pager), rect); |
218 | @@ -1974,7 +1990,7 @@ |
219 | if (gtk_widget_get_realized (widget)) |
220 | store_toplevel_position (scrollbar); |
221 | |
222 | - widget->allocation = *allocation; |
223 | + gtk_widget_set_allocation (widget, allocation); |
224 | } |
225 | |
226 | static void |
227 | |
228 | === modified file 'os/os-thumb.c' |
229 | --- os/os-thumb.c 2011-05-18 18:25:13 +0000 |
230 | +++ os/os-thumb.c 2011-05-19 14:54:29 +0000 |
231 | @@ -273,12 +273,19 @@ |
232 | if (gdk_screen_is_composited (gtk_widget_get_screen (widget))) |
233 | { |
234 | GdkVisual *visual; |
235 | + guint32 red_mask; |
236 | + guint32 green_mask; |
237 | + guint32 blue_mask; |
238 | |
239 | visual = gtk_widget_get_visual (widget); |
240 | |
241 | - if (visual->depth == 32 && (visual->red_mask == 0xff0000 && |
242 | - visual->green_mask == 0x00ff00 && |
243 | - visual->blue_mask == 0x0000ff)) |
244 | + gdk_visual_get_red_pixel_details (visual, &red_mask, NULL, NULL); |
245 | + gdk_visual_get_green_pixel_details (visual, &green_mask, NULL, NULL); |
246 | + gdk_visual_get_blue_pixel_details (visual, &blue_mask, NULL, NULL); |
247 | + |
248 | + if (gdk_visual_get_depth (visual) == 32 && (red_mask == 0xff0000 && |
249 | + green_mask == 0x00ff00 && |
250 | + blue_mask == 0x0000ff)) |
251 | priv->can_rgba = TRUE; |
252 | } |
253 | |
254 | |
255 | === modified file 'os/os-utils.h' |
256 | --- os/os-utils.h 2011-03-23 15:42:09 +0000 |
257 | +++ os/os-utils.h 2011-05-19 14:54:29 +0000 |
258 | @@ -28,8 +28,7 @@ |
259 | #define __OS_UTILS_H__ |
260 | |
261 | #include <glib.h> |
262 | -#include <gtk/gtkwidget.h> |
263 | -#include <gtk/gtkenums.h> |
264 | +#include <gtk/gtk.h> |
265 | |
266 | G_BEGIN_DECLS |
267 |
Well done Andrea! many thanks, looks good