fix UX state after button press on dynamic modal

This bug was hidden by the previous return of state before
the reliquish.

Now with the correct ordering, we end up getting a double-relinquish
which ends up trying to swap a modal into focus which doesn't exist.

This fixes that edge case

services/gam/src/modal/notification.rs

@@ -194,8 +194,10 @@ impl ActionApi for Notification {
             }
             _ => {
                 // relinquish focus before returning the result
-                self.gam.relinquish_focus().unwrap();
-                xous::yield_slice();
+                if self.manual_dismiss {
+                    self.gam.relinquish_focus().unwrap();
+                    xous::yield_slice();
+                }
 
                 send_message(
                     self.action_conn,