| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|
|
| |
For testing whether it improves the situation in https://gitlab.gnome.org/GNOME/nautilus/-/issues/2673
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When nautilus is asked to handle a regular file (e.g. its URI being passed as command line argument),
we open its parent folder and select the file. But for archives, we want to extract them immediately
instead (this is convenient when e.g. handling opening a downloaded file from the web browser).
However, we don't actually extract the archive unless we support are the default handler for that MIME type,
in which case we don't select it as a regular fiel either, which is inconsistent.
So, make sure we only handle archives specially if we actually meant to extract them.
Fixes https://gitlab.gnome.org/GNOME/nautilus/-/issues/2274
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
It doesn't make sense to change locations onto the same folder you
are dragging from. The easiest way to tell if we are dragging
onto the source drag is when the drag_item_action is set to 0.
This has the added benefit that drag_item_action is also set to
0 when hover over a file. We have been calling
`nautilus_files_view_handle_hover()` unnecessarily on file destination
(which ultimately had no effect).
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2670
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In `nautilus_dnd_get_preferred_action()` we check if the dropped file
and target file are equal, but only after the check if we are dropping
onto an archive. The `g_file_equal()` check if wrapped in a
`dropped != NULL` check which turns out to be unnecessary because
we return in the very beginning when `dropped == NULL`
Move around the drop target action checks to simplify and
prevent dropping anything on itself.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2737
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In real_set_selection, we call gtk_selection_model_get_selection()
to be able to efficiently update the selection (we don't care about
unselected items that we aren't going to select). That bitset,
per the documentation, shouldn't be modified. This bug (which made
this function effective useless) went unnoticed because we were
setting the selection elsewhere by poking the internals of the
listbase in set_focus_item. We don't want to do that anymore, and
this bug became apparent.
Make a copy of the bitset, and rename the existing bitsets to clarify
what we are doing. Also, set the selection after setting the
focus just in case that ever conflicts in the future.
|
| |
|
|
|
|
|
|
|
|
| |
When moving away from GtkMessageDialog, we forgot this small
program is not part of the main application. [0]
Therefore, we need to initiate libadwaita here, before using any of its widgets.
[0] a80aacdf2c44fd9c28a292ee599b9e74418faa10
|
|
|
|
|
|
|
|
|
|
|
|
| |
When moving away from gtk_dialog_run() [0], we forgot this small
program is not part of the main application. So, without anything
blocking the `main()` function, it's going to quit right away, without ever drawing the dialog.
Credit goes to Will Thompson for diagnosing the issue, and GTK
test suit for the while loop solution I've taken from there.
Closes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2209
[0] d87f5be77bb4ee03e363cb560aaefb00cf56b275
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since 8978c6375bbe0cd26fffded25bc4606cab2c88e6, `autorun.sh` (but not
`.autorun` or `autorun`) has been run via `/bin/sh`, not executed
directly. The justification in that commit was as follows
> nautilus-autorun-software: Use /bin/sh for autorun.sh execution
>
> If the autorun file is called autorun.sh it is clearly meant for
> execution through a sh processor. The benefit of using /bin/sh as the
> executable is that autorun will also work on FAT formatted media (which
> does not allow autorun.sh to have +x permission set)
However that patch was incomplete. The third argument to _check_file()
is 'must_be_executable'; if set, _check_file() will return FALSE when
the file does not have the executable bit set. It is possible to mount
FAT media in such a way that regular files get the execute bit, but this
is not the default.
As a result, if you place `autorun.sh` in the root directry of a FAT
device, Nautilus offers to run it, but w hen you take it up on that
offer it fails, claiming:
> Unable to locate the program
Complete the special handling of the autorun.sh name by not requiring it
to be executable.
See https://gitlab.gnome.org/GNOME/nautilus/-/issues/2675
|
|
|
|
|
|
|
| |
GtkStyleContext is deprecated in 4.10, use the corresponding
GtkWidget functions.
Also bump GTK version to use the new gtk_widget_get_color().
|
|
|
|
| |
mailman has been moved to discourse.
|
| |
|
|
|
|
| |
This was removed over 11 years ago in 7363013201ec2950958db529ea96c0f2442b3c53
|
|
|
|
|
|
|
| |
Widgets are visible by default on GTK4. Remove visible property
when setting to TRUE.
Source: https://gitlab.gnome.org/GNOME/gtk/-/blob/main/gtk/gtkwidget.c#L1275
|
|
|
|
|
|
|
|
| |
We are using a workaround to get the preferred action in x11
where we check the preferred action of the drag. This works for local
drags, but obviously not non-local drags. The workaround is necessary
because of a bug in gtk. Let's just default back to COPY for non-local
drags.
|
|
|
|
|
|
| |
Use g_signal_connect_object to automatically disconnect the signal
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2548
|
|
|
|
|
|
|
|
|
| |
The search bar from the main window doesn't have a placeholder,
which the HIG generally requires.
Add the necessary placeholder.
Fixes https://gitlab.gnome.org/GNOME/nautilus/-/issues/2698
|
|
|
|
|
|
|
|
| |
In a wayland session, leave is emitted on a drop or cancelled drop.
In x11 that's not the case. Explicitly clear the hover timeout
on drop.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2585
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
In gtk3 Nautilus used to construct the uri-list.
nautilus_uri_to_native_uri() was used to ensure that apps that didn't
have gvfs access could access the dropped file. Now gtk4 constructs
the uri-list, so a similar process is handled on the gtk side.
Related: #2632
|
|
|
|
| |
The last user was NautilusLink, and that's been gone for a while.
|
| |
|
|
|
|
| |
(cherry picked from commit 61530d7cc21f86420ede29d540b442f88da83725)
|
|
|
|
|
|
|
|
|
|
|
|
| |
For a very long time, we've been passing NULL as the
parent window to show_dialog() for failed rename
operations. This is in a deeply nested callback, so
rather than trying to pass the window all the way back
just get the current active window, which very likely is
the correct window given how quickly this operation will
fail.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/1199
|
|
|
|
| |
See https://github.com/flatpak/libportal/pull/104
|
|
|
|
| |
(cherry picked from commit 365411e0430b75aa4d37a98d1306df7cb3f22a12)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Pango library doesn't support specifying text-transform in utf8
codepoints. It instead uses positions in the raw string. This is why we
have problems with, for example, the Cyrillic label in the UI file
text_transform positions are specified as 0 for the start and 1 for
the end and the first Cyrillic letter consist of two bytes.
Remove attributes from the UI file and implement label text capitalization
using the function eel_str_capitalize() which does the same but in
a proper way for type_value_label and type_file_system_label.
Fixes https://gitlab.gnome.org/GNOME/nautilus/-/issues/2494
|
|
|
|
|
|
|
|
| |
Cyrillic languages"
This reverts commit 91f7fb675e558fb0b2c3dc5a6082386b0aef159a.
The commit message was broken, missing context.
|
| |
|
|
|
|
| |
To avoid leak.
|
| |
|
| |
|
|
|
|
|
|
| |
We increase its reference count again in
nautilus_window_slot_set_location (), so unref it afterwards to prevent
a leak going back all the way to nautilus_application_activate ()
|
|
|
|
|
|
|
|
|
|
|
| |
Currently, if the new_selection list is NULL, then the
check_select_old_location_containing_folder function sets it to a newly
allocated list.
Since we don't own the new_selection list, let's copy it before passing
it to the check_select_old_location_containing_folder function so that
it always returns a newly allocated list, which we make the
pending_selection point to in order to avoid a leak.
|
| |
|
|
|
|
|
|
|
| |
We are setting the query editor text, but not actually
starting the search.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2518
|
|
|
|
|
|
| |
If we are updating the query editor's text (i.e. through the
shell search provider), we should move the cursor position to the
end of the text, otherwise it's at the beginning of the text.
|
| |
|
| |
|
|
|
|
| |
(cherry picked from commit b89017dd1a8391cbea81e405abb094ef02046aa5)
|
|
|
|
|
|
|
|
|
| |
We don't need an extra action to show help-overlay.
It already managed by GTK.
Source: https://docs.gtk.org/gtk4/class.Application.html#automatic-resources
Topic: https://discourse.gnome.org/t/nautilus-unnecessary-action-to-show-help-overlay/12516
|
| |
|
|
|
|
|
|
|
|
|
| |
Add trailing ellipsis at the end of Empty Trash to fulfill
GNOME Human Interface Guidelines.
Trailing ellipsis is used when further input or confirmation is
required from the user before the action can be carried out.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2502
|
|
|
|
|
|
|
|
|
| |
Remove the trailing ellipsis from Visible Columns to
fulfill the GNOME Human Interface Guidelines. Trailing ellipsis is used
when further input or confirmation is required from the user before the
action can be carried out.
Fixes: https://gitlab.gnome.org/GNOME/nautilus/-/issues/2534
|
|
|
|
| |
This adds a support URL to the Nautilus forums for easier access to ask questions
|
| |
|