1325730 - Can't see scroll bar for network panel when the width is less than 1000px

Status: RESOLVED DUPLICATE of bug 1309826

(DevTools :: Netmonitor, defect)

Description

[fireattack [:fireattack]]

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.28 Safari/537.36

Steps to reproduce:

See screenshot:

No scrollbar: http://i.imgur.com/F3cR07j.png
Has scrollbar: http://i.imgur.com/ooeMBCj.png


Actual results:

It will only show scrollbar when the width is more than around 1000 px.

This to me is very excessive and unnecessary.

Also, why users can't adjust the width of each column?

Comment 2

[Tooru Fujisawa [:arai]]

merging information from bug 1336203.

regression range:
  https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=9a03728016ff7b76c4850375303e84cce00e4ac4&tochange=28445d6f646be36233d080caae871ed1c39a242d

apparently from bug 1309866.

this issue also happens when the request details is opened and the width becomes small.


also the scroll bar is shown for <1000px situation by following:
  1. make the width >1000px
  2. narrow it slowly

but when I narrow it quickly, the scroll bar goes away.

so, apparently the scrollbar is placed at wrong place.

Comment 3

[Tooru Fujisawa [:arai]]

Attachment: scrollbar postion according to boxObject

boxObject says the scrollbar is placed at the red box in the attached screenshot.

Comment 4

[Tooru Fujisawa [:arai]]

apparently, div.request-list-container is wider than div.network-table, and that results in placing the scrollbar outside of visible area.

Comment 5

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Thanks for the report!

@Fred, could you please take a look at this?

Honza

Comment 6

[Randell Jesup [:jesup] (needinfo me)]

honza: which fred did you want to have take a look?

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

(In reply to Randell Jesup [:jesup] from comment #6)
> honza: which fred did you want to have take a look?
Ah, thanks for the note :-)

@Fred, could you please take a look at this?

Honza

Comment 8

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Btw. There is also bug 1338437 (another scroll-bar issue)

Honza

Comment 9

[[e-mail]]

Ricky, since you are dealing with split bar in Bug 1309183, 
could you help confirm if this issue still happened after applied your patch?

Comment 10

[[e-mail]]

Ooh I get confused, but this bug doesn't related to split bar

Comment 11

[[e-mail]]

I found the issue is fixed after convert netmonitor xul to xhtml in Bug 1309826

Comment 12

[[e-mail]]

I've tested again and the issue is fixed after bug 1309826 landed

Comment 13

[Liz Henry (:lizzard) (relman/hg->git project)]

Fixed in 54 in the duplicate bug but 53 is still affected.

Comment 14

[Liz Henry (:lizzard) (relman/hg->git project)]

But I don't think we want to uplift the work in bug 1309826 to aurora at this point. 
Is there a smaller fix for this issue you can cherry pick or shall I wontfix it for 53?

Comment 15

[Ricky Chien [:rickychien] (inactive)]

It would need to uplift a series of patches if we want to fix for 53. I'd prefer to wontfix it for 53 since there is no smaller patch can fix this issue easily.

Comment 16

[Liz Henry (:lizzard) (relman/hg->git project)]

Great, thanks Ricky!