Bug 1683

I always use "other (any)." :-)

From the KControl Reorganization Effort etherpad (http://trinity.etherpad.trinitydesktop.org/40):

* Window Behavior -> Advanced Tab -> Window Actions tab

* Screen Saver -> Advanced Options -> to parent dialog

* Appearance & Themes -> Background -> Advanced Options -> parent dialog.

Created attachment 1730 [details]
KControl user interface patch

These two patches moves the "tabbed browsing" options of Konqueror from the Advanced dialog to a new category called "User interace". I didn't call it "Tabbed browsing" because we could add more options to the same category later. Anyhow the options are contained inside a group called "Tabbed Browsing".

The AdvancedTab*** files have been removed and instead userInter*** files have been added.

After you commit the patch to GIT, please mark again this bug as "NEW", since there are other advanced options that needs to be moved somewhere else later.

Created attachment 1731 [details]
Konqueror patch

Created attachment 1732 [details]
sample screenshot

Michele, the konqy patches look very nice! :-)

I did not test every check box, but of the several I tested all functioned as expected.

> I did not test every check box, but of the several I tested all functioned as
> expected.

I also didn't test all options, but the 4 or 5 I tried all worked ok.
Anyhow the code for those options is exactly the same, so I don't expect problems on that side :)

Created attachment 1814 [details]
updated KControl patch, after KDE->TDE rename in commit 26dbe18e on 2013-12-26

General note: the konqueror patch works fine and I have been using since the patch was available. But the patch should not be pushed to git until the help handbook is updated. :)

(In reply to comment #9)
> General note: the konqueror patch works fine and I have been using since the
> patch was available. But the patch should not be pushed to git until the help
> handbook is updated. :)

We are in the process of reviewing the handbooks, this will not be the only change required to the Konqueror handbook for sure.
So I would say that if Slavek/Tim want to push the patch, they should go ahead. When we review the related handbook, we just add the documentation for this as well.

We need bug 1813 resolved with the new status option of DOCREVIEW. Then we can push patches like this and have a way to track required handbook updates.

Before pushing patch to GIT I want to test one thing - whether patch cause that all texts will be in this moment untranslated. I know for english speakers this is not essential. But for other languages ​​it represents another piece which will be not translated in R14.0.0 against 3.5.13.x.

(In reply to comment #12)
> Before pushing patch to GIT I want to test one thing - whether patch cause that
> all texts will be in this moment untranslated. I know for english speakers this
> is not essential. But for other languages ​​it represents another piece which
> will be not translated in R14.0.0 against 3.5.13.x.

In theory it should still be ok. I am using the same entities and strings that  were in the Advanced Tab, so translation should remain the same.

Created attachment 1886 [details]
KControl patch for tdehtml / ui

Looks good, translations are as they should be.

Just one correction - m_pHoverCloseButton is not enabled / disabled immediately when switching m_pPermanentCloseButton.

I added a proper bookmark in the handbook for the new User Interface module. I renamed the desktop file from tdehtml_userInter.desktop to tdehtml_userinterface.desktop to be consistent with the naming of other desktop files.

I performed only nominal editing in the handbook. Generally, the handbook is like most handbooks and is out of date and inaccurate.

Patches pushed to git in commit a0e3f4ae.

The patch does not close the bug report because this is a general bug report for all "Advanced" dialogs.

With a new profile, the checkbox "Confirm when closing windows with multiple tabs" will always remain unchecked. Uncheck the option, select the Apply button, close the dialog, immediately reopen the dialog, and the option is enabled.

(In reply to comment #16)
Thanks Darrell. Bug confirmed: it always remains unchecked.
I think this bug may have been there even before this patch. I remember also that some options do not change their behavior until Konqueror is restarted, which is a bug as well.
We will have to address all these small issues.

The bug is not in 3.5.13.2 and not in an R14 package set from Jan 2013 (I tested in a VM we used for other testing). The bug is not exactly as I previously shared.

The bug depends upon the current configuration. When disabled in the config file then the user cannot enable. Conversely, when the option is enabled then the user cannot disable.

The only work-around is manually edit the config file.

Another Advanced dialog:

Konqueror -> Properties -> Permissions tab -> Advanced Permissions

Yet Another Extra Click and hiding the information is silly because there are other ways to find the same information, even in Konqueror.

> Yet Another Extra Click and hiding the information is silly because there
> are other ways to find the same information, even in Konqueror.
I don't think it was about hiding information.
Anyhow, well spotted (as usual). I think the "advanced permissions" would be good as an extra tab, such to have: General. Permissions, Advanced permissions, Preview.
How do you think?

Seems another tab substitutes the tab for the Advanced button. Still requires an additonal click.

The Advanced Permissions dialog can be shrunk. After shrinking, replacing the button with the dialog places all of the information in one place, in one group box. No additional tab needed. That would mean a subsequent larger Properties dialog default size because all tabs would have to be the same size as the Permissions tab.