Matthias Clasen
b3b83545c4
Don't return prematurely
...
This code was clearly meant to return _after_ the loop.
Don't put the return inside the loop body, then.
2012-11-29 23:10:35 -05:00
Matthias Clasen
c5c7323942
GtkAboutDialog: Initialize a variable
...
Coverity complains about it being uninitialized, otherwise.
2012-11-29 23:10:34 -05:00
Matthias Clasen
52af9aede6
GtkNotebook: Clarify a switch statement
...
We were intentionally falling through here, this commit
just adds comments to make that explicit.
2012-11-29 23:10:34 -05:00
Matthias Clasen
4557e4d7a5
GtkCellAccessible: Add missing break statements
...
This switch was entirely breakless. Amazing accessibility code,
found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
9ba32fec6e
GtkExpander: Clarify a nested switch
...
This code confused Coverity into thinking we were falling through,
when we were not. Add a few explicit breaks to clear that up.
2012-11-29 23:10:34 -05:00
Matthias Clasen
6400eeba35
Add a forgotten break in render_border
...
The GTK_BORDER_STYLE_DOUBLE case was clearly not meant to
fall through to the subsequent cases, yet it did.
Found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
a7ed598d37
GtkIconViewAccessible: Fix a copy-paste error
...
Don't access vadjustment after checking hadjustment for
not being NULL. Found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
3519226c5a
GtkActionHelper: Fix a copy-paste error
...
The code clearly meant to reset action->active here.
This was found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
bf26d46cb2
gtk_css_value_bg_size_equal: actually compare contain values
...
Due to a copy-paste error, the code failed to compare the
contain members of value1 and value2.
This was found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
ce6f43a522
Don't check the same crossing mode twice
...
This was broken since commit b2aaa94
in 2008. Its commit message
clearly states that the intention was to check for GTK_GRAB,
GTK_UNGRAB and STATE_CHANGED. Lets do that, then.
This was found by Coverity.
2012-11-29 23:10:34 -05:00
Matthias Clasen
5230cfe805
GtkLabel: Fix a pointless expression
...
Clearly, what was meant here was that we create a new attribute
list if either of the input attribute lists are non-NULL.
This was found by Coverity.
2012-11-29 23:10:34 -05:00
Timothy Arceri
50acce4074
In the file chooser, use a proper apostrophe rather than a prime mark (tick mark)
...
https://bugzilla.gnome.org/show_bug.cgi?id=689012
2012-11-29 11:34:29 -06:00
Benjamin Otte
4d0ca91c72
menuitem: Remove unused variables
2012-11-28 21:15:05 +01:00
Marcus Kraßmann
7f3595a6be
cssparser: Don't detect /*/ as a comment
2012-11-28 21:11:48 +01:00
Cosimo Cecchi
ea415cd350
menubar: deprecate internal-padding style property
...
https://bugzilla.gnome.org/show_bug.cgi?id=689168
2012-11-28 10:09:16 -05:00
Cosimo Cecchi
1a79ee4d3e
menubar: add support for CSS padding
...
In preparation for the removal of the internal-padding style property.
https://bugzilla.gnome.org/show_bug.cgi?id=689168
2012-11-28 10:09:16 -05:00
Cosimo Cecchi
01dc23cdec
menu: deprecate horizontal-padding and vertical-padding
...
https://bugzilla.gnome.org/show_bug.cgi?id=689168
2012-11-28 10:09:16 -05:00
Cosimo Cecchi
a69619edbc
menuitem: deprecate horizontal-padding style property
...
https://bugzilla.gnome.org/show_bug.cgi?id=689168
2012-11-28 10:09:16 -05:00
David King
941851fc86
docs: Add gtk-doc comment for GtkTreeRowReference
...
https://bugzilla.gnome.org/show_bug.cgi?id=93381
2012-11-26 14:12:14 +00:00
Philip Withnall
ceb866dfe6
Bug 595615 — Use proper ellipses
...
Use ‘…’ instead of ‘...’ in translatable strings.
Closes: https://bugzilla.gnome.org/show_bug.cgi?id=595615
2012-11-26 08:59:27 +00:00
Benjamin Otte
239c631ad3
treeview: Delay computing fixed height
...
In the setter, we only set fixed height mode and queue a revalidation of
the row heights.
https://bugzilla.gnome.org/show_bug.cgi?id=687816
2012-11-26 00:09:35 +01:00
Emmanuele Bassi
4498ff314e
sizegroup: Use g_hash_table_add()
...
This is the function to use when treating hash tables as a set.
2012-11-25 22:58:19 +01:00
Benjamin Otte
87b351e6e6
styleproperty: Use _gtk_css_initial_value_get()
...
See previous commit(s).
2012-11-25 22:29:14 +01:00
Benjamin Otte
1d805139db
cssvalue: Use _gtk_css_initial_value_get() for the inherit value
...
See previous commit(s).
2012-11-25 22:29:14 +01:00
Benjamin Otte
f204473ee0
cssvalue: Add _gtk_css_initial_value_get()
...
... so we don't bump a refcount whenever we get the initial singleton.
We want to use this function instead of
_gtk_css_style_property_get_initial_value() everywhere where we compute
values, because some initial values may depend on settings soon.
2012-11-25 15:31:59 +01:00
Benjamin Otte
2ab65ef771
izegroup: Improve performance
...
Keep a list of all groups and avoid groups we already handled. Speeds up
GtkToolpalette a lot.
2012-11-25 13:32:21 +01:00
Benjamin Otte
da7a4089fe
widget: Remove an unneeded queue_resize()
...
Resizes are queued via
gtk_widget_propagate_state()
=> gtk_style_context_set_state()
=> gtk_style_context_queue_invalidate()
=> gtk_style_context_validate()
=> _gtk_widget_style_context_invalidated()
so there's no need to queue an extra one.
2012-11-25 04:46:36 +01:00
Benjamin Otte
41486895d5
deprecations: Move files into deprecated/ dir
2012-11-25 04:16:43 +01:00
Benjamin Otte
0862afaded
gtkgradient: Deprecate
2012-11-25 03:47:51 +01:00
Benjamin Otte
fb643c1c12
symboliccolor: Deprecate
...
Symbolic colors are an implementation detail of the CSS engine and have
been superceded by GtkCssColorValue. We don't want them clobbering the
public API. In particular because the only use I could find in the
public API is people using it to shade colors.
2012-11-25 03:43:28 +01:00
Benjamin Otte
df25349d63
themingengine: Do shading with GtkHSLA
...
... instead of with symbolic colors.
2012-11-25 03:34:02 +01:00
Benjamin Otte
9aac4dffc5
hsla: Add _gtk_hsla_shade()
...
We use it in multiple places, so better split it out.
2012-11-25 03:25:15 +01:00
Benjamin Otte
4f9a8e6b3d
symboliccolor: Remove extra includes
...
That header was included in way too many places.
2012-11-25 02:59:15 +01:00
Benjamin Otte
42dc0ea0fd
styleprovider: Change function prototype
...
Make _gtk_style_provider_private_get_color() return a GtkCssValue (a
GtkCssColorValue to be exact) instead of GtkSymbolicColor.
With this, the symbolic color usage inside GTK is minimized.
2012-11-25 02:45:10 +01:00
Benjamin Otte
08ac1504d2
symboliccolor: Implement using GtkCssValue
2012-11-25 02:41:17 +01:00
Benjamin Otte
b8e58015eb
symboliccolor: symbolic color is no longer a CssValue
...
This is not needed anymore, because we have GtkCssColorValue for that
now.
2012-11-25 01:51:18 +01:00
Benjamin Otte
94b80cc774
symboliccolor: Change prototype of function
...
Makes it easier to use the function in the places where it's still used.
2012-11-24 21:56:47 +01:00
Benjamin Otte
5ca35bd5dd
settings: Remove support for color schemes
...
Color schemes were unused and their interaction with CSS3 themes was
undefined. So we decided to remove support for them.
This commit does that.
2012-11-24 21:27:33 +01:00
Benjamin Otte
932794e105
styleprovider: Remove (broken) implementations of get_style
...
The implementations were broken and get_style() is deprecated and
documented to return NULL now.
2012-11-24 21:12:00 +01:00
Benjamin Otte
324ff12553
stylecascade: Remove get_icon_factory() implementation
...
It was redundant.
2012-11-24 21:12:00 +01:00
Benjamin Otte
45b4fb02b1
styleprovider: Deprecate two non-working functions
...
Both of them weren't used inside GTK. And apparently they weren't used
outside of GTK either, as alex recently mentioned them being severly
broken.
2012-11-24 21:12:00 +01:00
David King
fbdb81bbdf
docs: gtk_file_chooser_get_filename() returns absolute paths
...
The documentation for gtk_file_chooser_get_filenames() states that the
returned filenames are absolute paths, and uses g_file_get_path() to
construct the filename. The same function is used to construct the
filename in gtk_file_chooser_get_filename(), so it should also return
absolute paths.
https://bugzilla.gnome.org/show_bug.cgi?id=371034
2012-11-24 19:58:10 +00:00
David King
73c436f43e
docs: Fix typo in gtk_tree_view_set_tooltip_column comment
...
https://bugzilla.gnome.org/show_bug.cgi?id=688884
2012-11-22 22:25:51 +00:00
David King
c627b22126
docs: Improve GtkTreeModel iteration pattern
...
Iterating over the model in this way means that use of continue is less
error-prone, as the increment is part of the loop construct.
https://bugzilla.gnome.org/show_bug.cgi?id=548793
2012-11-22 21:36:46 +00:00
David King
e677cee28f
docs: Correct GtkWidget::scroll-event mask documentation
...
Since GTK+ 3.3.18, GDK_SCROLL_MASK has been used as the mask for scroll
events. Update the documentation to reflect this.
https://bugzilla.gnome.org/show_bug.cgi?id=677339
2012-11-22 21:36:40 +00:00
Martin Pitt
2927383b5f
aboutdialog: Fix add_credit_section() annotations
...
The people argument is an array.
https://mail.gnome.org/archives/python-hackers-list/2012-November/msg00010.html
2012-11-22 14:51:52 +01:00
Michael Natterer
2d9454363f
gtk: don't let insensitive children eat scroll events when bubbling down
...
When event capturing is enabled, stop propagating scroll events
at insensitive widgets, but don't handle them (don't return TRUE),
so they can bubble up again and reach their handling widgets.
2012-11-22 13:53:33 +01:00
Cosimo Cecchi
ddceddaa84
tree-view: don't use gtk_style_context_set_background()
...
Render a background with gtk_render_background() in draw() instead.
Note that we still use gtk_style_context_set_background() for the header
window.
https://bugzilla.gnome.org/show_bug.cgi?id=688744
2012-11-20 13:41:39 -05:00
Cosimo Cecchi
da09447914
iconview: don't use gtk_style_context_set_background()
...
Render a background with gtk_render_background() during draw instead.
https://bugzilla.gnome.org/show_bug.cgi?id=688744
2012-11-20 13:41:39 -05:00
Benjamin Otte
e5948f5713
cssvalue: Fix copy/paste error
...
Introduced in e3f407a7
.
2012-11-20 12:11:07 +01:00