Update address confirm UX + confirmation strings.

.github/workflows/ci.yml

@@ -42,7 +42,7 @@ jobs:
       - name : Install yarn
         run: npm install --global yarn
       - name: Install Cardano JS Library (Ledger fork)
-        run: git clone https://github.com/LedgerHQ/ledgerjs-cardano-shelley.git
+        run: git clone https://github.com/vacuumlabs/ledgerjs-cardano-shelley.git
       - name : Run tests
         run: |
           BUILD_DEVICE_NAME="$(echo ${{ matrix.device }} | sed 's/nanosp/nanos2/')"

Makefile

@@ -29,8 +29,8 @@ APPNAME      = "Cardano ADA"
 
 # Application version
 APPVERSION_M = 7
-APPVERSION_N = 2
-APPVERSION_P = 0
+APPVERSION_N = 1
+APPVERSION_P = 2
 APPVERSION   = "$(APPVERSION_M).$(APPVERSION_N).$(APPVERSION_P)"
 
 # Application source files
@@ -56,7 +56,7 @@ VARIANT_VALUES = cardano_ada
 
 DEFINES += RESET_ON_CRASH
 # Enabling DEBUG flag will enable PRINTF and disable optimizations
-# DEVEL = 1
+DEVEL = 1
 # DEFINES += HEADLESS
 
 # Enabling debug PRINTF

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,9 +32,16 @@ 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)
-	const char* rejectedStatus;  // text displayed in rejection page (after reject confirmed)
+	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;
 	callback_t pendingDisplayPageFn;
@@ -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;
@@ -250,8 +259,18 @@ 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);
+	}
+	else {
+		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 +366,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 +393,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 +545,34 @@ 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
 }