diff --git a/src/TopRoomBar.cpp b/src/TopRoomBar.cpp index ffd57d50ac01d3e749d1dc0ecad96996233f7ebe..a45a751e75b5ed1ab15a1e153e963e973c1da2d6 100644 --- a/src/TopRoomBar.cpp +++ b/src/TopRoomBar.cpp @@ -196,13 +196,6 @@ TopRoomBar::updateRoomAvatar(const QString &avatar_image) update(); } -void -TopRoomBar::updateRoomAvatar(const QIcon &icon) -{ - avatar_->setIcon(icon); - update(); -} - void TopRoomBar::updateRoomName(const QString &name) { diff --git a/src/TopRoomBar.h b/src/TopRoomBar.h index 63ce847e13cd6788789ab3010932285138f16879..1aa5934b6fd0040dba9c79c641185581dfd289a2 100644 --- a/src/TopRoomBar.h +++ b/src/TopRoomBar.h @@ -29,7 +29,6 @@ class OverlayModal; class QPainter; class QLabel; -class QIcon; class QHBoxLayout; class QVBoxLayout; @@ -43,7 +42,6 @@ public: TopRoomBar(QWidget *parent = nullptr); void updateRoomAvatar(const QString &avatar_image); - void updateRoomAvatar(const QIcon &icon); void updateRoomName(const QString &name); void updateRoomTopic(QString topic); void updateRoomAvatarFromName(const QString &name); diff --git a/src/ui/Avatar.cpp b/src/ui/Avatar.cpp index 7ee58d03c7ad2ac60bf392d0cf3df5f05075df7b..b8703e8778dcc04adaae0e4ccc3b827adb16dcff 100644 --- a/src/ui/Avatar.cpp +++ b/src/ui/Avatar.cpp @@ -72,6 +72,8 @@ Avatar::setImage(const QString &avatar_url) static_cast<int>(size_ * pixmap_.devicePixelRatio()), this, [this](QPixmap pm) { + if (pm.isNull()) + return; type_ = ui::AvatarType::Image; pixmap_ = pm; update(); @@ -88,20 +90,14 @@ Avatar::setImage(const QString &room, const QString &user) static_cast<int>(size_ * pixmap_.devicePixelRatio()), this, [this](QPixmap pm) { + if (pm.isNull()) + return; type_ = ui::AvatarType::Image; pixmap_ = pm; update(); }); } -void -Avatar::setIcon(const QIcon &icon) -{ - icon_ = icon; - type_ = ui::AvatarType::Icon; - update(); -} - void Avatar::paintEvent(QPaintEvent *) { @@ -135,13 +131,6 @@ Avatar::paintEvent(QPaintEvent *) } switch (type_) { - case ui::AvatarType::Icon: { - icon_.paint(&painter, - QRect((width() - hs) / 2, (height() - hs) / 2, hs, hs), - Qt::AlignCenter, - QIcon::Normal); - break; - } case ui::AvatarType::Image: { QPainterPath ppath; diff --git a/src/ui/Avatar.h b/src/ui/Avatar.h index 2910350d17ced310796a6b5133c8cb8353c55be9..da8a57edd1fa05bd63715f1668713b6b9aa74418 100644 --- a/src/ui/Avatar.h +++ b/src/ui/Avatar.h @@ -1,6 +1,5 @@ #pragma once -#include <QIcon> #include <QImage> #include <QPixmap> #include <QWidget> @@ -18,7 +17,6 @@ public: explicit Avatar(QWidget *parent = nullptr, int size = ui::AvatarSize); void setBackgroundColor(const QColor &color); - void setIcon(const QIcon &icon); void setImage(const QString &avatar_url); void setImage(const QString &room, const QString &user); void setLetter(const QString &letter); @@ -40,7 +38,6 @@ private: QString avatar_url_, room_, user_; QColor background_color_; QColor text_color_; - QIcon icon_; QPixmap pixmap_; int size_; }; diff --git a/src/ui/Theme.h b/src/ui/Theme.h index ecff02b5bccb179c4639df00945d2d6d116969d1..349712800f213ceb280ec252799bc3e09fae43ca 100644 --- a/src/ui/Theme.h +++ b/src/ui/Theme.h @@ -7,7 +7,6 @@ namespace ui { enum class AvatarType { - Icon, Image, Letter };