GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

Adapt to Orca 47

The screen reader Orca has seen some fundamental changes between
Orca 46 and Orca 47. While they are improvements overall, they do
require changes to Dolphin to preserve the intended user
experience for Orca users.

The biggest change is perhaps that Orca will now not only announce
changes to the currently focused item, but also of its parent,
which means we do not need to pass focus around between file items
and the main view within Dolphin, but can keep focus on the file
items most of the time. This commit implements this.

The only exception of when we cannot have focus on the items within
the main view is when the current location is empty or not loaded
yet. Only then is the focus moved to the view itself and the
placeholderMessage is announced.

This commit worsens the UX for users of Orca 46 or older, so this
should only be merged once most users are on Orca 47 or later.

GIT_SILENT Sync po/docbooks with svn

Always focus the view after place activation

This one-liner makes sure keyboard focus always moves to the active
view when a place in the places panel is activated. Previously
focus would remain on the places panel if the activated location
was identical to the already displayed location.

This added consistency is helpful because keyboard-only users will
get used to be able to move from the places panel to the view by
pressing the Enter key, even though this does not always work
prior to this commit.

This issue was identified in an accessibility scan done by HAN
University of Applied Science.

GIT_SILENT Update Appstream for new release

(cherry picked from commit 73b83b866eae1e095d84d611ceb6dd094a54b98f)

KItemListSmoothScroller: stop animation on property or targetobject change

Due to changes in 15e0c2a98f480f203ce168b6fa9a2e1f2b0ed8f8 we do not
automatically stop animations now on property or targetobject change,
which causes warnings and timeouts.

This patch makes sure we stop them if their state is Running,
before modifying them.

See also:
https://invent.kde.org/system/dolphin/-/merge_requests/857#note_1083279

GIT_SILENT Sync po/docbooks with svn

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

GIT_SILENT made messages (after extraction)

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

Make "open path" and "open path in new tab" scroll to the selected item

**Open Path**

When user clicks on "Open Path" after searching for an item,
user expects the view to show the item immediately.

We wait for the KItemListSmoothScroller to be done with its animation before the
scrollbar sizes are being changed.

**Open Path in New Tab**

When user selects "Open Path in New Tab", we open a new tab to the folder where the file is, then select and set the file current. We need to get the correct tab when opening one, so it has been added as a return value.

BUG:495613

GIT_SILENT Sync po/docbooks with svn

DolphinTabPage: Update container view url on redirection

On url redirect, we should check which container url is being changed
and then update it accordingly. This makes sure the tab name is
updated.

We also should not disconnect redirection on view activation
since the redirection might be used by the other split.
The disconnection is done in `setSplitViewEnabled` instead.

This allows us to update the tab name every time the url changes,
even inside a splitview where the split which name is changed is
not active.

BUG:496414

fix: Flatpak must have --filesystem=~/.var/app

This permission is needed because --filesystem=host doesn't expose other flatpak files from ~/.var/app

Signed-off-by: Ryan Brue <ryanbrue.dev@gmail.com>

GIT_SILENT Sync po/docbooks with svn

DolphinViewContainer: setSearchPath value only if searchBox is active

Fixes a crash when searchbox is not active and tab gets renamed.

I forgot to add this to merge request
https://invent.kde.org/system/dolphin/-/merge_requests/864

CCBUG:496387

DolphinViewContainer: set searchPath on redirect, check for connected urlNavigator

This fixes a dolphin crash if tab with a search box open gets its path
changed while its not active.

This patch adds a guard for the m_urlNavigatorConnected->setLocation,
and makes sure the searchPath is set to the correct folder.

BUG:496387

GIT_SILENT Sync po/docbooks with svn

Switch to C++20

DolphinView: fix typos in signal connection

NO_CHANGELOG

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

DolphinView: beginAnchoredSelection after setSelected in selectNextItem

Currently when next item is set selected, it's not set as an anchor.
This means that shift-click will not select items between the selection
and the next item, and instead behaves like ctrl-click.
This is rather inconsistent behavior.

We should set the next selected item as an anchor so that shift-click
will work: This change starts the `beginAnchoredSelection`
process after `setSelection`.

For example, if user trashes an item and the next item is selected,
shift-click would stop working.

BUG:495371

GIT_SILENT Sync po/docbooks with svn

Update org.kde.dolphin.appdata.xml

Update org.kde.dolphin.appdata.xml

natural sort: exclude extension when comparing filenames

Currently natural sort compares the entire filenames
(basename.extension) when sorting. This causes eg.
"a 2.txt" to appear before "a.txt" when sorted by ascending.
This is unintuitive since people prioritize basenames more
than file extensions.

Instead, change natural sort to compare by basename only and
fallback to comparing extensions if basenames were equal.
This change causes "a.txt" to appear before "a 2.txt" and
matches how other platforms such as GNOME and Windows behave.

BUG: 416025
BUG: 470538
BUG: 421869
BUG: 312027

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

Updated supporters

Added supporters

Added supporter from fundraiser

CI Flatpak - Add required permission

GIT_SILENT Sync po/docbooks with svn

FoldersTabsSettingsPage: Remove double cast, remove unnecessary variable

This uses only qobject_cast here, having static_cast after it is
redundant.

Also remove the topLevelWidgets variable, since we can just
call the method instead.

generalviewsettingspage: Use qobject_cast instead of static_cast

In generalViewSettingsPage, ActiveWindow is not necessarily a
mainWindow, which can cause dolphin to crash. For example when user
opens folder properties and then dolphin settings, this part thinks
the folder settings is mainWindow.

Instead of static_cast, use qobject_cast, since it will return nullptr
if the DolphinMainWindow is not found.

BUG:492741

GIT_SILENT Sync po/docbooks with svn

Avoid flickering of space info on startup

Prior to this commit, when Dolphin was opening in a directory for
which the free space information cannot be retrieved, the free
space info in the status bar would still briefly be visible before
hiding for good.

This commit avoids this flickering by keeping the space info hidden
until space info has been successfully retrieved. There is no use
showing an empty/wrong space info before that anyway.

I assume the error in the previous code was that it assumed that
one could prevent a widget from being shown by overriding
QWidget::showEvent(). This does not work because this method is
only called to notify QWidgets of their state change.

This commit was primarily written because the brief showing of an
empty space info was messing with automatic tests.

Test that each object has distinguishable accessible info

This commit extends the dolphinmainwindowtest for the accessible tree
to test that each object a user can reach through the Tab key or the
Shift+Tab key combination is distinguishable by its accessible info
from the previous object.

Objects are considered distinguishable if they have different
accessible names or their first ancestor with an accessible name is
different.

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Upgrade release service version to 25.03.70.

GIT_SILENT Sync po/docbooks with svn

Skip splitter in Tab order

When moving through Dolphin by repeatedly pressing the Tab key,
there is one seemingly invisible object that catches keyboard
focus. On a default toolbar it receives keyboard focus between the
"Details View Mode" button and the first button of the location
bar. That object is the splitter of the location bar itself, which
never requires manipulation by keyboard. This commit skips this
object by setting its focusProxy to the primaryUrlNavigator()
instead.

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

when splitting view, use selected folder as url

BUG: 483937

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

Make Escape move focus from location bar to view

Pressing Escape on the location bar while in breadcrumb mode has no
effect at all. This commit changes this to instead move the focus
to the active view. This is more logical because a user pressing
escape while having focus on the location bar is no longer
interested in interacting and changing the location in the location
bar. They most likely want to act on the current location instead.

Have "Replace Location" toggle focus of the view

The "Replace Location" action moves focus to the location bar and
selects the full path there so it can easily be replaced with a
different path.

When the full path is already selected, triggering "Replace
Location" will toggle the editable state of the location bar.
This commit makes it, that this also moves the focus to the view.
This way, pressing Ctrl+L multiple times will switch back and forth
between focusing and selecting the location bar path and focusing
the view. This seems more sensible than only partially resetting
the state when the "Replace Location" keyboard shortcut is pressed
twice.

refactor: replace QString() with QStringLiteral() for better performance

CMakeLists: make KFileMetadata a required dependency

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Update Appstream for new release

(cherry picked from commit 0fb381a7edef8b3c60f1c01fb7cc9d9335556fb4)

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

GIT_SILENT made messages (after extraction)

Overhaul main view accessibility

This commit brings the main view of Dolphin into a usable state
accessibility-wise. Users of screen readers should have a way better
experience while browsing files and folders and navigating along the
file system hierarchy.

This commit fixes most of the remaining already-identified
accessibility issues listed in
https://invent.kde.org/teams/accessibility/collaboration/-/issues/28,
but not all. Namely, these should now be fixed:

1. Orca should read the element type in dolphin (file, folder, device,
link to folder, link to file)
2. Orca should read complete label in icon and compact view mode,
currently it only speaks the name, but there could be additional
information like the number of elements or the file size.
3. Orca is not able to announce Selecting / Unselecting files in
Dolphin. It also never announces how many items are selected in total.
(Announcing the total selection can be done by reading out the view
element or by pressing the Tab key to get to the status bar with the
relevant information.)
4. Dolphin opens on the home directory, but Orca doesn't tell you so.
Consider enclosing the area in a frame/panel which updates its
accessible name each time you modify the current path by entering or
leaving a directory.
5. I don't know what the folder presentation widget is, but it should
be presented as a grid view. Currently, we have a terrible experience
because the entire row of folders is read at once, with no indication
that we can move left and right with the arrows to go between the
elements of a row. When I found that out, however, I discovered that
when you're on the last icon of the first row and press right arrow,
you get to the first icon of the next row, but that's not announced,
instead, the whole row is announced at once
6. Orca should announce the current elements instead of "layered pane"
when the Folder / File view gets the focus in dolphin
7. Orca reads only name in Table View only of Dolphin
8. Items are sometimes confusingly announced as "collapsed" in contexts
in which there is no concept of collapsing/expanding e.g. in icon view
mode.

A lot of code was moved around and renamed. The three accessibility
classes, which all used to be in the same file, are moved into separate
files.

*Acknowledgement*
Thanks to Christian Hempfling and bgt lover for testing as well as
originally identifying a lot of the pain points being addressed here.

This work is part of a my project funded through the NGI0 Entrust Fund,
a fund established by NLnet with financial support from the European
Commission's Next Generation Internet programme, under the aegis of DG
Communications Networks, Content and Technology.
https://kde.org/announcements/2024_ngi_openletter/

GIT_SILENT Sync po/docbooks with svn

ViewProperties: Store view properties in extended file attributes

Existing settings are converted.

Works on most FS except FAT/exFAT which fallback to .directory files.

If the extended file attributes (in ADS in Windows) can't be saved, they are saved to file as before.

BUG: 322922

You can see file xattr using for instance for Unix filesystems:

getfattr -d /home/meven

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

Add a sort by duration option for videos

BUG: 482508

Signed-off-by: Somsubhra Bairi <somsubhra.bairi@gmail.com>

GIT_SILENT Sync po/docbooks with svn

Add test to avoid accidental divergence of desktop and phone ui

The dolphinui.rc and dolphinuiforphones.rc file describe parts of
the desktop and phone user interfaces for Dolphin. They are meant
to be different in specific ways, but identical in many others.

This test makes sure that the two ui files stay mostly identical by
requiring any differences between them need to be explicitly
stated/registered as an exception in the test. This way no
accidental divergence of user interfaces can happen.

Move settings category switcher to the top on mobile

There is not enough horizontal space to show the full Dolphin
settings window on the average phone. This commit saves some
horizontal space by moving the category switcher in Dolphin
settings from being a list on the left to being tabs on the top.

Disable inline renaming for phones by default

It currently does not seem to work together with virtual keyboards.

Add phone UI

Previous to this commit launching Dolphin on phones (e.g. those
running Plasma Mobile) would show Dolphin with its default user
interface optimised for desktop usage. This commit changes this so
instead a phone form factor optimised user interface is used.

The differences to the default UI configuration are:
-Toolbar at bottom
-Icon-only toolbar
-Different actions on the toolbar
-Places panel hidden
-Location bar at the top with a button to show places
-Zoom slider hidden (pinch gestures to zoom still work)

Through these changes Dolphin actually has a good user experience
on phones by default. All the features were already there.
Especially Steffen Hartleib's work to trigger selection mode on
long press leads to great UX when dealing with multiple files.

Still, this might be considered just a start towards making Dolphin
great on phone form factors. Secondary windows that Dolphin might
spawn are not yet adapted, but are usable on Plasma Mobile as they
are anyway.

kitemlistcontroller: process forward/back buttons when double-clicking

Tapping the forward or back mouse buttons quickly enough makes Dolphin
interpret the action as a double-click of the button in question and
handle it in mouseDoubleClickEvent() instead of its normal button
handler. This means that certain button presses might seem delayed or
"swallowed" when quickly navigating forwards or backwards through the
history.

Since a double-click of the forward or back button is currently
meaningless, fix this by emitting a normal mouseButtonPressed event for
those buttons in the double-click handler and skipping any further event
processing.

Co-authored-by: Felix Ernst <felixernst@zohomail.eu>
CCBUG: 485295

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT made messages (after extraction)

Fix double-click view background feature

c934e803647674b4692668f047b6ffa18121982a was meant to change the
double-click view background feature to only allow double-clicks
with the left mouse button. However, it mistakenly did the exact
opposite and allowed every double-click except ones with the left
mouse button to trigger the feature. This one-liner fixes this.

GIT_SILENT Sync po/docbooks with svn

Add context menu items to Trash view and items

Sort by and View mode are already available through hamburger menu; cut and copy are already available through shortcuts
BUG: 493808
BUG: 476955

GIT_SILENT Sync po/docbooks with svn

dolphinview: when rename dialog finishes, immediately update the model and the selection

On sucessful return of the rename dialog, we update
the model and the selection immediately to reflect
the new name. This is to avoid the short duration
after the rename during which the selection is lost.

Currently, after the rename dialog finishes, the selection
is briefly lost for about 1 second until the view
automatically refreshes.

This patch updates the model and selection immediately
after the dialog finishes, so the renamed file is still
selected.

BUG: 481717

dolphinview: after inline renaming, immediately update the selection

Currently, the inline renaming updates the model and
the list of selected files (m_selectedUrls) immediately
after the user presses enter, but the selection in the
view is not immediately updated. This results in the
selection to be briefly lost for about 1 second until
the view automatically refreshes.

This patch updates the selection immediately after
the user presses enter, so the renamed file is still
selected.

BUG: 481717

KFileItemModel: fix `setData` in the renaming usage

The `setData` function is used by inline renaming to
update the `text` role (i.e. file name) in the model
before actually do the renaming.

However, the current implementation only updates `text`
and url in KFileItem in this case, leaving the `url` role
and the `m_items` cache untouched. This makes the cache
incoherent, so `index()` won't find the renamed item. It
also makes the `url` role incoherent with the `text` role.

This also fixes the problem mentioned in bug #481561
where when inline renaming fails, it still shows the new
name. (Because `m_model->index` fails to find the new name
and rename it back on job failure.)

BUG: 481561

dolphinview: add error message when renaming failed in dialog mode

BUG: 481561

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

daemon: prevent daemon to quit on `openNewWindow`

Accessibility: Fix message boxes not being announced

This commit moves the focus to the message of newly shown passive
messages when accessibility software is used. Moving focus to an
object generally means that it will be announced, so this makes
sure that users of accessibility software know when they are being
notified of a change.

DolphinViewContainer: Delay changing the url

KCoreDirLister::itemsDeleted() signal is being emitted twice for the
same url. This results in Dolphin displaying an incorrect location.
Delay changing the url instead of delaying
DolphinView::currentDirectoryRemoved() so that the check for current
directory being removed in KFileItemModel::slotItemsDeleted() occurs
correctly, while still ensuring that KCoreDirLister is not prematurely
modified.

BUG: 492277, BUG: 473377

GIT_SILENT Update Appstream for new release

(cherry picked from commit 2f8b55cd88ebaa258a04a01b7b789592bb064fe2)

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT made messages (after extraction)

Add missing include moc

GIT_SILENT Re-enable Windows CD

GIT_SILENT Sync po/docbooks with svn

GIT_SILENT Sync po/docbooks with svn

SVN_SILENT made messages (.desktop file) - always resolve ours

In case of conflict in i18n, keep the version of the branch "ours"
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"

Dolphinpart: Define KPart capabilities instead of deprecated ServiceTypes

By now, Konqueror uses the new mechanism introduced in
https://invent.kde.org/frameworks/kparts/-/merge_requests/86.
Thus, we can avoid using the deprecated SeviceTypes which are also
removed from the official KPluginMetaData API.