diff -r b023a8d2866a -r ea65f74e6de4 photosgallery/viewframework/dataprovider/src/glxdetailsboundcommand.cpp
--- a/photosgallery/viewframework/dataprovider/src/glxdetailsboundcommand.cpp	Thu Jul 15 18:39:01 2010 +0300
+++ b/photosgallery/viewframework/dataprovider/src/glxdetailsboundcommand.cpp	Thu Aug 19 09:55:03 2010 +0300
@@ -17,11 +17,14 @@
 
 
 
+#include    "glxdetailsboundcommand.hrh"
+#include    "glxdetailsboundcommand.h"                  // Handles user commands
+#include    "glxuiutility.h"
+/** Error Id EMPY-7MKDHP **/ 
+#include    "mglxmetadatadialogobserver.h"              //for call back to dailog
 
 #include    <e32base.h>                                 // Base class for all classes to be instantiated on the heap
 #include    <ExifModify.h>                              //For CExifModify
-#include    "glxdetailsboundcommand.hrh"
-#include    "glxdetailsboundcommand.h"                  // Handles user commands
 #include    <glxmetadatadialog.rsg>
 #include    <glxuiutilities.rsg>
 #include    <glxuiutilities.rsg>
@@ -37,16 +40,11 @@
 #include    <glxlog.h>                                  // Logging
 #include    <glxtracer.h>
 #include    <glxviewbase.rsg>
-#include    "glxuiutility.h"
-#include <glxscreenfurniture.h>
-#include <glxresourceutilities.h>                // for CGlxResourceUtilities
-/** Error Id EMPY-7MKDHP **/ 
-#include 	"mglxmetadatadialogobserver.h" 				//for call back to dailog
+#include    <glxscreenfurniture.h>
+#include    <glxresourceutilities.h>                    // for CGlxResourceUtilities
+#include    <glxmediageneraldefs.h>                     // for KMaxMediaPopupTextLength
 
 
-//CONSTANTS
-const TInt KMaxMediaPopupTitleLength = 0x100;
-
 // ----------------------------------------------------------------------------
 // CGlxTitleBoundCommand::NewL
 // ----------------------------------------------------------------------------
@@ -91,11 +89,11 @@
 	        {
 	        ///@todo - remove popup when editable template is avaliable
 	        //Popup for testing
-	        HBufC* textBuf = HBufC::NewLC( KMaxMediaPopupTitleLength );
+	        HBufC* textBuf = HBufC::NewLC( KMaxMediaPopupTextLength );
 	        (textBuf->Des()).Copy((media->ValueText(KMPXMediaGeneralTitle)));
 	        TPtr textPtr = textBuf->Des();
 	        /// bug fix - EMPY-7MCKD6 
-	        TBuf<KMaxMediaPopupTitleLength> titleText(*textBuf);
+	        TBuf<KMaxMediaPopupTextLength> titleText(*textBuf);
 	        /// bug fix - EMPY-7MCKD6
 	        ///@todo - remove literal for popup title when resource string is avaliable
 			HBufC *buf = StringLoader::LoadLC(R_GLX_METADATA_VIEW_TITLE_NSERIES);
@@ -386,11 +384,11 @@
 		{
 		///@todo - remove popup when editable template is avaliable
 		//Popup for testing
-		HBufC* textBuf = HBufC::NewLC( KMaxMediaPopupTitleLength );
+		HBufC* textBuf = HBufC::NewLC( KMaxMediaPopupTextLength );
 		(textBuf->Des()).Copy((media->ValueText(KMPXMediaGeneralComment)));
 		TPtr textPtr = textBuf->Des();
 		/// bug fix - EMPY-7MCKD6
-		TBuf<KMaxMediaPopupTitleLength> descText(*textBuf);
+		TBuf<KMaxMediaPopupTextLength> descText(*textBuf);
 		/// bug fix - EMPY-7MCKD6
 		///@todo - remove literal for popup title when resource string is avaliable
 		
@@ -1022,7 +1020,6 @@
 void CGlxUsageRightsBoundCommand::InitMenuL( CEikMenuPane& aMenu ) const
 	{
 	TRACER("CGlxUsageRightsBoundCommand::InitMenuL");
-	aMenu.SetItemTextL( KGlxViewBoundMenuCommandId, R_GLX_METADATA_VIEW_OPTIONS_VIEW ); 
 	aMenu.SetItemDimmed( KGlxViewBoundMenuCommandId, EFalse );
 	}