Closed
Bug 943341
Opened 11 years ago
Closed 8 years ago
Investigate making the variables view inspection popup more transient
Categories
(DevTools :: Debugger, defect, P3)
DevTools
Debugger
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: vporof, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [lang=js])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Comment 1•11 years ago
|
||
So, I've been thinking to this overnight.
How about:
* when hovering more than XXX ms, the popup is displayed in "transient mode"
+ in that case, it's moved as soon as the user hovers another variable or out of the current variable (with a delay), my mind is not made on this yet
+ if the user clicks on the popup, the popup goes in "permanent mode" (see next point)
* when clicking on the var, the popup is displayed asap The click event should not be "preventDefaulted" to allow selection. It's in "permanent mode":
+ the popup is not dismissed automatically
+ it's dismissed if the user clicks elsewhere
+ I don't know yet if it should show another popup on the other var, probably not
Comment 2•11 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #1)
> * when clicking on the var, the popup is displayed asap The click event
> should not be "preventDefaulted" to allow selection. It's in "permanent
> mode":
> + the popup is not dismissed automatically
> + it's dismissed if the user clicks elsewhere
> + I don't know yet if it should show another popup on the other var,
> probably not
At some point in the not too distant future the debugger will stop being read-only, in which case such behavior would interfere with regular editing.
Comment 3•11 years ago
|
||
(In reply to Panos Astithas [:past] from comment #2)
> (In reply to Julien Wajsberg [:julienw] from comment #1)
> > * when clicking on the var, the popup is displayed asap The click event
> > should not be "preventDefaulted" to allow selection. It's in "permanent
> > mode":
> > + the popup is not dismissed automatically
> > + it's dismissed if the user clicks elsewhere
> > + I don't know yet if it should show another popup on the other var,
> > probably not
>
> At some point in the not too distant future the debugger will stop being
> read-only, in which case such behavior would interfere with regular editing.
Wouldn't we want to use dblclick for this?
Comment 4•11 years ago
|
||
Use dblclick for what, to start editing a file? No, I don't think so. No other editor does it.
Comment 5•11 years ago
|
||
I was thinking it would be like the inspector: dbl click to trigger editing. But now I understand what you're talking about: the whole file is in edit mode already.
Comment 6•11 years ago
|
||
Huge +1. I often accidentally trigger the pop up and intuitively try to just move the mouse away to make the pop up go away. You have to move it away and click, which is annoying.
Reporter | ||
Comment 7•11 years ago
|
||
(In reply to Nick Fitzgerald [:fitzgen] from comment #6)
> Huge +1. I often accidentally trigger the pop up and intuitively try to just
> move the mouse away to make the pop up go away. You have to move it away and
> click, which is annoying.
In that case maybe we should just increase the hover timeout.
Comment 8•11 years ago
|
||
(In reply to Victor Porof [:vp] from comment #7)
> (In reply to Nick Fitzgerald [:fitzgen] from comment #6)
> > Huge +1. I often accidentally trigger the pop up and intuitively try to just
> > move the mouse away to make the pop up go away. You have to move it away and
> > click, which is annoying.
>
> In that case maybe we should just increase the hover timeout.
I think that will just solve partial problem. Most of the time, I just tend to leave my mouse untouched while I try to do something else. I assume many users will be doing this.
I am also hugely inclined towards making the popup transient.
Comment 9•11 years ago
|
||
Not to mention that every other debugger that I have worked with goes with a transient approach:
- Eclipse
- Chrome
- IntelliJ
- Firebug
- Safari
- IE
- Visual Studio
Comment 10•11 years ago
|
||
(In reply to Victor Porof [:vp] from comment #7)
> (In reply to Nick Fitzgerald [:fitzgen] from comment #6)
> > Huge +1. I often accidentally trigger the pop up and intuitively try to just
> > move the mouse away to make the pop up go away. You have to move it away and
> > click, which is annoying.
>
> In that case maybe we should just increase the hover timeout.
Sure, but having to click is an annoyance; we should be able to just move the mouse away. Hover in, hover out.
Comment 11•11 years ago
|
||
No tests yet, but this behavior feels a lot more natural to me.
Assignee: nobody → nfitzgerald
Status: NEW → ASSIGNED
Reporter | ||
Comment 12•11 years ago
|
||
(In reply to Nick Fitzgerald [:fitzgen] from comment #11)
> Created attachment 8345394 [details] [diff] [review]
> wip 1
>
You can't interact with the panel's content now.
Comment 13•11 years ago
|
||
(In reply to Victor Porof [:vp] from comment #12)
> (In reply to Nick Fitzgerald [:fitzgen] from comment #11)
> > Created attachment 8345394 [details] [diff] [review]
> > wip 1
> >
>
> You can't interact with the panel's content now.
Ah good catch, need to clear the timeout when you enter the popup, and call hideContents when you mouse out of the popup.
Comment 14•11 years ago
|
||
I don't have time for this at the moment. Maybe someone else can clean this up and land it.
Assignee: nfitzgerald → nobody
Status: ASSIGNED → NEW
Updated•11 years ago
|
Priority: -- → P3
Updated•11 years ago
|
Whiteboard: [mentor=nfitzgerald@mozilla.com][lang=js]
Updated•11 years ago
|
Whiteboard: [mentor=nfitzgerald@mozilla.com][lang=js] → [good first bug][mentor=nfitzgerald@mozilla.com][lang=js]
Comment 15•11 years ago
|
||
Thinking about it some more, I assume you probably meant to mark this as just a mentored bug.
Whiteboard: [good first bug][mentor=nfitzgerald@mozilla.com][lang=js] → [mentor=nfitzgerald@mozilla.com][lang=js]
Comment 16•11 years ago
|
||
(In reply to Panos Astithas [:past] from comment #15)
> Thinking about it some more, I assume you probably meant to mark this as
> just a mentored bug.
Yup :)
Updated•11 years ago
|
Assignee | ||
Updated•10 years ago
|
Mentor: nfitzgerald
Whiteboard: [mentor=nfitzgerald@mozilla.com][lang=js] → [lang=js]
Updated•10 years ago
|
Blocks: dbg-frontend
Comment 17•10 years ago
|
||
Panos pointed out this bug to me, I didn't know it existed and made a comment in the original bug: https://bugzilla.mozilla.org/show_bug.cgi?id=725235#c94 which I'm repeating here:
"
I know this is an old discussion, but after a long time using the debugger and the variables popup, I still find this annoying most of the times, so I thought I would report it here and maybe we can get a discussion started to eventually end up on a bug being filed.
I think:
- it takes too long for the popup to appear
- it should go away when you move your mouse out of the expression but:
- it should do so only after a delay that allows the user to quickly move back the mouse to the expression if needed
- it shouldn't occur if the mouse goes over the bubble itself.
I think this is pretty much what Chrome does. Also, beside my point of view, I've also had several reports of our implementation being a little annoying.
I'd like to hear what other people think about this.
"
Comment 18•10 years ago
|
||
In comment 1, I also outlined something about the "permanent mode"; for example, if you click inside the bubble to open a tree, it would be a PITA if the bubble is undisplayed. So my proposal was that as soon as we click inside the bubble, it becomes non-transient.
But yeah, it should be transient until something triggers "permanent mode".
Comment 19•10 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #18)
> In comment 1, I also outlined something about the "permanent mode"; for
> example, if you click inside the bubble to open a tree, it would be a PITA
> if the bubble is undisplayed. So my proposal was that as soon as we click
> inside the bubble, it becomes non-transient.
>
> But yeah, it should be transient until something triggers "permanent mode".
Yeah, that sounds like a good idea.
I think the people that have told me the didn't like having to click outside of the popup just wanted it to work like on chrome.
I do agree that it's a pity though that you loose the context of what has been expanded when the popup is hidden.
I don't think it's something that occurs all that often though. When you're expanding properties in the popup, you're focused on reading the children properties and scrolling through, so it's quite rare that you'd move your mouse out of the popup at that stage. Having said that, if we could restore the expanded state if you move your mouse back to the same variable, that would solve this issue too, right? (and it could be easy if we just remember the last variable that was inspected and left the content in the panel).
Comment 20•10 years ago
|
||
If you hover on another variable by mistake, then you'll still lose it :)
I think we just need to try it for real to know how this feels.
Comment 22•10 years ago
|
||
I ranted about this a while ago - while some of the following has changed due to debugger evolution, most of it applies here (IMO)
https://bugzilla.mozilla.org/show_bug.cgi?id=967096#c14
And if you can fix the referenced bug while you're in this code, great - I'll be more than pleased - it's a 24/7 problem here.
Comment 23•10 years ago
|
||
(In reply to Russ from comment #22)
> I ranted about this a while ago - while some of the following has changed
> due to debugger evolution, most of it applies here (IMO)
>
> https://bugzilla.mozilla.org/show_bug.cgi?id=967096#c14
>
> And if you can fix the referenced bug while you're in this code, great -
> I'll be more than pleased - it's a 24/7 problem here.
Hi Russ. I'm sorry to hear you had a bad experience :-( I've read your rant and agree with most of the points in it. This really is something that needs to be fixed. My queue is pretty full at the moment, but I still intend to get to this in Q1 if at all possible.
If you can't wait that long, and if you really wanted to, you could probably fix this problem yourself. It shouldn't be that hard to fix if you know where to look. If you are interested, I could find a mentor to help you out, which would save you a lot of time.
Of course, I realise your time is probably just as precious as mine, so I understand if you don't want to do that. In which case all I can do is to ask you to be patient for a bit longer :-)
Comment 24•10 years ago
|
||
Hi Eddy
Thanks. No, I'm not a moz dev - just a 24/7 dev user. The on ramp for me would be huge plus, I'm exclusively contracted to my employer :/
Comment 25•10 years ago
|
||
(In reply to Russ from comment #24)
> Hi Eddy
>
> Thanks. No, I'm not a moz dev - just a 24/7 dev user. The on ramp for me
> would be huge plus, I'm exclusively contracted to my employer :/
Fair enough Russ. I'll try to get to this bug as soon as I can.
Comment 26•10 years ago
|
||
(The fix I'm trying to get in through bug 967853 is nearby and related but I don't think it will conflict after looking briefly at the attached wip1 patch)
Comment 27•9 years ago
|
||
My apologies to the people who have been waiting for this, but this bug kind of slipped of my radar, and I am not actively working on it at the moment
Assignee: ejpbruel → nobody
Updated•9 years ago
|
Mentor: nfitzgerald
Comment 28•9 years ago
|
||
I'm a js developer and this really makes me crazy: Waiting, no debugger popup, clicking somewhere, delay, then finally there it is.... It is still present and very annoying :( I was just about screencasting how Chrome does the debugger popup, until I found this bug here. The way the Chrome devtools opens and closes the popup feels very natural, like already pointed out in Comment 17 by pbro (almost 2 years ago).
Comment 29•9 years ago
|
||
Also, the image popup in the network and inspector panel (transient) do just feel right to me - and are fast. That the debugger popup behaves different makes the devtool's UI inconsistent.
Comment 30•8 years ago
|
||
Now that there's a very nice transient(!) Popup for the “Cause” column of the Network Monitor (see Bug 1134073 and its dependent Bug 1270111), I wonder if the debuggers driving-me-crazy-click-hover-click-somewhere-else-hover-again ;) popup could be approached.
Comment 31•8 years ago
|
||
Closing.
This work will now be tracked in this issue: https://github.com/devtools-html/debugger.html/issues/626
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → INVALID
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•