Bugzilla

(In reply to Alessandro Castellani (:aleca) from comment #43)
> > Have you addressed the concern form comment #10, that is, moving through the drop-down for the identities already triggers the function?
> 
> Yes, I have.
> As I wrote in comment 39 , the `onRecipientsChanged` method is necessary to handle the send button status if the user has any prefilled Cc or Bcc fields per identity.
> I moved it in order to be triggered only if the identity is actually changed and doesn't match the current one.

That sounds like an improvement, but it doesn't sound enough to fix the concern of comment 10, described in bug 1635124.
The problem is that navigating up and down the list with cursor up/down will instantly apply the navigated identity although the user has not yet confirme it with ENTER. So if I cursor-navigate from identity1 to identity5, i understand that `onRecipientsChanged()` will still needlessly fire 4 times because the dropdown widget applies the options prematurely. As a side effect, customize-from also fires prematurely on navigation, so keyboard-only users will find it very hard if not impossible to customize the from address of any identity other than the last.

(In reply to Alessandro Castellani (:aleca) from comment #43)
> > Have you addressed the concern form comment #10, that is, moving through the drop-down for the identities already triggers the function?
> 
> Yes, I have.
> As I wrote in comment 39 , the `onRecipientsChanged` method is necessary to handle the send button status if the user has any prefilled Cc or Bcc fields per identity.
> I moved it in order to be triggered only if the identity is actually changed and doesn't match the current one.

That sounds like an improvement, but it doesn't sound enough to fix the concern of comment 10, described in bug 1635124.
The problem is that navigating up and down the list with cursor up/down will instantly apply the navigated identity although the user has not yet confirmed it with ENTER (premature change of identity). So if I cursor-navigate from identity1 to identity5, i understand that `onRecipientsChanged()` will still needlessly fire 4 times because the dropdown widget applies navigated options prematurely. As a side effect, customize-from also fires prematurely on navigation, so keyboard-only users will find it very hard if not impossible to customize the from address of any identity other than the last.