Arrow twisties disappear in narrow window

RESOLVED FIXED in Firefox 48

Status

()

Firefox
Developer Tools: Memory
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: magicp, Assigned: fitzgen)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 48
Points:
---

Firefox Tracking Flags

(firefox47 affected, firefox48 fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

2 years ago
Created attachment 8727739 [details]
devtools-memory-do-not-hide-twisties.mp4

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

Steps to reproduce:

1. Start Nightly
2. Open DevTools > Memory
3. Switch view to "Dominators" (or "Aggregate", Dominators view is easy for reproduce)
4. Take snapshot
5. Narrowing heap-view


Actual results:

Twisties are hidden.


Expected results:

Don't hide twisties in narrow window.
(Reporter)

Updated

2 years ago
Blocks: 1221506
Has STR: --- → yes
status-firefox47: --- → affected
status-firefox48: --- → affected
Component: Untriaged → Developer Tools: Memory
OS: Unspecified → All
Hardware: Unspecified → All
This is not intentional, and I have tried fixing this multiple times (clearly I am terrible with CSS). I mentioned this issue in another bug, but can't remember which one... Yes, *very* annoying!

Ah, here is the bug: bug 1248891

Will turn this bug into the arrow issue, let that bug be about the table cells.
Priority: -- → P2
See Also: → bug 1248891
Summary: [DevTools][Memory] Don't hide twisties in narrow window → Arrow twisties disappear in narrow window
(Reporter)

Comment 2

2 years ago
Probably this issue will be solved, if overflow property is removed. And Bug 1254458 also.

memory.css line:464

.heap-tree-number,
.heap-tree-percent,
.heap-tree-item-name {
  white-space: nowrap;
-  overflow: hidden;
}
Created attachment 8728508 [details] [diff] [review]
Fix disappearing arrow twisties in memory tool when narrow
Attachment #8728508 - Flags: review?(vporof)
Assignee: nobody → nfitzgerald
Status: NEW → ASSIGNED
Created attachment 8728509 [details]
arrows.gif

This demonstrates how the arrows do not disappear anymore with this patch applied.

Thanks magicp!
Comment on attachment 8728508 [details] [diff] [review]
Fix disappearing arrow twisties in memory tool when narrow

Review of attachment 8728508 [details] [diff] [review]:
-----------------------------------------------------------------

r+ but see below.

::: devtools/client/themes/memory.css
@@ -452,5 @@
>  .heap-tree-number,
>  .heap-tree-percent,
>  .heap-tree-item-name {
>    white-space: nowrap;
> -  overflow: hidden;

This doesn't take care of the toolbox itself resizing, and the table will start overflowing horizontally in that case, and a scrollbar will appear. This might be ok, but if not desirable, a better fix would be making only these changes:

.heap-tree-item-field.heap-tree-item-name > .theme-twisty {
  min-width: 14px; /* this used to be only `width: 14px` in dark/light-theme.css */
}

.heap-tree-item-field.heap-tree-item-name > :last-child {
  overflow: hidden;
  text-overflow: ellipsis;
}
Attachment #8728508 - Flags: review?(vporof) → review+
Thanks, Victor!

We have another bug open for making horizontal scrolling work better. We do want to be able to scroll deeply nested things into view, rather than hide them behind ellipses.
Keywords: checkin-needed

Comment 7

2 years ago
https://hg.mozilla.org/integration/fx-team/rev/d0c962286d13
Keywords: checkin-needed

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d0c962286d13
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox48: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in before you can comment on or make changes to this bug.