Commit graph

5234 commits

Author SHA1 Message Date
Elvis Angelaccio 735b171451 Set version in project()
As discussed on release-team [1], this will enable automatically setting
product versions on bugzilla [2].

[1]: https://mail.kde.org/pipermail/release-team/2017-July/010474.html
[2]: https://community.kde.org/Guidelines_and_HOWTOs/Application_Versioning#Bugzilla_versions
2017-09-03 11:18:04 +02:00
Robert Hoffmann 94fab8c80e
Limit folder panel to home directory if inside home
Summary:
Added the option to limit the displayed folders in the folder panel (F7) to the tree below the user's home directory if the current URL is inside the home directory.
This can be configured in the preferences General/Behaviour tab by checking the corresponding check box.

Reviewers: #dolphin, elvisangelaccio, emmanuelp

Reviewed By: #dolphin, elvisangelaccio, emmanuelp

Subscribers: emmanuelp, elvisangelaccio, #konqueror, #dolphin

Differential Revision: https://phabricator.kde.org/D7477
2017-09-01 10:15:40 +02:00
Kai Uwe Broulik 652f57d281 Prefer place name over actual name in title bar
When inside a place, the address bar already gives it precedence over the actual folder name.
By doing this in the title bar also, we make it consistent and can mask ugly technical terminology
like "trash:/" and instead show the nice localized "Trash" place name as well as "Home" instead
of lowercase internal user name.

BUG: 211959

Differential Revision: https://phabricator.kde.org/D4826
2017-08-29 17:15:24 +02:00
Francisco Navarro Morales 8be1e2aa07 Add Genre to additional information for audio files. Fixes T5599
Summary: I have just imitated the code that made this funcionality for artist and album additional information for audio files.

Reviewed By: #dolphin, emmanuelp

CCBUG: 374558

Differential Revision: https://phabricator.kde.org/D7215
2017-08-27 23:11:31 +02:00
Kai Uwe Broulik dfbaf21218 [AdditionalInfoDialog] Prevent hiding "text" role
You cannot do this from the menu but the option was there (but ignored) in the "View Properties" dialog.

BUG: 302622

Differential Revision: https://phabricator.kde.org/D7386
2017-08-25 16:48:48 +02:00
Elvis Angelaccio 3bfdf0d2f0 Port to new KStandardAction slot syntax
For more compile-time safety.
2017-08-24 17:32:06 +02:00
Kai Uwe Broulik 1018848416 Support middle clicking of Back/Forward/Up/Home toolbar buttons
This opens the resulting page in a new tab.

BUG: 358649

Differential Revision: https://phabricator.kde.org/D7390
2017-08-24 15:45:28 +02:00
Christoph Feck 24baf24f5f Fix typo in udi string
Spotted by Chris Holland.

BUG: 383182
2017-08-23 12:53:41 +02:00
l10n daemon script 52e58f4274 GIT_SILENT made messages (after extraction) 2017-08-10 03:19:16 +02:00
Elvis Angelaccio d74f62208c Drop duplicate i18n strings
This comment (introduced in commit 39f89141b0) no longer seems to
apply, so drop the duplicate translations and just use the default
strings from kconfigwidgets.

Differential Revision: https://phabricator.kde.org/D6779
2017-08-09 23:15:30 +02:00
Elvis Angelaccio 5481709016 Port to KStandardAction::RenameFile
It was introduced in kconfig(widgets) 5.25. Using the standard action
results in less code and no need to hardcode the F2 shortcut.

Differential Revision: https://phabricator.kde.org/D6777
2017-08-09 22:59:38 +02:00
Elvis Angelaccio f38960e034 Port to KHelpMenu
Less code and KHelpMenu respects KIOSK restrictions.

Differential Revision: https://phabricator.kde.org/D7095
2017-08-03 18:50:39 +02:00
Elvis Angelaccio 39fcc89450 Merge branch 'Applications/17.08' 2017-07-28 14:08:33 +02:00
Elvis Angelaccio 594a367834 Fix loading of dolphinui.rc
dolphin.qrc is compiled into a static library, which means we need to
force its loading with the Q_INIT_RESOURCE macro.

See http://doc.qt.io/qt-5/resources.html#using-resources-in-a-library

BUG: 382848
FIXED-IN: 17.07.90

Differential Revision: https://phabricator.kde.org/D6958
2017-07-28 14:05:53 +02:00
David Edmundson 82bbdb6e6b Merge branch 'Applications/17.08' 2017-07-28 12:35:28 +01:00
David Edmundson 63e2d65bc8 Use https for knsrc providers
CCBUG: 382820
2017-07-28 12:34:42 +01:00
Elvis Angelaccio 4b5ea05b67 Port to KStandardAction::MoveToTrash
It was introduced in kconfig(widgets) 5.25. Using the standard action
results in less code and no need to hardcode the Del shortcut.

Test Plan:
- Change shortcut in System Settings -> Shortcuts -> Standard Shortcuts
- Make sure the new shortcut is used by dolphin.

Differential Revision: https://phabricator.kde.org/D6778
2017-07-27 16:15:03 +02:00
Elvis Angelaccio 82b2dbd3ec Merge branch 'Applications/17.08' 2017-07-27 16:05:18 +02:00
Elvis Angelaccio c3de867622 Fix drop menu position with urlnavigator drops
Commit 1e251d2f6a in kio broke drop menus when dropping on the URL
navigator (menus show up in the DolphinView rather than the URL bar).
This happens because in DolphinView::dropUrls() we set `this` as the
widget passed to KJobWidgets::setWindow() (in DragAndDropHelper::dropUrls()).

We need to replace `this` with the actual widget that received the QDropEvent
and that can mapToGlobal() the relative pos of the drop event.
Unfortunately this widget is not KUrlNavigator itself, but one of its
KUrlNavigatorButton children (private class, not exported). So
unfortunately we need a new API in KIO that exposes this child widget.

Differential Revision: https://phabricator.kde.org/D6684
2017-07-27 15:49:52 +02:00
David Faure cf273cd8a9 Bundle the XMLGUI files into qrc resources.
This is a step towards being able to run applications uninstalled,
and it simplifies deployment on non-linux.
2017-07-25 11:39:57 +02:00
Elvis Angelaccio 9a957c3b5c Don't add null widget to layout
m_expandableFolders is created only in DetailsMode. This fixes a QLayout
runtime warning.
2017-07-21 12:52:10 +02:00
Elvis Angelaccio 9f86b5a266 Fix layout warning
This fixes the following warning:

    QLayout: Attempting to add QLayout "" to PreviewsSettingsPage "", which already has a layout

`topLayout->addLayout(fileSizeBoxLayout)` takes ownership of
`fileSizeBoxLayout`, so there is no reason to use `this` as parent.
2017-07-21 12:09:17 +02:00
Luigi Toscano fccb89fc5c Merge remote-tracking branch 'origin/Applications/17.08' 2017-07-18 19:00:04 +02:00
Tobias C. Berner 286eaa7402 Add role for file creation time.
Summary: At least on FreeBSD UFS and ZFS support birthtimes of files.

Reviewers: #freebsd, #dolphin

Subscribers: #konqueror

Differential Revision: https://phabricator.kde.org/D6243

(cherry picked from commit d8061b4d9e)
2017-07-18 18:57:47 +02:00
Tobias C. Berner d8061b4d9e Add role for file creation time.
Summary: At least on FreeBSD UFS and ZFS support birthtimes of files.

Reviewers: #freebsd, #dolphin

Subscribers: #konqueror

Differential Revision: https://phabricator.kde.org/D6243
2017-07-18 18:44:14 +02:00
Albert Astals Cid ec3da79560 GIT_SILENT Upgrade KDE Applications version to 17.07.80. 2017-07-17 22:10:37 +02:00
Albert Astals Cid ea142de633 GIT_SILENT Upgrade KDE Applications version to 17.11.70. 2017-07-17 20:28:07 +02:00
Luigi Toscano 72ddf319b6 Merge remote-tracking branch 'origin/Applications/17.04' 2017-07-15 14:40:03 +02:00
Elvis Angelaccio 69c3eba0ad Fix knewstuff popup menu position on wayland
On Wayland we need to pass a parent to popup menus.
See https://community.kde.org/Guidelines_and_HOWTOs/Wayland_Porting_Notes#Popup_Menus

So we use the new setParentWidget() method from KNewStuff (introduced in
Frameworks 5.37).
2017-07-13 17:30:36 +02:00
Elvis Angelaccio aa771da32d Add missing emit keywords
GIT_SILENT
2017-07-13 17:11:33 +02:00
Albert Astals Cid df0741ce37 GIT_SILENT Upgrade KDE Applications version to 17.04.3. 2017-07-10 00:33:40 +02:00
Kai Uwe Broulik da42a9eefb [Places Panel] Disallow editing devices
I introduced this bug when I changed the layout of the menu.
While it would be nice if one could edit devices, it is currently not possible
and would require significant re-engineering effort in Solid.

While the "Edit" menu works just fine, changes are not persisted.

Reviewed-By: emmanuelp

Differential Revision: https://phabricator.kde.org/D6294
2017-06-27 14:03:31 +02:00
Kai Uwe Broulik 0d2f86de18 Support "Deletion Time" role in Trash
This allows to sort by and show (both as additional data in icon view and column in column
view) deletion date of files in Trash.

CHANGELOG: It is now possible to view and sort by "Deletion Time" in Trash

BUG: 153492
FIXED-IN: 17.08.0

Differential Revision: https://phabricator.kde.org/D6269
2017-06-20 11:32:29 +02:00
Elvis Angelaccio 96eff55e75 Merge branch 'Applications/17.04'
* Applications/17.04:
  Change in "Open in new tab" feature in Dolphin
  Ignore drops-onto-items from invalid places items
  Revert "Increase smooth scrolling animation duration from 100 to 300 ms and set easing curve to InOutQuart"
2017-06-11 19:33:36 +02:00
Emirald Mateli ec9f4ed17c Change in "Open in new tab" feature in Dolphin
Summary:
This patch proposes a change to the "open in new tab" feature.

The "open in new tab" feature will try to open selected items (files or folders) in a new tab, however, if there are no valid items to be opened in a new tab then nothing will happen, making it look like a bug. This patch adds the functionality that when there are no valid items(files or folders) to be opened in a new tab the current folder will be opened.

Test Plan:
1. Select a file(pdf, text, image etc) in Dolphin
2. Click on the "Open in new tab" toolbar button

Expected: since the file is not a valid target to open in a new tab, the current directory should be opened (as is the case where selection is empty)
Actual: Nothing happens after the button is pressed

Reviewed By: #dolphin, elvisangelaccio

Differential Revision: https://phabricator.kde.org/D6182
2017-06-11 19:28:39 +02:00
Elvis Angelaccio c85ca11455 Ignore drops-onto-items from invalid places items
If the QMimeData object created by PlacesItemModel doesn't have any url
set (e.g. when dragging unmounted devices), it is detected by the
resulting DropJob as "drop raw data" because the mimeData has one format set
(the internalMimeType() used for dragging between places items).

This results in a crash because the DropJob schedules a PasteJob, but in
the meantime the QDrag from Dolphin ends and deletes the mimeData object
that was passed to the paste job.

The fix is to prevent the DropJob in the first place. We can introduce a
new internal mimetype that we use to blacklist drops-onto-items (while
still allowing drops-between-items). This way PlacesItemModel can set
the blacklist flag if the mimeData is being created without urls.

BUG: 373005
FIXED-IN: 17.04.3

Test Plan:
Drag and drop an unmounted device to another place item or the DolphinView, doesn't crash anymore.
Dropping the unmounted device between two places item still works.

Differential Revision: https://phabricator.kde.org/D5535
2017-06-11 19:11:56 +02:00
Elvis Angelaccio a2f9e005a5 Drop unnecessary method 2017-06-06 19:16:34 +02:00
Albert Astals Cid 5246e19a40 GIT_SILENT Upgrade KDE Applications version to 17.04.2. 2017-06-05 20:52:31 +02:00
Elvis Angelaccio 883b908b4b Revert "Increase smooth scrolling animation duration from 100 to 300 ms and set easing curve to InOutQuart"
This reverts commit 58c5eae195.

Reasons against this change:

1. Scroll is now too fast with devices such as touchpads or trackpoints.
2. Scroll behavior is now inconsistent between Dolphin and other applications
   (e.g. the Plasma file dialog). Breeze's default animation duration is 100 ms.
3. Many people complained and this feature is currently not configurable. We
   should introduce a QStyle::SH_Widget_Animation_Duration hint that would
   allow us to not hardcode durations in Dolphin (i.e. respect whatever
   duration the users set in their QStyle).
   Proposal in https://codereview.qt-project.org/#/c/195712/

Reviewers: #vdg, #plasma, emmanuelp, davidedmundson

Reviewed By: #plasma, davidedmundson

Differential Revision: https://phabricator.kde.org/D5883
2017-05-31 14:07:16 +02:00
Elvis Angelaccio 1faef1c92d Introduce DolphinMainWindowTest
For now it contains a test case for the crash in bug #379135

CCBUG: 379135

Reviewers: emmanuelp, dfaure

Differential Revision: https://phabricator.kde.org/D5936
2017-05-27 10:28:09 +02:00
Elvis Angelaccio 4216c7e6ba Move non-exported code to a static library
This allows us to speed up the compilation because we don't need to
build twice the source files we use in the unit tests.

Test Plan: Builds, dolphin works and tests pass.

Reviewers: emmanuelp, dfaure

Differential Revision: https://phabricator.kde.org/D5935
2017-05-27 10:23:47 +02:00
l10n daemon script 4d9c396a98 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"
2017-05-25 04:21:54 +02:00
Luca Beltrame 95f9dc9de0
Merge branch 'Applications/17.04' 2017-05-22 23:32:36 +02:00
Anthony Fieroni 43da84eefc Correct searchbox, split view transitions between tabs
Differential Revision: https://phabricator.kde.org/D5864

BUG: 379135
BUG: 380032
FIXED-IN: 17.04.2

Signed-off-by: Anthony Fieroni <bvbfan@abv.bg>
2017-05-22 20:35:29 +03:00
Pino Toscano 588db66536 add few more Q_DECL_OVERRIDE 2017-05-12 00:07:15 +02:00
Hannah von Reth e0f6b74226 Port tests from ecm_mark_as_test to ecm_add_test
Summary: This enables us to run the tests on Windows.

Reviewers: elvisangelaccio, kfunk, dfaure

Reviewed By: elvisangelaccio

Subscribers: #konqueror, #dolphin

Differential Revision: https://phabricator.kde.org/D5810
2017-05-11 20:05:25 +02:00
Elvis Angelaccio f6095c86c1 Fix kfileitemmodeltest
Commit 6dcbb8127c introduced the isHidden role, update the test
accordingly.
2017-05-11 19:52:14 +02:00
Hannah von Reth 6c3cacd2b3 Remove define to disable exports
As the tests are now correctly linked to dolphinprivate we need the
actual export macro.
2017-05-11 12:33:15 +02:00
Elvis Angelaccio 0c8ef2977e Merge branch 'Applications/17.04' 2017-05-08 16:09:15 +02:00
Elvis Angelaccio 73f885f9d3 Don't ignore tag clicks in the tooltips
Summary:
Now that we can use the metadata widgets in the tooltips, we can also
open the tags:// url if the user clicks some tag in a tooltip.

The behavior is now consistent with the metadata widget in the information panel.

Test Plan: Click a tag when the metadata tooltip shows up.

Reviewers: emmanuelp

Subscribers: #konqueror, #dolphin

Differential Revision: https://phabricator.kde.org/D5658
2017-05-08 16:04:19 +02:00