Bug 1602372 Comment 19 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
(In reply to Magnus Melin [:mkmelin] from comment #16)
> Review of attachment 9119970 [details] [diff] [review]:
> -----------------------------------------------------------------
> This looks ok to me.

So my review of comment 14 identified 4 instances where the current patch may cause loss of keyboard focus (also after removing rows), so does your review+ ("This looks ok to me.") indicate that this is acceptable?
Revision 1 by
on 
(In reply to Magnus Melin [:mkmelin] from comment #16)
> Review of attachment 9119970 [details] [diff] [review]:
> -----------------------------------------------------------------
> This looks ok to me.

So my patch review (comment 14) identified 4 instances where the current patch may cause loss of keyboard focus (also after removing rows), so does your subsequent review+ (comment 16) indicate that this is acceptable?

Back to Bug 1602372 Comment 19