feat: Remove breadcrumb from "visibilitychange" event#102
Conversation
We handled the blur case but not the focus case in #94
ryan953
left a comment
There was a problem hiding this comment.
is there a race condition or anything between the visibilitychanged event and onfocus? Like, if a page is minimized, it'll be both hidden and blurred. Then when it comes back to the desktop which event will fire first (and turned into a breadcrumb), and which gets dropped?
|
There is (and on blur too) -- from what I can tell from the interwebs, the event order is not guaranteed. |
|
If there's no order guarantee then we could get pairs like So what if we unconditionally created teh breadcrumb, but then guarded the |
|
We're going to axe the "visible" breadcrumb, as it is a point of confusion for users (i.e. what is the difference between blur/focus and visibility?) We will be able to get the information we need with focus/blur alone. |
…ry-replay#102) We're going to axe the "visible" breadcrumb, as it is a point of confusion for users (i.e. what is the difference between blur/focus and visibility?) We will be able to get the information we need with focus/blur alone.
We're going to axe the "visible" breadcrumb, as it is a point of confusion for users (i.e. what is the difference between blur/focus and visibility?) We will be able to get the information we need with focus/blur alone.