Bug 378701 - Dolphin views: Option to remove preview frames per-directory
Summary: Dolphin views: Option to remove preview frames per-directory
Status: CONFIRMED
Alias: None
Product: dolphin
Classification: Applications
Component: view-engine: icons mode (show other bugs)
Version: 17.03.80
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Dolphin Bug Assignee
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-12 14:42 UTC by andre.cbarros
Modified: 2023-05-23 13:45 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
Add the option to disable frames with shadows (21.63 KB, patch)
2017-04-12 14:42 UTC, andre.cbarros
Details

Note You need to log in before you can comment on or make changes to this bug.
Description andre.cbarros 2017-04-12 14:42:14 UTC
Created attachment 104992 [details]
Add the option to disable frames with shadows

I've been patching dolphin for a long time to remove the shadow it puts around icons on some directories. Never submitted it because it is still not 100% functional, it fails to immediately refresh the "view" mode and I need to change to some other directory and go back to have it honored. Anyway, it is useful and may be easily corrected by someone more familiarized with the code tree. Also, it does not introduce bugs and respect user option for each directory affected.

If it is refused, I would appreciate just the correction to the fault behavior (i.e., the refresh issue).

Best regards
André
Comment 1 Elvis Angelaccio 2017-04-12 18:08:24 UTC
Please submit the patch here: https://phabricator.kde.org/differential/diff/create/
Comment 2 andre.cbarros 2017-04-13 02:12:42 UTC
I could not figure out how to register to post, if you may give me some hint it would be awesome.

André
Comment 3 andre.cbarros 2017-04-13 02:25:34 UTC
Never mind, disabled protection plugins on my browser and registered.
Comment 4 Nate Graham 2017-09-04 04:36:18 UTC
*** Bug 295526 has been marked as a duplicate of this bug. ***
Comment 5 Nate Graham 2017-09-04 04:36:42 UTC
Yes indeed, please submit the patch. There seems to be some interest in this, judging by the duplicates of https://bugs.kde.org/show_bug.cgi?id=295526
Comment 6 Christoph Feck 2017-09-13 20:39:21 UTC
The patch was submitted, see https://phabricator.kde.org/D5420