ACME-626 : Simplify the preview drawing code and positioning

1 files changed, 5 insertions, 48 deletions

diff --git a/indra/newview/llfloatersocial.cpp b/indra/newview/llfloatersocial.cpp
index a0dbb7ea95..fce9d1b28f 100644
--- a/indra/newview/llfloatersocial.cpp
+++ b/indra/newview/llfloatersocial.cpp
@@ -178,76 +178,33 @@ void LLSocialPhotoPanel::draw()
 { 
 	LLSnapshotLivePreview * previewp = static_cast<LLSnapshotLivePreview *>(mPreviewHandle.get());
 
-	LLPanel::draw();
-
-	if(previewp && previewp->getThumbnailImage())
+	if (previewp && previewp->getThumbnailImage())
 	{
-		bool working = false; //impl.getStatus() == Impl::STATUS_WORKING;
 		const LLRect& thumbnail_rect = mThumbnailPlaceholder->getRect();
+		const LLRect& snapshot_rect = mSnapshotPanel->getRect();
 		const S32 thumbnail_w = previewp->getThumbnailWidth();
 		const S32 thumbnail_h = previewp->getThumbnailHeight();
 
 		// calc preview offset within the preview rect
 		const S32 local_offset_x = (thumbnail_rect.getWidth() - thumbnail_w) / 2 ;
-		const S32 local_offset_y = (thumbnail_rect.getHeight() - thumbnail_h) / 2 ; // preview y pos within the preview rect
 
 		// calc preview offset within the floater rect
 		S32 offset_x = thumbnail_rect.mLeft + local_offset_x;
-		S32 offset_y = thumbnail_rect.mBottom + local_offset_y;
+		S32 offset_y = thumbnail_rect.mBottom - (snapshot_rect.mTop - thumbnail_rect.mTop);
 
 		mSnapshotPanel->localPointToOtherView(offset_x, offset_y, &offset_x, &offset_y, getParentByType<LLFloater>());
 
 		gGL.matrixMode(LLRender::MM_MODELVIEW);
 		// Apply floater transparency to the texture unless the floater is focused.
 		F32 alpha = getTransparencyType() == TT_ACTIVE ? 1.0f : getCurrentTransparency();
-		LLColor4 color = working ? LLColor4::grey4 : LLColor4::white;
+		LLColor4 color = LLColor4::white;
 		gl_draw_scaled_image(offset_x, offset_y, 
 			thumbnail_w, thumbnail_h,
 			previewp->getThumbnailImage(), color % alpha);
 
 		previewp->drawPreviewRect(offset_x, offset_y) ;
-
-		// Draw some controls on top of the preview thumbnail.
-		static const S32 PADDING = 5;
-		static const S32 REFRESH_LBL_BG_HEIGHT = 32;
-
-		// Reshape and position the posting result message panels at the top of the thumbnail.
-		// Do this regardless of current posting status (finished or not) to avoid flicker
-		// when the result message is displayed for the first time.
-		// if (impl.getStatus() == Impl::STATUS_FINISHED)
-		{
-			LLRect result_lbl_rect = mSucceessLblPanel->getRect();
-			const S32 result_lbl_h = result_lbl_rect.getHeight();
-			result_lbl_rect.setLeftTopAndSize(local_offset_x, local_offset_y + thumbnail_h, thumbnail_w - 1, result_lbl_h);
-			mSucceessLblPanel->reshape(result_lbl_rect.getWidth(), result_lbl_h);
-			mSucceessLblPanel->setRect(result_lbl_rect);
-			mFailureLblPanel->reshape(result_lbl_rect.getWidth(), result_lbl_h);
-			mFailureLblPanel->setRect(result_lbl_rect);
-		}
-
-		// Position the refresh button in the bottom left corner of the thumbnail.
-		mRefreshBtn->setOrigin(local_offset_x + PADDING, local_offset_y + PADDING);
-
-		if (mNeedRefresh)
-		{
-			// Place the refresh hint text to the right of the refresh button.
-			const LLRect& refresh_btn_rect = mRefreshBtn->getRect();
-			mRefreshLabel->setOrigin(refresh_btn_rect.mLeft + refresh_btn_rect.getWidth() + PADDING, refresh_btn_rect.mBottom);
-
-			// Draw the refresh hint background.
-			LLRect refresh_label_bg_rect(offset_x, offset_y + REFRESH_LBL_BG_HEIGHT, offset_x + thumbnail_w - 1, offset_y);
-			gl_rect_2d(refresh_label_bg_rect, LLColor4::white % 0.9f, TRUE);
-		}
-
-		gGL.pushUIMatrix();
-		S32 x_pos;
-		S32 y_pos;
-		mSnapshotPanel->localPointToOtherView(thumbnail_rect.mLeft, thumbnail_rect.mBottom, &x_pos, &y_pos, getParentByType<LLFloater>());
-
-		LLUI::translate((F32) x_pos, (F32) y_pos);
-		mThumbnailPlaceholder->draw();
-		gGL.popUIMatrix();
 	}
+	LLPanel::draw();
 }
 
 LLSnapshotLivePreview* LLSocialPhotoPanel::getPreviewView()