Cardview update: It is more compressed. Pending: Add pricture for work- and home address..

1 files changed, 9 insertions, 9 deletions

diff --git a/libopie2/opiepim/ocontact.cpp b/libopie2/opiepim/ocontact.cpp
index 3473baa..e34feeb 100644
--- a/libopie2/opiepim/ocontact.cpp
+++ b/libopie2/opiepim/ocontact.cpp
@@ -469,37 +469,37 @@ QString OContact::toRichText() const
 	text += "<b><img src=\"addressbook/email\"> " + QObject::tr("Default Email: ") + "</b>"
 		+ Qtopia::escapeString(defEmail);
 	marker = true;
     }
 
     // business address
     if ( !businessStreet().isEmpty() || !businessCity().isEmpty() ||
 	 !businessZip().isEmpty() || !businessCountry().isEmpty() ) {
-	text += QObject::tr( "<br><br><b>Work Address:</b>" );
+	text += QObject::tr( "<br><b>Work Address:</b>" );
 	marker = true;
     }
 
     if ( !(value = businessStreet()).isEmpty() ){
 	    text += "<br>" + Qtopia::escapeString(value);
 	    marker = true;
     }
 
     switch( addressformat ){
     case Zip_City_State:{ //  Zip_Code City, State
 	    state =  businessState();
 	    if ( !(value = businessZip()).isEmpty() ){
 		    text += "<br>" + Qtopia::escapeString(value) + " ";
 		    marker = true;
 		    
-	    } else
-		    text += "<br>";
-
+	    }
 	    if ( !(value = businessCity()).isEmpty() ) {
 		    marker = true;
+		    if ( businessZip().isEmpty() && !businessStreet().isEmpty() )
+			    text += "<br>";
 		    text += Qtopia::escapeString(value);
 		    if ( state )
 			    text += ", " + Qtopia::escapeString(state);
 	    } else if ( !state.isEmpty() ){
 		    text += "<br>" + Qtopia::escapeString(state);
 		    marker = true;
 	    }
 	    break;
@@ -561,41 +561,41 @@ QString OContact::toRichText() const
     }
     str = businessPager();
     if ( !str.isEmpty() ){
 	text += "<br><b>" + QObject::tr("Business Pager: ") + "</b>"
 		+ Qtopia::escapeString(str);
 	marker = true;
     }
 
-    text += "<br>";
+    // text += "<br>";
 
     // home address
     if ( !homeStreet().isEmpty() || !homeCity().isEmpty() ||
 	 !homeZip().isEmpty() || !homeCountry().isEmpty() ) {
-	text += QObject::tr( "<br><br><b>Home Address:</b>" );
+	text += QObject::tr( "<br><b>Home Address:</b>" );
 	marker = true;
     }
 
     if ( !(value = homeStreet()).isEmpty() ){
 	text += "<br>" + Qtopia::escapeString(value);
 	marker = true;
     }
 
     switch( addressformat ){
     case Zip_City_State:{ //  Zip_Code City, State
 	    state =  homeState();
 	    if ( !(value = homeZip()).isEmpty() ){
 		    text += "<br>" + Qtopia::escapeString(value) + " ";
 		    marker = true;
-	    } else
-		    text += "<br>";
-
+	    }
 	    if ( !(value = homeCity()).isEmpty() ) {
 		    marker = true;
+		    if ( homeZip().isEmpty() && !homeStreet().isEmpty() )
+			    text += "<br>";
 		    text += Qtopia::escapeString(value);
 		    if ( !state.isEmpty() )
 			    text += ", " + Qtopia::escapeString(state);
 	    } else if (!state.isEmpty()) {
 		    text += "<br>" + Qtopia::escapeString(state);
 		    marker = true;
 	    }
 	    break;