Closed
Bug 672254
Opened 13 years ago
Closed 13 years ago
The arrow panel's height should be limited by the screen height.
Categories
(Core :: XUL, defect)
Core
XUL
Tracking
()
RESOLVED
FIXED
mozilla8
People
(Reporter: jaws, Assigned: enndeakin)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
(deleted),
patch
|
mak
:
feedback+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
(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•13 years ago
|
||
Enn is the person who can help you with this.
Reporter | ||
Comment 2•13 years ago
|
||
Neil, does this bug give you enough steps to reproduce/describe the issue well enough?
Assignee | ||
Comment 3•13 years ago
|
||
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?
Assignee | ||
Comment 4•13 years ago
|
||
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?
Assignee | ||
Updated•13 years ago
|
Attachment #546597 -
Flags: feedback?(jwein)
Comment 5•13 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 :-)
Comment 6•13 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.
Assignee | ||
Comment 7•13 years ago
|
||
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•13 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!
Comment 9•13 years ago
|
||
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+
Assignee | ||
Updated•13 years ago
|
Attachment #546597 -
Flags: feedback?(jwein)
Comment 10•13 years ago
|
||
Neil, I guess is ok to assign this to you since it has a patch
Assignee: nobody → enndeakin
Status: NEW → ASSIGNED
Assignee | ||
Comment 11•13 years ago
|
||
Attachment #551110 -
Flags: review?(neil)
Updated•13 years ago
|
Attachment #551110 -
Flags: review?(neil) → review+
Comment 12•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 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.
Description
•