From 473a701bf503d53f58112849f303542e5ea0f427 Mon Sep 17 00:00:00 2001
From: Magne Sjaastad <magne.sjaastad@ceetronsolutions.com>
Date: Tue, 26 Aug 2014 13:34:20 +0200
Subject: [PATCH] Result info : Improved display of text

---
 ApplicationCode/ProjectDataModel/RimReservoirView.cpp | 8 +++++---
 ApplicationCode/ProjectDataModel/RimReservoirView.h   | 2 +-
 ApplicationCode/UserInterface/RiuViewer.cpp           | 4 ++--
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/ApplicationCode/ProjectDataModel/RimReservoirView.cpp b/ApplicationCode/ProjectDataModel/RimReservoirView.cpp
index 6e434b06c6..b36883f60d 100644
--- a/ApplicationCode/ProjectDataModel/RimReservoirView.cpp
+++ b/ApplicationCode/ProjectDataModel/RimReservoirView.cpp
@@ -1035,7 +1035,7 @@ RiuViewer* RimReservoirView::viewer()
 //--------------------------------------------------------------------------------------------------
 /// Get pick info text for given part ID, face index, and intersection point
 //--------------------------------------------------------------------------------------------------
-bool RimReservoirView::pickInfo(size_t gridIndex, size_t cellIndex, cvf::StructGridInterface::FaceType face, const cvf::Vec3d& point, QString* pickInfoText) const
+bool RimReservoirView::pickInfo(size_t gridIndex, size_t cellIndex, cvf::StructGridInterface::FaceType face, const cvf::Vec3d& point, QString itemSeparator, QString* pickInfoText) const
 {
     CVF_ASSERT(pickInfoText);
 
@@ -1060,10 +1060,12 @@ bool RimReservoirView::pickInfo(size_t gridIndex, size_t cellIndex, cvf::StructG
                 
                 QString faceText = faceEnum.text();
 
-                *pickInfoText = QString("Hit grid %1, cell [%2, %3, %4] face %5, ").arg(gridIndex).arg(i).arg(j).arg(k).arg(faceText);
+                *pickInfoText += QString("Hit grid %1").arg(gridIndex) + itemSeparator;
+                *pickInfoText += QString("Cell : [%1, %2, %3]").arg(i).arg(j).arg(k) + itemSeparator;
+                *pickInfoText += QString("Face : %1").arg(faceText) + itemSeparator;
 
                 QString formattedText;
-                formattedText.sprintf("intersection point: [E: %.2f, N: %.2f, Depth: %.2f]", domainCoord.x(), domainCoord.y(), -domainCoord.z());
+                formattedText.sprintf("Intersection point : [E: %.2f, N: %.2f, Depth: %.2f]", domainCoord.x(), domainCoord.y(), -domainCoord.z());
 
                 *pickInfoText += formattedText;
                 return true;
diff --git a/ApplicationCode/ProjectDataModel/RimReservoirView.h b/ApplicationCode/ProjectDataModel/RimReservoirView.h
index 261df2cbad..e0f77fbf31 100644
--- a/ApplicationCode/ProjectDataModel/RimReservoirView.h
+++ b/ApplicationCode/ProjectDataModel/RimReservoirView.h
@@ -159,7 +159,7 @@ public:
 
 
     // Picking info
-    bool                                    pickInfo(size_t gridIndex, size_t cellIndex, cvf::StructGridInterface::FaceType face, const cvf::Vec3d& point, QString* pickInfoText) const;
+    bool                                    pickInfo(size_t gridIndex, size_t cellIndex, cvf::StructGridInterface::FaceType face, const cvf::Vec3d& point, QString itemSeparator, QString* pickInfoText) const;
     void                                    appendCellResultInfo(size_t gridIndex, size_t cellIndex, cvf::StructGridInterface::FaceType face, QString* resultInfoText) ;
     void                                    appendNNCResultInfo(size_t nncIndex, QString* resultInfo);
     static void                             appendTextFromResultSlot(RigCaseData* eclipseCase, size_t gridIndex, size_t cellIndex, size_t timeStepIndex, RimResultSlot* resultSlot, QString* resultInfoText);
diff --git a/ApplicationCode/UserInterface/RiuViewer.cpp b/ApplicationCode/UserInterface/RiuViewer.cpp
index 29476f1678..e0654348a0 100644
--- a/ApplicationCode/UserInterface/RiuViewer.cpp
+++ b/ApplicationCode/UserInterface/RiuViewer.cpp
@@ -416,10 +416,10 @@ void RiuViewer::handlePickAction(int winPosX, int winPosY)
                        
                         cvf::StructGridInterface::FaceType face = rivSourceInfo->m_cellFaceFromTriangleMapper->cellFace(faceIndex);
 
-                        m_reservoirView->pickInfo(gridIndex, cellIndex, face, localIntersectionPoint, &pickInfo);
+                        m_reservoirView->pickInfo(gridIndex, cellIndex, face, localIntersectionPoint, ", ", &pickInfo);
 
                         // Build up result from from both pick info and result values
-                        m_reservoirView->pickInfo(gridIndex, cellIndex, face, localIntersectionPoint, &resultInfo);
+                        m_reservoirView->pickInfo(gridIndex, cellIndex, face, localIntersectionPoint, "\n", &resultInfo);
                         resultInfo += "\n";
                         m_reservoirView->appendCellResultInfo(gridIndex, cellIndex, face, &resultInfo);
 #if 0