Revert the logic of 343a9749abe4f3

It might be better to be explicit also with black, after all.

diff --git a/src/insets/InsetBox.cpp b/src/insets/InsetBox.cpp
index 1643a0df52ce48e9491998aa85bcd7c8c9db86fb..edfa6a569c0cd06c7d75fb5dac518bcd28e8fe43 100644 (file)
--- a/src/insets/InsetBox.cpp
+++ b/src/insets/InsetBox.cpp
@@ -899,10 +899,9 @@ string const InsetBox::getBackgroundColor() const
 
 bool InsetBox::useFColorBox() const
 {
-       // we only need an \fcolorbox if the framecolor is something else
-       // than black in the output or if the backgroundcolor is not none
-       // (also needed with white, consider non-white page coloring)
-       return getFrameColor() != "black" || params_.backgroundcolor != "none";
+       // we need an \fcolorbox if the framecolor or the backgroundcolor
+       // is non-default. We also do it with black and white for consistency.
+       return params_.framecolor != "default" || params_.backgroundcolor != "none";
 }