ExternalTemplate.cpp:

[lyx.git] / src / insets / RenderGraphic.cpp

diff --git a/src/insets/RenderGraphic.cpp b/src/insets/RenderGraphic.cpp
index 3520290a973bff05ae6a1c44fa2e355fbd114187..df1a93a1804f565301945bf8b5c60a94ef899a74 100644 (file)
--- a/src/insets/RenderGraphic.cpp
+++ b/src/insets/RenderGraphic.cpp
@@ -38,16 +38,13 @@ namespace lyx {
 RenderGraphic::RenderGraphic(Inset const * inset)
 {
        loader_.connect(boost::bind(&Inset::updateFrontend, inset));
-       icon_.connect(boost::bind(&Inset::updateFrontend, inset));
 }
 
 
 RenderGraphic::RenderGraphic(RenderGraphic const & other, Inset const * inset)
-       : RenderBase(other), loader_(other.loader_), icon_(other.icon_),
-               params_(other.params_)
+       : RenderBase(other), loader_(other.loader_), params_(other.params_)
 {
        loader_.connect(boost::bind(&Inset::updateFrontend, inset));
-       icon_.connect(boost::bind(&Inset::updateFrontend, inset));
 }
 
 
@@ -63,15 +60,6 @@ void RenderGraphic::update(graphics::Params const & params)
 
        if (!params_.filename.empty())
                loader_.reset(params_.filename, params_);
-       // If icon is set to empty, icon_ will not be reset to empty
-       // but will not be displayed. This is to avoid repeated loading
-       // of the same icon when figure status changes.
-       if (!params_.icon.empty()) {
-               support::FileName const icon = support::libFileSearch("images/",
-                       params_.icon, "png");
-               if (!icon.empty()) // using an empty bounding box
-                       icon_.reset(icon, graphics::Params());
-       }
 }
 
 
@@ -79,8 +67,7 @@ namespace {
 
 bool displayGraphic(graphics::Params const & params)
 {
-       return params.display != graphics::NoDisplay &&
-               lyxrc.display_graphics != graphics::NoDisplay;
+       return params.display && lyxrc.display_graphics;
 }
 
 
@@ -145,53 +132,51 @@ bool readyToDisplay(graphics::Loader const & loader)
 
 void RenderGraphic::metrics(MetricsInfo & mi, Dimension & dim) const
 {
+       if (displayGraphic(params_)) {
+               if (loader_.status() == graphics::WaitingToLoad)
+                       loader_.startLoading();
+               if (!loader_.monitoring())
+                       loader_.startMonitoring();
+       }
+
        bool image_ready = displayGraphic(params_) && readyToDisplay(loader_);
+       if (image_ready) {
+               dim.wid = loader_.image()->width() + 2 * Inset::TEXT_TO_INSET_OFFSET;
+               dim.asc = loader_.image()->height();
+               dim_ = dim;
+               return;
+       }
 
        dim.asc = image_ready ? loader_.image()->height() : 50;
        dim.des = 0;
 
-       if (image_ready) {
-               dim.wid = loader_.image()->width() + 2 * Inset::TEXT_TO_INSET_OFFSET;
-       } else {
-               int font_width = 0;
-
-               FontInfo msgFont(mi.base.font);
-               msgFont.setFamily(SANS_FAMILY);
+       int font_width = 0;
 
-               // FIXME UNICODE
-               docstring const justname = from_utf8(params_.filename.onlyFileName());
-               if (!justname.empty()) {
-                       msgFont.setSize(FONT_SIZE_FOOTNOTE);
-                       font_width = theFontMetrics(msgFont).width(justname);
-               }
+       FontInfo msgFont(mi.base.font);
+       msgFont.setFamily(SANS_FAMILY);
 
-               docstring const msg = statusMessage(params_, loader_.status());
-               if (!msg.empty()) {
-                       msgFont.setSize(FONT_SIZE_TINY);
-                       font_width = max(font_width,
-                               theFontMetrics(msgFont).width(msg));
-               }
+       // FIXME UNICODE
+       docstring const justname = from_utf8(params_.filename.onlyFileName());
+       if (!justname.empty()) {
+               msgFont.setSize(FONT_SIZE_FOOTNOTE);
+               font_width = theFontMetrics(msgFont).width(justname);
+       }
 
-               dim.wid = max(50, font_width + 15);
+       docstring const msg = statusMessage(params_, loader_.status());
+       if (!msg.empty()) {
+               msgFont.setSize(FONT_SIZE_TINY);
+               font_width = max(font_width,
+                       theFontMetrics(msgFont).width(msg));
        }
 
+       dim.wid = max(50, font_width + 15);
+
        dim_ = dim;
 }
 
 
 void RenderGraphic::draw(PainterInfo & pi, int x, int y) const
 {
-       if (displayGraphic(params_)) {
-               if (loader_.status() == graphics::WaitingToLoad)
-                       loader_.startLoading();
-               if (!loader_.monitoring())
-                       loader_.startMonitoring();
-               if (icon_.status() == graphics::WaitingToLoad)
-                       icon_.startLoading();
-               if (!icon_.monitoring())
-                       icon_.startMonitoring();
-       }
-
        // This will draw the graphics. If the graphics has not been
        // loaded yet, we draw just a rectangle.
 
@@ -229,9 +214,6 @@ void RenderGraphic::draw(PainterInfo & pi, int x, int y) const
                                     y - 4, msg, msgFont);
                }
        }
-       if (!params_.icon.empty() && readyToDisplay(icon_))
-               pi.pain.image(x + Inset::TEXT_TO_INSET_OFFSET, y - dim_.asc, 
-                       10, 10, *icon_.image());
 }