Update address confirm UX + confirmation strings.

src/getPublicKeys_ui.c

@@ -107,7 +107,7 @@ void getPublicKeys_respondOneKey_ui_runStep()
 		if (ctx->currentPath == ctx->numPaths) {
 			#ifdef HAVE_NBGL
 			if (!ctx->silent_export) {
-				display_status("PUBLIC KEY\nEXPORTED");
+				display_status("Public key\nexported");
 			}
 			#endif // HAVE_NBGL
 			keys_advanceStage();

src/signTxOutput_ui.c

@@ -450,7 +450,7 @@ void signTxOutput_handleConfirm_ui_runStep()
 		char msg[100] = {0};
 		snprintf(msg, SIZEOF(msg), "%s\n%s", subctx->ui_text1, subctx->ui_text2);
 		ASSERT(strlen(msg) + 1 < SIZEOF(msg));
-		display_confirmation(msg, "",  "OUTPUT\nCONFIRMED", "Output\nrejected", this_fn, respond_with_user_reject);
+		display_confirmation(msg, "",  "Output\nconfirmed", "Output\nrejected", this_fn, respond_with_user_reject);
 		#endif // HAVE_BAGL
 	}
 	UI_STEP(HANDLE_CONFIRM_STEP_RESPOND) {

src/signTx_ui.c

@@ -1692,6 +1692,6 @@ void signTx_handleWitness_ui_runStep()
 void endTxStatus(void)
 {
 	#ifdef HAVE_NBGL
-	display_status("TRANSACTION\nSIGNED");
+	display_status("Transaction\nsigned");
 	#endif // HAVE_NBGL
 }

src/ui_nbgl.c

@@ -32,8 +32,15 @@ enum {
 	CONFIRMATION_STATUS_TOKEN,
 };
 
+typedef enum {
+    STATUS_TYPE_TRANSACTION,
+    STATUS_TYPE_ADDRESS,
+} statusType_t;
+
 typedef struct {
-	const char* confirmedStatus; // text displayed in confirmation page (after long press)
+	bool standardStatus;
+    statusType_t statusType;
+    const char* confirmedStatus; // text displayed in confirmation page (after long press)
 	const char* rejectedStatus;  // text displayed in rejection page (after reject confirmed)
 	callback_t approvedCallback;
 	callback_t rejectedCallback;
@@ -123,6 +130,8 @@ static void reset_transaction_current_context(void)
 void nbgl_reset_transaction_full_context(void)
 {
 	reset_transaction_current_context();
+    uiContext.standardStatus = false;
+    uiContext.statusType = STATUS_TYPE_TRANSACTION;
 	uiContext.pendingElement = 0;
 	uiContext.lightConfirmation = false;
 	uiContext.rejectedStatus = NULL;
@@ -251,7 +260,14 @@ static void display_cancel_status(void)
 	if (uiContext.rejectedStatus) {
 		nbgl_useCaseStatus(uiContext.rejectedStatus, false, cancellation_status_callback);
 	} else {
-		nbgl_useCaseStatus("Transaction rejected", false, cancellation_status_callback);
+        switch (uiContext.statusType) {
+            case STATUS_TYPE_TRANSACTION:
+                nbgl_useCaseReviewStatus(STATUS_TYPE_TRANSACTION_REJECTED, cancellation_status_callback);
+                break;
+            case STATUS_TYPE_ADDRESS:
+                nbgl_useCaseReviewStatus(STATUS_TYPE_ADDRESS_REJECTED, cancellation_status_callback);
+                break;
+        }
 	}
 }
 
@@ -347,6 +363,16 @@ static void confirmation_status_callback(void)
 	if (uiContext.confirmedStatus) {
 		nbgl_useCaseStatus(uiContext.confirmedStatus, true, ui_idle_flow);
 		nbgl_reset_transaction_full_context();
+    } else if(uiContext.standardStatus) {
+        switch (uiContext.statusType) {
+            case STATUS_TYPE_TRANSACTION:
+                nbgl_useCaseReviewStatus(STATUS_TYPE_TRANSACTION_SIGNED, ui_idle_flow);
+                break;
+            case STATUS_TYPE_ADDRESS:
+                nbgl_useCaseReviewStatus(STATUS_TYPE_ADDRESS_VERIFIED, ui_idle_flow);
+                break;
+        }
+        nbgl_reset_transaction_full_context();
 	} else {
 		nbgl_useCaseSpinner("Processing");
 	}
@@ -364,35 +390,6 @@ static void display_confirmation_status(void)
 	}
 }
 
-static void display_address_callback(void)
-{
-	uint8_t address_index = 0;
-
-	// Address field is not displayed in pairList, so there is one element less.
-	uiContext.pairList.nbPairs = uiContext.currentElementCount - 1;
-	uiContext.pairList.pairs = tagValues;
-
-	uiContext.confirmedStatus = "ADDRESS\nVERIFIED";
-	uiContext.rejectedStatus = "Address verification\ncancelled";
-
-	for (uint8_t i = 0; i < uiContext.currentElementCount; i++) {
-		if (strcmp(uiContext.tagTitle[i], "Address")) {
-			tagValues[i].item = uiContext.tagTitle[i];
-			tagValues[i].value = uiContext.tagContent[i];
-		} else {
-			address_index = i;
-		}
-	}
-
-	nbgl_useCaseAddressConfirmationExt(uiContext.tagContent[address_index], light_confirm_callback, &uiContext.pairList);
-	reset_transaction_current_context();
-
-	#ifdef HEADLESS
-	nbgl_refresh();
-	trigger_callback(&display_confirmation_status);
-	#endif
-}
-
 static void trigger_callback(callback_t userAcceptCallback)
 {
 	set_app_callback(userAcceptCallback);
@@ -545,15 +542,37 @@ void display_choice(const char* text1, const char* text2, callback_t userAcceptC
 void display_address(callback_t userAcceptCallback, callback_t userRejectCallback)
 {
 	TRACE("Displaying Address");
-	uiContext.rejectedStatus = "Address verification\ncancelled";
-
 	set_callbacks(userAcceptCallback, userRejectCallback);
-	nbgl_useCaseReviewStart(&C_cardano_64, "Verify Cardano\naddress",
-	                        NULL, "Cancel", display_address_callback,
-	                        display_cancel_status);
+
+    uint8_t address_index = 0;
+
+	// Address field is not displayed in pairList, so there is one element less.
+	uiContext.pairList.nbPairs = uiContext.currentElementCount - 1;
+	uiContext.pairList.pairs = tagValues;
+
+    uiContext.standardStatus = true;
+    uiContext.statusType = STATUS_TYPE_ADDRESS;
+
+	for (uint8_t i = 0; i < uiContext.currentElementCount; i++) {
+		if (strcmp(uiContext.tagTitle[i], "Address")) {
+			tagValues[i].item = uiContext.tagTitle[i];
+			tagValues[i].value = uiContext.tagContent[i];
+		} else {
+			address_index = i;
+		}
+	}
+
+    nbgl_useCaseAddressReview(uiContext.tagContent[address_index],
+                              &uiContext.pairList,
+                              &C_cardano_64,
+                              "Verify Cardano address",
+                              NULL,
+                              light_confirm_callback);
+	reset_transaction_current_context();
+
 	#ifdef HEADLESS
 	nbgl_refresh();
-	trigger_callback(&display_address_callback);
+	trigger_callback(&display_confirmation_status);
 	#endif
 }