By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.
Bug 1268 - Konqueror: No GUI control exists for controlling whether to display archives as folders
Summary: Konqueror: No GUI control exists for controlling whether to display archives ...
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: tdebase (show other bugs)
Version: 3.5.13 [Trinity]
Hardware: All All
: P5 enhancement
Assignee: Timothy Pearson
URL:
Depends on:
Blocks:
 
Reported: 2012-10-13 06:01 CDT by Jan Stolarek
Modified: 2013-03-02 21:06 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
Konqeror's tree view (31.83 KB, image/png)
2012-10-13 06:01 CDT, Jan Stolarek 		Details
Partial patch to provide a GUI check box (3.00 KB, patch)
2012-10-13 20:46 CDT, Darrell 		Details | Diff
Patch ot provide a GUI check box for controlling the display of archived files (3.19 KB, patch)
2012-10-14 01:45 CDT, Darrell 		Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Stolarek 2012-10-13 06:01:31 CDT 
Created attachment 878 [details]
Konqeror's tree view

Konqueror displays archives as folders in the tree view in the side pane on the right. There should be an option (and I am sure there was such option in KDE3) to disable this behaviour.
Comment 1 Darrell 2012-10-13 15:57:45 CDT 
The option does exist. :)

This is Yet Another 'Hidden' Feature carry over from KDE3. To me, 'hidden' features are usability bugs:

https://bugs.kde.org/show_bug.cgi?id=98070

To fix the problem, open $TDEHOME/share/config/konqsidebartng.rc with a text editor. Then at the top of the file add the following:

ShowArchivesAsFolders=false

Do not add the option to any groups. Just add the entry at the top of the rc file.

You might need to close all konqueror instances, possibly even restart Trinity.

With that work-around, I'm changing the bug report summary to address adding the GUI control.

Good catch and thanks for reporting.
Comment 2 Darrell 2012-10-13 20:46:08 CDT 
Created attachment 883 [details]
Partial patch to provide a GUI check box

I started a patch to provide a check box to control whether konqueror displays archived files as folders. Whereas I can get the check box to show in konqueror settings, the build fails when I add lines to read from or write to the rc file. The build failure message:

tdebase/kcontrol/konq/behaviour.cpp:235: error: 'cbshowArchivesAsFolders' was not declared in this scope
tdebase/kcontrol/konq/behaviour.cpp:260: error: 'cbshowArchivesAsFolders' was not declared in this scope

The problematic lines:

235: cbshowArchivesAsFolders->setChecked( sidebarconfig.readBoolEntry( "showArchivesAsFolders", false ) );

260: sidebarconfig.writeEntry( "ShowArchivesAsFolders", cbshowArchivesAsFolders->isChecked() );
Comment 3 Darrell 2012-10-13 22:41:22 CDT 
Odd. I use tree view and archived files do not appear as folders. The default is true. After editing konqsidebartng.rc to true (no such option in konqsidebartng_kicker.rc), I still don't have archived files as folders. I tested further with a fresh profile.

Is there some other magic needed to get archived files to appear as folders?
Comment 4 Darrell 2012-10-13 23:54:58 CDT 
Oh, never mind. I was looking at the right side file pane to view the archived files as folders. The feature only works in the left side navigation sidebar.

Hopefully somebody can help with the patch.
Comment 5 Darrell 2012-10-14 01:45:36 CDT 
Created attachment 884 [details]
Patch ot provide a GUI check box for controlling the display of archived files

The problem with the first patch is I can't read. I had a lowercase s rather than an uppercase S in the variable name. Camel case crap.

The updated patch works in Slackware 13.1 32-bit. I plan to test in Slackware 14.0 64-bit too.

The new check box control is located in Settings->Configure Konqueror->Behavior->Misc Options. In kcontrol, TDE Components->File Manager->Behavior.

Whether making the change from kcontrol or konqueror, any open instance of Konqueror needs to be closed and restarted to see the change. Trinity does not need to be restarted.

The patch can be pushed to GIT after somebody reviews or tests.
Comment 6 Jan Stolarek 2012-10-16 03:01:23 CDT 
(In reply to comment #1)
> To fix the problem, open $TDEHOME/share/config/konqsidebartng.rc with a text
> editor. Then at the top of the file add the following:
> 
> ShowArchivesAsFolders=false
Thanks! This works :)
Comment 7 Darrell 2013-03-02 21:06:39 CST 
Patch in attachment 884 [details] pushed to GIT in commit 7255491d. Resolved.