[Bug 675501] gtkquartz.h is not in the gtk+-3.5.2.tar.xz archive

Rearrange the gtk_private_h_sources so that they're all added to EXTRA_DIST instead of only the ones included in the current build.

Rename gtkquartz-menu to gtkmodelmenu-quartz

Makes name consistent with other quartz-only modules and makes it clear that this works with the GMenuModel system rather than the older GtkMenu system.

bgo#675421 - Set the wm_role on GtkFileChooserDialog

The window's role is 'GtkFileChooserDialog', so that window managers can match it
for positioning.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

sizerequest: Improve warning message

Actually print out the function we're warning about

iconview: Fix negative natural size request on empty iconview

broadway: Fix vfunc prototype

build: Don't invoke pkg-config directly

Instead, always use PKG_CHECK_MODULES(). That macro actually gets it
right. In particular the erroring out part when you miss xkbcommon or
wayland-client.

iconview: Don't expand items to more than natural size

This ensures that items stay left-aligned instead of slowly expanding into
empty space when widening the iconview. It's also what the iconview did
pre-refactoring.

Note that for cases where natural width != minimum width, the cells
might still expand and shrink back.

tests: Remove unused include

Fixes build

iconview: Redo layouting

iconview: Respect item width in size requestr code

iconview: Reduce complexity of sizing code

Always assume max-columns and min-rows. The old approach was kinda
insane.

As an example, try to write an algorithm that optimizes the minimum size
for infinite (take a reasonably large number like 2520) word-wrapped
Monospace text cells containing the text "XXXXX XXX XXX XXXXX" (keep in
mind that this is the easy problem, because it's assuming equal cell
renderers). There's 4 ways to reasonably lay out this text:
19 glyphs (19x1):
  XXXXX XXX XXX XXXXX
18 glyphs (9x2):
  XXXXX XXX
  XXX XXXXX
21 glyphs (7x3):
  XXXXX
  XXX XXX
  XXXXX
20 glyphs (5x4):
  XXXXX
  XXX
  XXX
  XXXXX
The best thing to do usually is using the 9x2 approach, but that's
neither the one using the natural nor the one using the minimum size.

As a side note, this does not include spacing and padding, which might
also influence the decision. Nor does it include height-for-width
considerations. Look at this table (numbers given in glyphs, not pixels,
as for pixel-sizes it gets even more interesting):
  given  best solution
  width  columns  sizing  glyphs per cell
   6      1       6x4           20
   7      1       7x3           21
   8      1       7x3           24
   9      1       9x2           18
  10      1/2     9x2/5x4       20
  11      1/2     9x2/5x4       22
  12      1/2     9x2/5x4       24
  13      1/2     9x2/5x4       26
  14      2       7x3           21
  15      3       5x4           20
  16      3       5x4           21.3
  17      3       5x4           22.7
  18      2       9x2           18
  19      1/2    19x1/8x2       19
  20      1/2/4  19x1/8x2/5x4   20
  21      1-4     any           21
  22      1-4     any           22
  23      1-4     any           23
  24      1-4     any           24
  25      5       5x4           20
  26      5       5x4           20.8
  27      3       9x2           18
  28      3       9x2           18.7
  29      3       9x2           19.3
  30      3/6     9x2/5x4       20

Now of course, nobody wants the number of columns to randomly change in
inexplicable ways while they enlarge or shrink an iconview, so we not
only have to optimize for smallest or other size measurements, but we
also have to optimize for "most pleasing to the eye".

And last but not least, I'd like to once again remind you - if you kept
up until now - that this discussion was for identically-sized cells
only.

tests: Make testcellarea use an iconview

iconview: Redo size requests

Instead of just returning the last allocated numbers, we now compute the
proper sizes from scratch. This is a bit less trivial, but it results in
proper height-for-width handling.

iconview: Don't cache iters

This is a huge quest to remove all caching from GtkIconview to simplify
the code. As it turns out, iconview performance is a joke, so the caches
are kinda unnecessary.

If we need caching, we can add it in a useful way later.

iconview: Remove needless requirement

If we use
  &item->cell_area
instead of
  (GdkRectangle *) item
there is no need anymore to keep the cell_area as the first member of
the the item. And we cget compile-time checks for changes to the item
struct.

gtk_entry_[get|set]_inner_border(): Clarify the deprecation docs.

[socket]: fix gtk-doc warnings.

gdk: do not send leave event on touchpad button release

This is along the changes introduced in
bd55519f7e6bb5da2fdf6f5e99bd79213059f81b,
A TOUCHPAD device behaves like a mouse"

https://bugzilla.gnome.org/show_bug.cgi?id=674549

Updated Hebrew translation.

Updated Galician translations

themingengine: don't hardcode the slider lines in the theming engine

It's wrong to hardcode the slider lines here; those should be rendered
as an additional background layer using background-image if the theme
specifies so.

https://bugzilla.gnome.org/show_bug.cgi?id=652304

entry-completion: set size request on the toplevel window

Instead of setting it on the child scrolled window. This is needed
because the whole window's allocation must be equal to the one of the
entry (in case the popup-set-width property is TRUE); if we set the size
request on a children of the window, there might be other children with
borders/paddings in between the toplevel and the child we set the size
request too, which will break alignment.

https://bugzilla.gnome.org/show_bug.cgi?id=672947

entry-completion: don't subtract entry borders from popup allocation

The completion popup should be aligned with the GtkEntry allocation, not
less than that.

https://bugzilla.gnome.org/show_bug.cgi?id=672947

Updated Spanish translation

cellrenderertext: Add placeholder-text property

It works similar to placeholder text in a GtkEntry, if the cell renderer
is editable and it's empty, the placeholder text is displayed.
https://bugzilla.gnome.org/show_bug.cgi?id=645514

docs: Improve GtkMessageDialog description

Add mention of default setting of GtkWindow:skip-taskbar-hint.

https://bugzilla.gnome.org/show_bug.cgi?id=587873

docs: Move GtkSelectionData to gtk3-Selections

Put the (albeit empty) documentation for the GtkSelectionData struct
with the gtk_selection_data_*() methods.

https://bugzilla.gnome.org/show_bug.cgi?id=663072

docs: Correct gtk_tree_row_reference_new() description

Remove the text that mentions listening to tree model signals, and
instead say that any changes are automatically propagated.

https://bugzilla.gnome.org/show_bug.cgi?id=478344

Add support for disabling middle click paste

Adds a gtk-enable-primary-paste setting that is backed by
and X setting, and make GtkEntry and GtkTextView respect it.

https://bugzilla.gnome.org/show_bug.cgi?id=665243

printing: Rename GTK_UNIT_PIXEL to GTK_UNIT_NONE

...and document that many functions don't accept "none" as a unit.

https://bugzilla.gnome.org/show_bug.cgi?id=460908

gdk: add bindings-friendly alias for GDK_EVENT_2BUTTON_PRESS

GDK_EVENT_2BUTTON_PRESS and GDK_EVENT_3BUTTON_PRESS can't be used from
some bindings because they'd translate to something syntactically
invalid. Add GDK_EVENT_DOUBLE_BUTTON_PRESS and
GDK_EVENT_TRIPLE_BUTTON_PRESS aliases to work around that.

https://bugzilla.gnome.org/show_bug.cgi?id=671025

window: Remove needless call to ensure_state_flag_backdrop() on map

This call was forcing needless work since gtk_window_map() already
does a gdk_window_show() which initially sets GDK_WINDOW_STATE_FOCUSED
that we then handle regularly on the widget's window state event
handler.

https://bugzilla.gnome.org/show_bug.cgi?id=673237

build: Fix out-of-tree build of widget-factory demo application

https://bugzilla.gnome.org/show_bug.cgi?id=674192

Updated Dutch po-properties translation by Rachid, reviewed by Reinout
van Schouwen.

Fix unclosed comments

colorbutton: don't hardcode a 16px margin around the color sample

Since themes might want different paddings around the color sample
according to where it's being used, don't hardcode a 16px one here.
The theme can specify a padding to get the same effect.

Reported by Christian Persch <chpe@gnome.org>

application: Fix build without x11

application: Add unique IDs for GtkApplicationWindow

This will allow to refer to specific GtkApplicationWindows remotely by ID.

https://bugzilla.gnome.org/show_bug.cgi?id=674409

fontsel: Add a deprecation note in the constructor docs

To match the compiler attribute, otherwise the introspection scanner
will not pick up the deprecation.

Bug 670499-gtk/fallback-c89.c: Add fallback for nearbyint()

This adds a C89 implementation for nearbyint() as it is a function that
is only made available in C99.

Bug 670499-gdk/gdkrgba.c: Include fallback-c89.c

Include (gdk/)fallback-c89.c as isnan() and isinf() is used.

Bug 670499-Add gdk/fallback-c89.c

Add a fallback-c89.c for the gdk/ subdirectory as there is code that uses
functions that are introduced by C99.  This currently adds fallback
implementations for MSVC for isnan() and isinf()

Dist this "new" source file as well

Bug 670499-configure.ac: Check for nearbyint(), isinf() and isnan()

nearbyint(), isinf() and isnan() are C99 functions, so check for them.

Also clean up configure.ac a little bit as the checks for rint() and
round() can be a bit simpler, according to Matthias' suggestions.

Bug 670449-gtk/gtksettings.c: Fix build on Windows

On Windows, gtkwin32themeprivate.h is needed as
_gtk_win32_theme_get_default() is called on that platform to avoid C4013
warnings/errors (aka implicit declaration of ... for GCC folks).

pathbar: fix a copy/paste error from last commit

We need to update the child pointer to the current iterator when going
trough the button list.

stylecontext: Disallow adding/removing GtkSettings style providers

Adding "system" providers like the GtkSettings object shouldn't be
allowed at all, so ensure that it indeed is not.

pathbar: Only invalidate styles in allocate if stuff really changed

I still don't like queueing restyles in size_allocate(), but I can't
come up with a better mechanims for the way the pathbar works today.

Updated Greek translation

debug: Add GTK_DEBUG=no-css-cache

See inline comments for what it does. Its main use is figuring out if
something has been caused by GTK's caching of CSS properties or if it's
a different problem.

build: Set G_ENABLE_DEBUG for minimal --enable-debug=minimum

With that flag set, we get GDK_DEBUG and GTK_DEBUG support. And those
provide useful features we want when debugging.

window: Validate style context before first allocation

When we don't do that, we completely botch sizing popups. Not good.

Fixes remaining failing reftests

reftests: Remove unneeded event boxes

The event boxes break when backgrounds are transparent.

Fixes border-image-repeat.ui reftest.

reftests: Make reset css include new CSS properties

cssshadow: plug a memory leak

Bump version

Bump to 3.5.2

Add some more documentation about GDK_SCROLL_SMOOTH

cssshadow: Default fallback color is transparent

... not the current color. Fixes unresolvable.ui test.

cssmatcher: Handle case of empty widget path

This is tested by the stylecontext test, but doesn't appear in practice.

cssprovider: Actually match the state for style properties

Copy/paste error when refactoring.

Caught by stylecontext test.

versionmacros: min-require the current stable version

.. instead of the previous stable version.

This ensures that if we use GDK_DEPRECATED_IN_3_6, it will actually emit
a warning in GTK 3.5, and not wait with that until GTK 3.7. This is
particularly useful for stuff that gets deprecated right now. This code
should emit warnings right now, so we know what we're doing while
deprecating.

docs: Fix copy/paste typo

gtk: Get rid of deprecation warnings

tests: Ifdef out unused code

The test using those functions was disabled in
b310ba29d0c14cdd566b4af1dd10599735980985

build: Remove now-unused GTK_DISABLE_DEPRECATED undefs

Update reference output for css parser tests

Seems these were not kept up-to-date with the recent
CSS parser changes.

GtkApplication: adjust to GApplication changes

GApplication now makes the session bus and object path available as a
public API on the application instance.  Use that instead of trying to
guess values for ourselves.

This causes this version of Gtk+ to depend on GLib 2.32.2, so bumping
version dependency accordingly.

https://bugzilla.gnome.org/show_bug.cgi?id=671249

GtkApplication: fixes for NULL session bus

We currently have a couple of cases where GtkApplication assumes that
the session bus will be non-NULL causing critical error output or (in
the case of trying to publish menus) an infinite loop.

Three fixes:

 - if the session bus is NULL due to not having registered the
   GtkApplication yet then give a g_critical on the entry point to the
   menu setters instead of going into an infinite loop.  Document this.

 - check for NULL session bus even when calling the menu setters at the
   right time in order to prevent the infinite loop for
   non-programer-error cases (ie: because we had trouble connecting to
   the session bus)

 - check for NULL session bus when publishing the X11 properties on the
   GtkApplicationWindow and skip publishing them if we're not on the bus

https://bugzilla.gnome.org/show_bug.cgi?id=671249

infobar: deprecate broken GtkInfoBar style properties

The way we use these style properties to set regular properties on
containers accessible from the public API is really just broken, and
could lead to undefined values for the spacing and border-width
container properties (since they could be set from public API and then
changed from under in a style_update handler from GTK).

Take this as an occasion to deprecate these style properties, which do
not make a lot of sense anyway, now that GtkInfoBar supports regular CSS
padding and border.

GtkPlug: fix handling of key events for different layouts

GtkPlug directly handles X KeyPress/Release events, instead of using
translation in GDK (which expects XI2 events for XI2). When this
was done, the handling of the group was stubbed out and never replaced.

Export gdk_keymap_x11_group_for_state() and gdk_keymap_x11_is_modifier()
so we can fill out the fields correctly.

https://bugzilla.gnome.org/show_bug.cgi?id=675167

Fix info-bar reftest

InfoBar must take into account the border and padding when requesting
its allocation, since it is then drawing them. Besides, the border and
background should always be drawn, even when the message type is OTHER.

Fix infobar reftest for label background changes

Fix the reftest after previous commit

https://bugzilla.gnome.org/show_bug.cgi?id=670743

reftests: fix background-size reftest

We have to be careful and not set the background on the labels
themselves.

https://bugzilla.gnome.org/show_bug.cgi?id=670743

Paint label background

Make GtkLabel respect background css properties.

https://bugzilla.gnome.org/show_bug.cgi?id=670743

themingbackground: fix copy/paste typo

We need to fetch the area value for the origin, not for the clip.

Disable configuration-dependent keymap tests for now

These need to be made independent of the xkb configuration somehow.
As things are now, they will either fail when run on a naked X
server in make check, or fail when run in my session.

Fix build with --enable-debug

Updates for 3.5.1

Mention XDG Base Dir migration in README for 3.6

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Migrate to XDG config dir for bookmarks

Will read from old location if new location isn't found but will
write it out to the new location.

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Prefer xdg data directory to legacy .icons directory

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Try to load css themes from the XDG config directory first

Will fall back to previous locations.

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Migrate to XDG config dir for custom papers

Will read from old location if new location isn't found but will
only write to the new location.

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Don't load binary modules from the home directory by default

We should rely on the module path environment variables being
set correctly for the architecture rather than load from .gtk-3.0

https://bugzilla.gnome.org/show_bug.cgi?id=646631

Fix the colord support in GtkPrinterCups when the PPD is loaded async

If the PPD is not available when the printer is loaded then the PPD options are
not available and we can't get the correct qualifier to use with colord.

When the PPD becomes available, refresh the profile title to reflect reality.

https://bugzilla.gnome.org/show_bug.cgi?id=674890

GtkGrid: Add a note about overlapping children

https://bugzilla.gnome.org/show_bug.cgi?id=669989

Revert "WIP: make GtkNotebook window-draggable"

This reverts commit b3a66c59ade144c094d4dbc568e33a5ef85c1a7d.

treemodelsort: fix iter_previous so it can go back to the first item

It was getting the previous element, then checking whether it was the
first one, that has to be inverted so it doesn't stop on the second
item.

https://bugzilla.gnome.org/show_bug.cgi?id=674587

WIP: make GtkNotebook window-draggable

https://bugzilla.gnome.org/show_bug.cgi?id=672427

https://bugzilla.gnome.org/show_bug.cgi?id=674587

Add a 'fine adjustment' mode to ranges

Shift-click in the slider now starts a drag in 'fine adjustment'
mode, where we move the slider 10-times slower than the mouse.

This can be very helpful when scrolling through a very long document
or webpage, and moving the scrollbar even a single pixel already
jumps too far in the content.

https://bugzilla.gnome.org/show_bug.cgi?id=563688

Change button bindings for range widgets around

It seems to be general consensus that button 1 should do the jumping,
so we now jump to the clicked position on primary button clicks and
page on secondary button clicks. Touch behaves like primary.

https://bugzilla.gnome.org/show_bug.cgi?id=563688

stylecontext: Add GTK_STYLE_CLASS_DIM_LABEL

Fixes: https://bugzilla.gnome.org/674468

gtk-demo: fix the tool palette demo

The palette was confined to its minimal size, when in horizontal mode.

toolpalette: Configure adjustment properly

https://bugzilla.gnome.org/show_bug.cgi?id=673650

toolpalette: Rename variable

The variable that holds the total size for all widget allocations
shouldn't be called page_start.

icontheme: search first in all themes if the requested icon is symbolic

Symbolic icons use a "-symbolic" suffix to distinguish themselves from
highcolor variants. Note that the dash character here has a different
meaning than the specificity level defined in the icon-naming-spec [1],
as it identifies a property of the icon itself.

Since they might be provided by a parent theme (e.g. the HighContrast theme
relies on the gnome icon theme for them), when we are looking up one we
should first escape the generic icon inheritance mechanism defined in the
icon-naming-spec [1], and privilege a symbolic icon, if it exists in a
parent theme, before applying the inheritance evaluation.

This fixes symbolic icons not working properly when used in the
HighContrast theme with the GTK_ICON_LOOKUP_GENERIC_FALLBACK flag set.

[1]
http://standards.freedesktop.org/icon-naming-spec/latest/ar01s03.html

https://bugzilla.gnome.org/show_bug.cgi?id=674806

themingengine: simplify render_extension() default implementation

Avoid doing useless translations/rotations, since themes will most
likely set different CSS gradients using left/right/top/bottom style
classes, or use a plain color.

https://bugzilla.gnome.org/show_bug.cgi?id=674809

themingengine: render a frame in gtk_render_handle()

We already render a background there after all.

https://bugzilla.gnome.org/show_bug.cgi?id=674808

spinbutton: port to GtkIconHelper

Instead of doing our own lookup for symbolic icons and rendering the
pixbuf, use GtkIconHelper as other widgets do, which saves a bunch of
code.

https://bugzilla.gnome.org/show_bug.cgi?id=674807