Bug 168480

this might be a kdelibs issue, as I found out that it happens in kopete too.

Furthermore settings icon size in systemsettings has no effect.

steps to reproduce:

1. Go to systemsettings->appeareance->icons->advanced->"main toolbar"
2. Change the icons to something else than the default.
3. Open your kopete, kontact whatever and notice that the icons still have the default size :(

I can confirm that this happens in any kde application. This must be a kdelibs bug. Setting the default toolbar icon size from systemsettings doesn't work too. KMainWindow seems to always load 22x22 icons without checking the settings.

One part of the problem is now fixed in trunk... http://websvn.kde.org/?view=rev&revision=849092

Toolbars now remember their settings. However, they still don't respect the size from systemsettings (see comment 2).

The bug report for the "not respecting system settings" bug is here:

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

I don't know if its a config mess probblem here, but with current trunk, applications still do not remember the icon size... :(

I found out that the remembering only works if the icon size is set to default size (22) in systemsettings for toolbars. Otherwise this bug still exists. So there is a mess somewhere in systemsettings.

I hope someone can sort out this until the release of 4.2.

This seems to be a dup of https://bugs.kde.org/show_bug.cgi?id=167280

*** Bug 167280 has been marked as a duplicate of this bug. ***

This bug is still present. The problem is that if the toolbar size that is set in systemsettings as the default is chosen in an application, when the application restarts, it loads the hard-coded 22x22 default.

To reproduce:
1) Set the size to 32x32 in systemsettings. Note that the applications will still use 22x22 (that's bug 169067).
2) Open konqueror and set the size to 16x16 or 48x48 and notice that if you close it and reopen it, it will remember it.
3) Now set the size to 32x32 in konqueror (which is the same as chosen in systemsettings!), close it and reopen it. You will notice that it loads in 22x22.

Confirmed with kde 4.2 trunk r884287.

*** Bug 178018 has been marked as a duplicate of this bug. ***

Here using:

Qt: 4.4.3 + qt-copy-patches-889120
KDE: 4.1.86 (KDE 4.1.86 (KDE 4.2 >= 20081221))
kdelibs svn rev. 904448 / kdebase svn rev. 904448
on ArchLinux x86_64 - Kernel 2.6.27.10

I can't reproduce the bug following the steps in comment 10.
@George: can you still reproduce it with an updated build ?

Also, this report can be marked as duplicate of bug 152266

*** Bug 152266 has been marked as a duplicate of this bug. ***

(In reply to comment #13)
> Also, this report can be marked as duplicate of bug 152266

... or vice versa, as bug 152266 is not very clear to me and mixes up many issues in the comments... :)

The bug is still reproducable in svn rev 903389, however I forgot to mention in comment 10 that if you are testing the bug with the main toolbar of the application, you have to change the size setting for "Main Toolbar" from systemsettings or else the bug doesn't appear. Apparently the main toolbar of every application has different settings from all the other toolbars....

I've checked the bug report against KDE 4.2.0 and it seems it's still not fixed. 

The toolbar icon size from systemsettings is ignored by all applications. Changing the icon size for one application works until the application is restarted, after the restart the new settings are lost.

Ok, I'm going to have another stab at fixing this and related bugs, and I'll ask for help on kdecore to see if we can get some more experienced brains than mine interested.  I'm completely unable to reproduce this particular part though:

"Changing the icon size for one application works until the application is
restarted"

so I'll probably be asking for a list of applications this affects (I've tried it with Dolphin, Konqueror and Kwrite so far, and all worked correctly - the changed sizes were remembered when restarting the applications) and maybe some config files.

The only feedback I can add is that I'm a real-estate freak so I have set ALL icon sizes in settings to 16, except those on the desktop (32).
What I get in applications is 22 and if I set them to 16 by hand, they're reset to 22 the next time the app starts.

On a sidenote: the 'Text position: Only icons' setting is also not respected. All taskbars show icons+text to start with. But if I change *that* setting in the application it *is* remembered the next time.

Ok, test number #1:

- Open Dolphin from the command-line, using
  dolphin --nofork
- Right-click the Main Toolbar -> Icon Size -> Huge 48x48
- Close Dolphin.
- Re-open, again from the command-line.
- Verify that the size of the Icons are still Huge 48x48.

[this works for me]

If not, please attach your kdeglobals and dolphinrc (delete any sensitive data that you have in there, if you want) - they will both be in your $KDEHOME/share/config directory ($KDEHOME is usually either ~/.kde or ~/.kde4).

That's funny! It remembers the 48x48 setting (point taken) but not the 16x16 setting.
After setting it to 16x16 back and restarting, the menu context says it is 'default' (which is correct-by-settings) but renders it as 22x22, which *might* be KDE default, but certainly not *my* default... I think some defaults are getting messed up along the way.

Do you still need my kdeglobals and dolphinrc after this?

@Martin:

"Do you still need my kdeglobals and dolphinrc after this?"

Yes, please :)

Done some more tests, and as far as I can tell the icon size for each application (as in test from comment #10) is not saved if it's the same as the global settings size.

here are my two tests:

test #1:
- System Settings -> Appearance -> Icons -> Advanced -> Main Toolbar is 16!
- Open Dolphin from the command-line, using dolphin --nofork
- Right-click the Main Toolbar -> Icon Size -> Huge 48x48 
- Close Dolphin
- Re-open, again from the command-line
- Verify that the size of the Icons are still Huge 48x48 (ok)

test #1 is ok for me

test #2:
- System Settings -> Appearance -> Icons -> Advanced -> Main Toolbar is 48!
- Open Dolphin from the command-line, using dolphin --nofork
- Right-click the Main Toolbar -> Icon Size -> Huge 48x48 
- Close Dolphin
- Re-open, again from the command-line
- Verify that the size of the Icons are still Huge 48x48 (failed)

test #2 is not ok for me, dolphin starts with 22x icons

Created attachment 30591 [details]
dolphinrc

As requested, dolphinrc, censored for personal data

Created attachment 30592 [details]
kdeglobals

And kdeglobals...

@Daniel:

Good analysis, and this matches precisely what I'm seeing in the code.  I'll have to manually keep track of whether an IconSize is the Default (probably by using the logic "If the size does not appear in the config file, or the user has explicitly selected the 'Default' entry in the menu, then make note that the icon size should track the default value"), as the current method:

"
   if (!cg.hasDefault("IconSize") && iconSize().width() == iconSizeDefault())
    cg.revertToDefault("IconSize");
  else
    cg.writeEntry("IconSize", iconSize().width());
"

is logically broken.  This means that this "not getting saved" bug is merely a simple consequence of this dodgy logic coupled with the bug in http://bugs.kde.org/show_bug.cgi?id=169067, which is good news :)

@Martin:

Great, that should be good enough to be getting on with.  Thanks to both of your for your help: If you are compiling from trunk at all, I might ask you to test patches as we go along! :) No ETA on a fix, yet, though.

I depend on Jaunty packages, so that would be next RC or 4.2 release for me I'm afraid.

Created attachment 30607 [details]
ktoolbar.cpp.patch

I've been browsing trough the ktoolbar.cpp code, and I think I managed to find a simple fix for this bug, please see the attached patch (against trunk). Someone who knows that code better than me should review it and commit if ok. I've done several tests and didn't manage to find any new problems.

@Simon, according to the API docs for KConfigGroup::hasDefault() that's the way to do it

@Daniel:

That's certainly going to be part of a fix (I had a similar thing here:

http://bugs.kde.org/show_bug.cgi?id=169067#c4

) but getting everything working together, including properly respecting the meaning of the word "Default" and adapting to changes made to global settings in System Settings, will probably need to be a bit more involved, I think :/

*** Bug 182564 has been marked as a duplicate of this bug. ***

I wrote an answer to a similar bug two days ago; but now I can't find it. I'm rather new to bugs.kde, so perhaps I did something wrong or I'm just unable to find it or the bug report was deleted, who knows.
Anyway, what I said in that report was more or less what I said here, a rather simple workaround:
http://forum.kde.org/-solved-kickoff-and-toolbars-icon-size-t-29186.html

I hope it helps.

Cheers.

Sorry, I copied the whole thread link in KDE forum. The direct link for the workaround is this one:
http://forum.kde.org/-solved-kickoff-and-toolbars-icon-size-t-29186.html#pid40364

This problem is still present in 4.2.1, is a fix planned for the 4.2 series?

I'd love to fix it, but there's still some weird stuff in there that I don't fully understand:

http://www.nabble.com/KToolbar-Confusion-td21731850.html

I'll send a follow-up e-mail with more specific questions at some point, but I will be swamped with real-life work for the foreseeable future so if anyone else wants to step up, please do :)

*** Bug 189348 has been marked as a duplicate of this bug. ***

*** Bug 190778 has been marked as a duplicate of this bug. ***

SVN commit 968441 by sstjames:

There seems to be some difficulty with specifying KConfig::Global when we are already using kdeglobals, in that it never writes out the settings, so do what the icons kcm does and use KGlobal::config()  - as dfaure points out, this has the advantage of being more efficient, anyway.  This is related to the following bugs (iconText and iconSize have much the same root causes), which I am also currently trying to fix:

CCBUG:168480 
CCBUG:169067

 M  +6 -6      kcmstyle.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=968441

*** Bug 193618 has been marked as a duplicate of this bug. ***

   
SVN commit 986786 by dfaure:

Rework the handling of settings (icon size and tool-button-style (e.g. text under icons)) in KToolBar, in order to obey the following priority order at all times:
KDE-Global config < App-XML attributes < user settings in KConfig < in-memory XML attributes
In particular, this fixes changing the kde-global icon size for toolbars, which is now applied to both running and future apps
 (had to add a signal to KIconLoader for the first case).
With unit tests (quite complete for icon size; to be finished for tool-button-style; to be written for hiding/showing)
Not sure yet if this will be in 4.3, depends on the number of regressions found in trunk...
BUG: 169067


 M  +1 -0      icons/kiconloader.cpp  
 M  +6 -0      icons/kiconloader.h  
 M  +1 -0      tests/CMakeLists.txt  
 A             tests/ktoolbar_unittest.cpp   [License: LGPL (v2/3+eV)]
 M  +3 -0      tests/ktoolbartest.cpp  
 M  +23 -76    tests/kxmlgui_unittest.cpp  
 M  +1 -6      widgets/kmainwindow.cpp  
 M  +2 -2      widgets/kmainwindow.h  
 M  +251 -332  widgets/ktoolbar.cpp  
 M  +37 -30    widgets/ktoolbar.h  
 M  +1 -4      xmlgui/kxmlguibuilder.cpp  


http://websvn.kde.org/?view=rev&revision=986786

*** Bug 175212 has been marked as a duplicate of this bug. ***

Seems fixed for me in 4.3RC2, Thx!!

Kate still suffers from this bug. See #204796

*** Bug 198984 has been marked as a duplicate of this bug. ***

This bug is still present in Kate (KDE 4.5.4, see bug 204796)

Martin I run trunk (next 4.6) and this is fixed. In System Settings I changed the icon size to 48 for the main toolbar and immediately after Applying all toolbars icons went to size 48, included Kate.

For you so, is it only Kate not taking the global icon size? System Settings itself should change as soon as you apply a new size.

As far as I remember this discussion, the problem was linked to my icons being (extremely) small: I use 16 for all icons, except desktop and this patch fixed all applications *except* Kate! That's why I openened bug 204796, but nobody's listening there... I can set icons to default in KAte, which makes them small (16x16) as I want, but after closing Kate and reopening it resorts back to medium (22x22), see comment #20. The difference now is that Kate doesn't think it displays default, but knowingly puts it back to 22x22 before my eyes.