M src/Configmenu.cc → src/Configmenu.cc

@@ -91,8 +91,7 @@ return;
 
   switch(item->function()) {
   case 1: { // dither
-    screen.getImageControl()->
-      setDither((! screen.getImageControl()->doDither()));
+    screen.setImageDither(!screen.getImageControl()->doDither());
 
     setItemSelected(index, screen.getImageControl()->doDither());
 

M src/Screen.cc → src/Screen.cc

@@ -243,8 +243,6 @@ openbox.getCacheLife(), openbox.getCacheMax());
   image_control->installRootColormap();
   root_colormap_installed = True;
 
-  image_control->setDither(resource.image_dither);
-
   load();       // load config options from Resources
   LoadStyle();
 
@@ -852,12 +850,13 @@ 
 
 void BScreen::setImageDither(bool d, bool reconfig) {
   resource.image_dither = d;
+  image_control->setDither(d);
   std::ostrstream s;
   s << "session.screen" << getScreenNumber() << ".imageDither" << ends;
   config.setValue(s.str(), resource.image_dither);
   s.rdbuf()->freeze(0);
   if (reconfig)
-    openbox.reconfigure();
+    reconfigure();
 }