Closed Bug 1279400 Opened 5 years ago Closed 5 years ago

Fix devtools-tooltip-events-container style and behavior

Categories

(DevTools :: Inspector, defect, P3)

defect

Tracking

(firefox50 affected)

RESOLVED DUPLICATE of bug 1278181
Tracking Status
firefox50 --- affected

People

(Reporter: magicp.jp, Unassigned)

References

Details

Attachments

(1 file)

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
Has STR: --- → yes
Component: Untriaged → Developer Tools: Inspector
OS: Unspecified → All
Hardware: Unspecified → All
Whiteboard: [devtools-html] [triage]
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.
Depends on: 1267403
magicp: see question above.
Flags: needinfo?(magicp.jp)
Priority: -- → P3
Whiteboard: [devtools-html] [triage] → [reserve-html]
(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.
Flags: needinfo?(magicp.jp)
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.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1278181
No longer blocks: devtools-html-1
Whiteboard: [reserve-html]
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.