Do not draw avatar if not visible on screen

1 files changed, 27 insertions, 26 deletions

diff --git a/src/MessageBoard.cpp b/src/MessageBoard.cpp
index 5e101aa..c56419d 100644
--- a/src/MessageBoard.cpp
+++ b/src/MessageBoard.cpp
@@ -124,38 +124,39 @@ namespace dchat
                         timestamp.setPosition(sf::Vector2f(floor(position.x + (AVATAR_DIAMETER + AVATAR_PADDING_SIDE) * Settings::getScaling() + usernameText.getLocalBounds().width + USERNAME_TIMESTAMP_SIDE_PADDING * Settings::getScaling()), floor(position.y + 2.0f * Settings::getScaling() + usernameTextHeight * 0.5f - timestampTextHeight * 0.5f)));
                         window.draw(timestamp);
                     }
-                }
-                
-                // Max avatar size = 1mb
-                const ContentByUrlResult avatarResult = cache.getContentByUrl(message->user->avatarUrl, 1024 * 1024);
-                if(avatarResult.type == ContentByUrlResult::Type::CACHED)
-                {
-                    sf::Shader *circleShader = ResourceCache::getShader("shaders/circleMask.glsl", sf::Shader::Fragment);
-                    circleShader->setUniform("texture", sf::Shader::CurrentTexture);
                     
-                    if(avatarResult.cachedType == ContentByUrlResult::CachedType::TEXTURE)
+                    // Max avatar size = 1mb
+                    const ContentByUrlResult avatarResult = cache.getContentByUrl(message->user->avatarUrl, 1024 * 1024);
+                    if(avatarResult.type == ContentByUrlResult::Type::CACHED)
                     {
-                        // TODO: Store this sprite somewhere, might not be efficient to create a new sprite object every frame
-                        sf::Sprite sprite(*avatarResult.texture);
-                        auto textureSize = avatarResult.texture->getSize();
-                        sprite.setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
-                        sprite.setScale(sf::Vector2f(AVATAR_DIAMETER * Settings::getScaling() / (float)textureSize.x, AVATAR_DIAMETER * Settings::getScaling() / (float)textureSize.y));
-                        window.draw(sprite, circleShader);
+                        sf::Shader *circleShader = ResourceCache::getShader("shaders/circleMask.glsl", sf::Shader::Fragment);
+                        circleShader->setUniform("texture", sf::Shader::CurrentTexture);
+                        
+                        if(avatarResult.cachedType == ContentByUrlResult::CachedType::TEXTURE)
+                        {
+                            // TODO: Store this sprite somewhere, might not be efficient to create a new sprite object every frame
+                            sf::Sprite sprite(*avatarResult.texture);
+                            auto textureSize = avatarResult.texture->getSize();
+                            sprite.setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
+                            sprite.setScale(sf::Vector2f(AVATAR_DIAMETER * Settings::getScaling() / (float)textureSize.x, AVATAR_DIAMETER * Settings::getScaling() / (float)textureSize.y));
+                            window.draw(sprite, circleShader);
+                        }
+                        else if(avatarResult.cachedType == ContentByUrlResult::CachedType::GIF)
+                        {
+                            auto gifSize = avatarResult.gif->getSize();
+                            avatarResult.gif->setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
+                            avatarResult.gif->setScale(sf::Vector2f(AVATAR_DIAMETER * Settings::getScaling() / (float)gifSize.x, AVATAR_DIAMETER * Settings::getScaling() / (float)gifSize.y));
+                            avatarResult.gif->draw(window, circleShader);
+                        }
                     }
-                    else if(avatarResult.cachedType == ContentByUrlResult::CachedType::GIF)
+                    else
                     {
-                        auto gifSize = avatarResult.gif->getSize();
-                        avatarResult.gif->setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
-                        avatarResult.gif->setScale(sf::Vector2f(AVATAR_DIAMETER * Settings::getScaling() / (float)gifSize.x, AVATAR_DIAMETER * Settings::getScaling() / (float)gifSize.y));
-                        avatarResult.gif->draw(window, circleShader);
+                        sf::CircleShape avatarCircle(AVATAR_DIAMETER * 0.5f * Settings::getScaling(), 60 * Settings::getScaling());
+                        avatarCircle.setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
+                        avatarCircle.setFillColor(ColorScheme::getBackgroundColor() + sf::Color(30, 30, 30));
+                        window.draw(avatarCircle);
                     }
                 }
-                else
-                {
-                    sf::CircleShape avatarCircle(AVATAR_DIAMETER * 0.5f * Settings::getScaling(), 60 * Settings::getScaling());
-                    avatarCircle.setPosition(sf::Vector2f(floor(position.x), floor(position.y)));
-                    window.draw(avatarCircle);
-                }
                 
                 position.y += usernameTextHeight + USERNAME_PADDING_BOTTOM * Settings::getScaling();
             }