The arrow panel's height should be limited by the screen height.

RESOLVED FIXED in mozilla8

Status

()

Core
XUL
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jaws, Assigned: Neil Deakin (mostly unavailable until September))

Tracking

(Blocks: 1 bug)

Trunk
mozilla8
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(In reply to comment #228)
> The download panel should have scrollbars when there's too much downloads in
> the list. With current tryserver build, if there's too much downloads only
> the 17 first (on a 1920x1200 screen) are reachable.

Thanks for the feedback: this looks like a regression in panels with
type="arrow". They are not limited by the height of the screen, while
normal panels are (though they overlap with the task bar on Windows).
These issues should be looked into by someone familiar with arrow panels.

See comment #230 on bug 567934 for more information: https://bugzilla.mozilla.org/show_bug.cgi?id=564934#c230

Comment 1

6 years ago
Enn is the person who can help you with this.
Neil, does this bug give you enough steps to reproduce/describe the issue well enough?
Do you mean that the panel is cropped at the egde of the screen or that it continues past the end of the screen?

Do you have something in the panel that is scrollable?
Created attachment 546597 [details] [diff] [review]
test patch

Probably the boxes in the arrowpanel binding just need to be made flexible, such that when the panel is resized because it is too big for the screen, the content will stretch down.

Can you test this patch to see if it works for you?
Attachment #546597 - Flags: feedback?(jwein)

Comment 5

6 years ago
(In reply to comment #4)
> Can you test this patch to see if it works for you?

I'm rebuilding the patch queue leading to bug 564934 right now, I'll add this
patch and let you know if Jared doesn't beat me to it :-)

Updated

6 years ago
Blocks: 672365

Comment 6

6 years ago
This patch solves the issue that's specific to the arrow panels, thanks!

I've filed bug 672365 for the remaining issue with the task bar on Windows,
that affects both normal panels and arrow panels.
Normal panels should probably appear behind the taskbar as other windows do.

For this specific downloads arrow panel, if it does not have any textboxes or other means of text entry in it, it should be sufficient to use level="top".

Comment 8

6 years ago
(In reply to comment #7)
> For this specific downloads arrow panel, if it does not have any textboxes
> or other means of text entry in it, it should be sufficient to use
> level="top".

This can work for us, making bug 672365 a non-blocking issue in our case.
Thanks for the suggestion!

Updated

6 years ago
Blocks: 652115
Comment on attachment 546597 [details] [diff] [review]
test patch

this solves most issues I saw in a resizeable bookmarks panel, could you get review on it?
Attachment #546597 - Flags: feedback+
Attachment #546597 - Flags: feedback?(jwein)
Neil, I guess is ok to assign this to you since it has a patch
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Created attachment 551110 [details] [diff] [review]
make arrowpanels flexible
Attachment #551110 - Flags: review?(neil)

Updated

6 years ago
Attachment #551110 - Flags: review?(neil) → review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/968dee8a144a
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Hardware: x86 → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.