179347 2017-11-06 15:30:37 -0800 Web Inspector: Canvas Tab: reloading from canvas overview after making a recording will show the recording instead of overview 2017-12-04 17:41:01 -0800 1 1 1 Unclassified WebKit Web Inspector WebKit Nightly Build All All RESOLVED WORKSFORME http://acko.net InRadar P2 Normal --- 173807 175485 1 bburg webkit-unassigned bburg hi inspector-bugzilla-changes webkit-bug-importer oldest_to_newest 1368808 0 bburg 2017-11-06 15:30:37 -0800 Steps to reproduce: 1. Create canvas recording on acko.net main canvas 2. Go back to the canvas overview 3. Reload the page => Jumps to the last created recording, rather than staying on overview page This doesn't make any sense from a UX perspective. Devin wrote some code to treat orphaned recordings the same as imported recordings, but they shouldn't be auto selected this way. 1368809 1 webkit-bug-importer 2017-11-06 15:31:03 -0800 <rdar://problem/35375421> 1368816 2 hi 2017-11-06 15:57:28 -0800 I don't think that they are auto-selected. My guess as to what is happening is that we are never switching views away from the Recording, which is why it stays visible. From what I remember, this is controlled by CanvasTabContentView.prototype.restoreStateFromCookie (and saveStateToCookie), so if we want to do this then we probably just want to modify those functions. Personally, if I am viewing a recording (which is pretty independent of the page itself), I wouldn't want the recording to be deselected/hidden if I refresh the page. Due to the exportability of Recordings, I don't see them as being "restricted" to a particular instance of a given page. I see them more as a snapshot in time of something that happened, and WebInspector is just the means by which we can view that snapshot. I can see the argument against this train of thought, however, so I could go either way. 1369017 3 bburg 2017-11-07 08:45:29 -0800 (In reply to Devin Rousso from comment #2) > I don't think that they are auto-selected. My guess as to what is happening > is that we are never switching views away from the Recording, which is why > it stays visible. From what I remember, this is controlled by > CanvasTabContentView.prototype.restoreStateFromCookie (and > saveStateToCookie), so if we want to do this then we probably just want to > modify those functions. > > Personally, if I am viewing a recording (which is pretty independent of the > page itself), I wouldn't want the recording to be deselected/hidden if I > refresh the page. Due to the exportability of Recordings, I don't see them > as being "restricted" to a particular instance of a given page. I see them > more as a snapshot in time of something that happened, and WebInspector is > just the means by which we can view that snapshot. I can see the argument > against this train of thought, however, so I could go either way. I'm not proposing to hide the recording details and go back to overview when the page reloads. I'm proposing that we stay on the overview when the page reloads, but currently it jumps to a recording I wasn't looking at previously. 1369127 4 hi 2017-11-07 12:12:15 -0800 (In reply to Brian Burg from comment #3) > I'm not proposing to hide the recording details and go back to overview when > the page reloads. I'm proposing that we stay on the overview when the page > reloads, but currently it jumps to a recording I wasn't looking at > previously. I'm not seeing this issue on ToT. I followed your steps to reproduce, and when I refreshed the page I stayed on the overview view. 1370402 5 bburg 2017-11-10 10:19:13 -0800 I can't find a reliable reproduction, so let's close this.