1279400 - Fix devtools-tooltip-events-container style and behavior

Status: RESOLVED DUPLICATE of bug 1278181

(DevTools :: Inspector, defect, P3)

Description

[magicp]

Attachment: devtools-tooltip-events-container-aurora-vs-nightly.png

User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:50.0) Gecko/20100101 Firefox/50.0
Build ID: 20160609130607

Steps to reproduce:

1. Start latest Nightly
2. Go to about:home
3. Open devtools > Inspector
4. Click markupview-events on html element
5. Click any events


Actual results:

devtools-tooltip-events-container is smaller than Aurora, and also its height does not expand vertically when expanding any events. Then two vertical scrollbars will appear.


Expected results:

Apply previous build style and behavior

Comment 1

[magicp]

Regression range:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=63ec66da50faad032454a44d3af3124c35102b3e&tochange=62ad4f6b5c3428e4b33e1e4ba75288c94c45ec92

Comment 2

[Julian Descottes [:jdescottes]]

As we are moving the devtools out of XUL, the tooltips and floating panels are displayed within the toolbox area.
The tooltip actually expands when you open the code previews, but only until it reaches the top/bottom of the toolbox.

Two related bugs:
- Bug 1267403: reuse a XUL panel as a wrapper for the tooltips. Means we could use all the viewport's height and have the same behavior as today.
- Bug 1278181: change the design of the event details tooltip to be more compact. This will probably mean removing the code preview from the tooltip in favor of a more explicit link to open the event handler in the debugger tab.

For now I am going to block this on Bug 1267403. 

magicp: can you comment on Bug 1278181 if you feel strongly against removing the code previews? I'm in favor of removing them but if users find them useful we should reconsider and give more priority to Bug 1267403.

Comment 3

[Julian Descottes [:jdescottes]]

magicp: see question above.

Comment 4

[magicp]

(In reply to Julian Descottes [:jdescottes] from comment #2)

> magicp: can you comment on Bug 1278181 if you feel strongly against removing
> the code previews? I'm in favor of removing them but if users find them
> useful we should reconsider and give more priority to Bug 1267403.

I'm in favor of removing them too.

Comment 5

[Julian Descottes [:jdescottes]]

Bug 1267403 landed, but unfortunately we cannot use the XUL panel wrapper for the event details tooltip due to a technical issue (logged in Bug 1285261).

This means we will now go ahead with Bug 1278181 and update the design of the event details tooltip in order to be more compact.
Therefore I am closing this bug as a duplicate of 1278181.