| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
This looks odd in combination with the top underflow.
|
|
|
|
|
|
|
| |
Add needed COLUMN INDEX to "new bookmark" row based on previous
bookmark index plus one.
Fixes bug 742727
|
|
|
|
| |
Pointed out in https://bugzilla.gnome.org/show_bug.cgi?id=741250
|
|
|
|
|
|
|
|
| |
When returning 1 for a pair (a,b), a sort function must
return -1 for the pair (b,a), otherwise things can go
badly.
https://bugzilla.gnome.org/show_bug.cgi?id=739729
|
|
|
|
|
|
|
| |
We were concatenating "file://" to the result of g_get_user_special_dir() to build
a URI, but this is not enough on Windows. Use g_filename_to_uri() instead.
See https://bugzilla.gnome.org/show_bug.cgi?id=739453
|
|
|
|
|
|
|
| |
Now that we can use GNotification from here, send a notification when
umounting from GtkPlacesSidebar.
https://bugzilla.gnome.org/show_bug.cgi?id=740896
|
|
|
|
|
|
| |
gtk_get_current_event() can return NULL. Don't crash if it does.
https://bugzilla.gnome.org/show_bug.cgi?id=739150
|
|
|
|
| |
Found in valgrind.
|
|
|
|
|
|
|
|
|
| |
Commit f6870e5b79fd9ab introduced a some memory leaks and could be
improved in some areas.
Fix the memory leaks and apply the improvements.
https://bugzilla.gnome.org/show_bug.cgi?id=737983
|
|
|
|
|
|
|
|
|
|
|
| |
When a drive is ejected or a volume unmounted the current directory
doesn't change most of the times being empty or being a directory that
user shouldn't take care about, like /run/media
Seems more useful to change to $HOME directory in that case so the user
can see something useful and familiar just after unmounting.
https://bugzilla.gnome.org/show_bug.cgi?id=737983
|
|
|
|
|
|
|
|
|
|
| |
Currently we change the current location if we click the eject button of
a mount.
Check whether the user actually clicked the eject button and don't
change location in that case.
https://bugzilla.gnome.org/show_bug.cgi?id=737983
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We get multiple notifications from the bookmark manager when
something changes. Every time, we reconstruct the sidebar contents
completely, by clearing the store. The bookmarks are added with
async calls though, and the code was forgetting to cancel outstanding
async requests, leading to multiple instances of the same bookmark
getting added. Use the cancellable we already have to prevent that.
This could be made much more efficient by not recreating the entire
sidebar quite so often (3-5 times for a single bookmark rename).
https://bugzilla.gnome.org/show_bug.cgi?id=737679
|
|
|
|
|
|
| |
Always quote drive/volume names.
https://bugzilla.gnome.org/show_bug.cgi?id=735192
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=736512
|
|
|
|
|
|
|
|
|
| |
If the query fails because it has been cancelled, it means that
dispose() has been called, so don't try and update anything.
This fixes a segfault with Nautilus in certain situations.
https://bugzilla.gnome.org/show_bug.cgi?id=736512
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=736512
|
|
|
|
|
| |
This is the common practice for objects that start async operations.
https://bugzilla.gnome.org/show_bug.cgi?id=735953
|
|
|
|
|
|
|
| |
Removes two FIXMEs, and ensures that the places sidebar does not block
while enumerating bookmarks or application shortcuts to remote volumes.
https://bugzilla.gnome.org/show_bug.cgi?id=735953
|
|
|
|
|
| |
Based on a patch by Yosef Or Boczko,
https://bugzilla.gnome.org/show_bug.cgi?id=731443
|
|
|
|
|
|
|
|
|
| |
This was silently broken - the code was just assuming that the
text cell renderer is item no. 6 on the list of all cells. That
doesn't work so well if the cell renderers are set up elsewhere
and get rearranged.
Fix this by keeping an explicit pointer to the the text cell.
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=731908
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=730745
|
|
|
|
|
|
|
| |
Instead of just checking the string. This catches things like double
slashes, relative paths and so on.
https://bugzilla.gnome.org/show_bug.cgi?id=730142
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=722211
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=722211
|
|
|
|
| |
Instead of Return value:
|
| |
|
|
|
|
| |
https://wiki.gnome.org/Design/OS/Typography
|
| |
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=723119
|
|
|
|
|
|
|
|
|
| |
Removable USB drives or memory sticks should be powered down
when the eject button is pressed. For this, we need to call
g_drive_stop() instead of g_drive_eject(), provided the drive
can be stopped.
https://bugzilla.gnome.org/show_bug.cgi?id=723121
|
|
|
|
| |
Sadly, this means the code can have never been tested :-(
|
|
|
|
| |
It doesn't hurt to just add the ref here, as the FIXME suggests.
|
| |
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=722029
|
|
|
|
|
|
| |
We go to extra length to set the desktop_uri to NULL when
desktop == home, but then we were adding the (non-functional)
place item anyway. Don't do that.
|
|
|
|
|
|
|
| |
Be more careful when comparing uris during DND - they may
be NULL.
https://bugzilla.gnome.org/show_bug.cgi?id=720264
|
|
|
|
|
|
| |
This was pointed out in
https://bugzilla.gnome.org/show_bug.cgi?id=719505
|
|
|
|
|
|
|
|
| |
Change the GtkSettings default for "shell-shows-desktop" back to TRUE
and also change the default value of the "show-desktop" property on
GtkPlacesSidebar so that the defaultvalue test passes.
https://bugzilla.gnome.org/show_bug.cgi?id=712302
|
|
|
|
|
| |
gtk_settings_get_default() doesn't return a reference, so don't unref it
during dispose.
|
|
|
|
|
|
|
| |
Use the just-added shell-shows-desktop GtkSetting to determine if we
should also show the "Desktop" folder in the places sidebar.
https://bugzilla.gnome.org/show_bug.cgi?id=712302
|
|
|
|
|
|
|
| |
The GtkFileInfo is only used if there's no bookmark name,
or to get the icon for the remote share or local folder.
https://bugzilla.gnome.org/show_bug.cgi?id=711548
|
| |
|
|
|
|
|
|
|
| |
The file chooser used to do this, the places sidebar
still needs to learn it.
https://bugzilla.gnome.org/show_bug.cgi?id=711636
|
|
|
|
|
|
|
| |
This is necessary to implement the filechooser property
of the same name.
https://bugzilla.gnome.org/show_bug.cgi?id=711574
|
|
|
|
| |
gtkplacessidebar.c was looking foreign.
|
|
|
|
| |
Pointed out in by Nelson Benitez in bug 707679.
|
|
|
|
| |
This reverts commit 588ffa8c3241eff2f8e2d99b911a6bcb539347bd.
|
|
|
|
| |
This reverts commit 1b839d4b72a2cedb2bb633b5acd57239860693f4.
|