Retrieve actual scroll bar width when rendering thumbnails

RESOLVED WONTFIX

Status

Firefox Graveyard
Panorama
RESOLVED WONTFIX
7 years ago
2 years ago

People

(Reporter: ttaubert, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

7 years ago
We currently assume a static width of 25px for vertical scroll bars. We should retrieve the actual width and subtract that from window.innerWidth.
Blocks: 603789

Comment 1

7 years ago
bugspam
Target Milestone: Future → ---
(Reporter)

Updated

7 years ago
No longer blocks: 603789
(Reporter)

Updated

7 years ago
Blocks: 653099
(Reporter)

Comment 2

7 years ago
bugspam
No longer blocks: 653099
(Reporter)

Comment 3

7 years ago
bugspam
Blocks: 660175
Created attachment 546468 [details] [diff] [review]
v1
Assignee: nobody → raymond
Status: NEW → ASSIGNED
Attachment #546468 - Flags: feedback?(tim.taubert)
Comment on attachment 546468 [details] [diff] [review]
v1

document.body.clientWidth can exceed the view port, afaik.

Also, all this content-accessing code needs to go away for e10s.
Attachment #546468 - Flags: review-
(In reply to comment #5)
> Comment on attachment 546468 [details] [diff] [review] [review]
> v1
> 
> document.body.clientWidth can exceed the view port, afaik.
> 
> Also, all this content-accessing code needs to go away for e10s.

ok, I see.  I thought the clientWidth presents the width view port (based on https://developer.mozilla.org/en/DOM/element.clientWidth).

Any suggestions how I can get the vertical scrollbar width?
(Reporter)

Comment 7

7 years ago
(In reply to comment #6)
> Any suggestions how I can get the vertical scrollbar width?

IIRC, this is not implemented, yet (in gecko) but someone said that it shouldn't be too difficult. So maybe we should open a bug for the enhancement and let it block this one.
Depends on: 672474
(Reporter)

Comment 8

7 years ago
bugspam

(Fx7 was branched, removing open bugs from Fx7 meta bug, we won't create new meta bugs for upcoming Fx versions)
No longer blocks: 660175
Status: ASSIGNED → NEW
(Reporter)

Updated

7 years ago
Attachment #546468 - Flags: feedback?(tim.taubert)
Assignee: raymond → nobody
(Reporter)

Comment 9

5 years ago
Will be fixed by bug 727765.
Depends on: 727765
No longer depends on: 672474
Panorama has been removed from Firefox 45, currently in Beta and scheduled for release on March 7th. As such, I'm closing all existing Panorama bugs.

If you are still using Panorama, you will see a deprecation message in Firefox 44, and when 45 is released your tab group data will be migrated to bookmarks, with a folder for each group. There are also a few addons offering similar functionality.

See https://support.mozilla.org/en-US/kb/tab-groups-removal for more info.

We're removing Panorama because it has extremely low usage (about 0.01% of users), and has a large number of bugs and usability issues. The cost of fixing all those issues is far too high to justify, and so we'll instead be focusing our time and energy on improving other parts of Firefox.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

2 years ago
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.