Improve grouped media display.

Telegram/SourceFiles/boxes/send_files_box.cpp

@@ -492,6 +492,17 @@ EditCaptionBox::EditCaptionBox(QWidget*, HistoryMedia *media, FullMsgId msgId) :
 		image = doc->thumb;
 	} break;
 
+	case MediaTypeGrouped: {
+		if (const auto photo = media->getPhoto()) {
+			dimensions = QSize(photo->full->width(), photo->full->height());
+			image = photo->full;
+		} else if (const auto doc = media->getDocument()) {
+			dimensions = doc->dimensions;
+			image = doc->thumb;
+			_animated = true;
+		}
+	} break;
+
 	case MediaTypeFile:
 	case MediaTypeMusicFile:
 	case MediaTypeVoiceFile: {

Telegram/SourceFiles/data/data_shared_media.cpp

@@ -290,9 +290,7 @@ base::optional<bool> SharedMediaWithLastSlice::IsLastIsolated(
 		| [](FullMsgId msgId) {	return App::histItemById(msgId); }
 		| [](HistoryItem *item) { return item ? item->getMedia() : nullptr; }
 		| [](HistoryMedia *media) {
-			return (media && media->type() == MediaTypePhoto)
+			return media ? media->getPhoto() : nullptr;
-				? static_cast<HistoryPhoto*>(media)->getPhoto()
-				: nullptr;
 		}
 		| [](PhotoData *photo) { return photo ? photo->id : 0; }
 		| [&](PhotoId photoId) { return *lastPeerPhotoId != photoId; };

Telegram/SourceFiles/history/history_media_grouped.cpp

@@ -426,14 +426,12 @@ bool HistoryGroupedMedia::computeNeedBubble() const {
 	if (!_caption.isEmpty()) {
 		return true;
 	}
-	for (const auto &element : _elements) {
+	if (const auto message = _parent->toHistoryMessage()) {
-		if (const auto message = element.item->toHistoryMessage()) {
+		if (message->viaBot()
-			if (message->viaBot()
+			|| message->Has<HistoryMessageReply>()
-				|| message->Has<HistoryMessageForwarded>()
+			|| message->displayForwardedFrom()
-				|| message->Has<HistoryMessageReply>()
+			|| message->displayFromName()) {
-				|| message->displayFromName()) {
+			return true;
-				return true;
-			}
 		}
 	}
 	return false;

Telegram/SourceFiles/history/history_media_types.cpp

@@ -640,9 +640,9 @@ void HistoryPhoto::drawGrouped(
 		const auto roundRadius = ImageRoundRadius::Large;
 		const auto roundCorners = ImageRoundCorner::None
 			| ((corners & RectPart::TopLeft) ? ImageRoundCorner::TopLeft : ImageRoundCorner::None)
-			| ((corners & RectPart::TopRight) ? ImageRoundCorner::TopLeft : ImageRoundCorner::None)
+			| ((corners & RectPart::TopRight) ? ImageRoundCorner::TopRight : ImageRoundCorner::None)
-			| ((corners & RectPart::BottomLeft) ? ImageRoundCorner::TopLeft : ImageRoundCorner::None)
+			| ((corners & RectPart::BottomLeft) ? ImageRoundCorner::BottomLeft : ImageRoundCorner::None)
-			| ((corners & RectPart::BottomRight) ? ImageRoundCorner::TopLeft : ImageRoundCorner::None);
+			| ((corners & RectPart::BottomRight) ? ImageRoundCorner::BottomRight : ImageRoundCorner::None);
 		App::complexOverlayRect(p, geometry, roundRadius, roundCorners);
 	}
 
@@ -857,8 +857,8 @@ bool HistoryPhoto::needsBubble() const {
 	}
 	if (auto message = _parent->toHistoryMessage()) {
 		return message->viaBot()
-			|| message->Has<HistoryMessageForwarded>()
 			|| message->Has<HistoryMessageReply>()
+			|| message->displayForwardedFrom()
 			|| message->displayFromName();
 	}
 	return false;
@@ -1333,8 +1333,8 @@ bool HistoryVideo::needsBubble() const {
 	}
 	if (auto message = _parent->toHistoryMessage()) {
 		return message->viaBot()
-			|| message->Has<HistoryMessageForwarded>()
 			|| message->Has<HistoryMessageReply>()
+			|| message->displayForwardedFrom()
 			|| message->displayFromName();
 	}
 	return false;
@@ -2809,8 +2809,8 @@ bool HistoryGif::needsBubble() const {
 	}
 	if (auto message = _parent->toHistoryMessage()) {
 		return message->viaBot()
-			|| message->Has<HistoryMessageForwarded>()
 			|| message->Has<HistoryMessageReply>()
+			|| message->displayForwardedFrom()
 			|| message->displayFromName();
 	}
 	return false;
@@ -5318,8 +5318,8 @@ bool HistoryLocation::needsBubble() const {
 	}
 	if (auto message = _parent->toHistoryMessage()) {
 		return message->viaBot()
-			|| message->Has<HistoryMessageForwarded>()
 			|| message->Has<HistoryMessageReply>()
+			|| message->displayForwardedFrom()
 			|| message->displayFromName();
 	}
 	return false;

Telegram/SourceFiles/history/history_message.cpp

@@ -125,6 +125,7 @@ bool HasMediaItems(const HistoryItemsList &items) {
 			switch (media->type()) {
 			case MediaTypePhoto:
 			case MediaTypeVideo:
+			case MediaTypeGrouped:
 			case MediaTypeFile:
 			case MediaTypeMusicFile:
 			case MediaTypeVoiceFile: return true;

Telegram/SourceFiles/history/history_message.h

@@ -163,6 +163,7 @@ public:
 		if (isAttachedToPrevious()) return false;
 		return true;
 	}
+	bool displayForwardedFrom() const;
 	bool displayEditedBadge(bool hasViaBotOrInlineMarkup) const;
 	bool uploading() const;
 	bool displayRightAction() const override;
@@ -321,7 +322,6 @@ private:
 	int performResizeGetHeight();
 	void applyEditionToEmpty();
 
-	bool displayForwardedFrom() const;
 	void paintFromName(Painter &p, QRect &trect, bool selected) const;
 	void paintForwardedInfo(Painter &p, QRect &trect, bool selected) const;
 	void paintReplyInfo(Painter &p, QRect &trect, bool selected) const;

Telegram/SourceFiles/history/history_service.cpp

@@ -286,6 +286,9 @@ HistoryService::PreparedText HistoryService::preparePinnedText() {
 			switch (media ? media->type() : MediaTypeCount) {
 			case MediaTypePhoto: return lang(lng_action_pinned_media_photo);
 			case MediaTypeVideo: return lang(lng_action_pinned_media_video);
+			case MediaTypeGrouped: return lang(media->getPhoto()
+				? lng_action_pinned_media_photo
+				: lng_action_pinned_media_video);
 			case MediaTypeContact: return lang(lng_action_pinned_media_contact);
 			case MediaTypeFile: return lang(lng_action_pinned_media_file);
 			case MediaTypeGif: {

Telegram/SourceFiles/history/history_widget.cpp

@@ -4513,8 +4513,8 @@ void HistoryWidget::onThumbDocumentUploaded(
 
 void HistoryWidget::onPhotoProgress(const FullMsgId &newId) {
 	if (const auto item = App::histItemById(newId)) {
-		const auto photo = (item->getMedia() && item->getMedia()->type() == MediaTypePhoto)
+		const auto photo = item->getMedia()
-			? static_cast<HistoryPhoto*>(item->getMedia())->getPhoto()
+			? item->getMedia()->getPhoto()
 			: nullptr;
 		updateSendAction(item->history(), SendAction::Type::UploadPhoto, 0);
 		Auth().data().requestItemRepaint(item);

Telegram/SourceFiles/info/media/info_media_list_widget.cpp

@@ -847,10 +847,8 @@ std::unique_ptr<BaseLayout> ListWidget::createLayout(
 		return nullptr;
 	}
 	auto getPhoto = [&]() -> PhotoData* {
-		if (auto media = item->getMedia()) {
+		if (const auto media = item->getMedia()) {
-			if (media->type() == MediaTypePhoto) {
+			return media->getPhoto();
-				return static_cast<HistoryPhoto*>(media)->getPhoto();
-			}
 		}
 		return nullptr;
 	};