143663 - Flickering file type scrolling list in Helper Applications

Status: NEW

(Core :: XUL, defect)

Description

[Raf Geens]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.0rc2) Gecko/20020510
BuildID:    2002051006

If the file type list contains more than 5 entries and you have to scroll to
view them all, scrolling causes the last element currently visible in the list
to flicker and interfere with the area just below it.
When you reach the bottom of the list, the scroll bar together with the list
starts flickering and rapidly jumping up and down.

Comment 1

[Boris Zbarsky [:bzbarsky]]

Sounds like an outliner bug... the part about "When you reach the bottom of the
list" is already reported elsewhere.

Comment 2

[Martyr]

Confirming in 1.0 RC 2 Mac OS 9.2.x, although on that platform it's when you
reach three types. The number isn't the thing in an absolute sense, but rather,
when you exceed the number of lines that the box has. Should probably change
affected OS to ALL and status to NEW. 
 

Comment 3

[parish]

I've been seeing this on W2K and XP for months as well.

Also, if the path in the Handled By: field is very long, like C:\Document and
Settings\marko\Application Data\foobar\foobar.exe then the Filetypes pane
resizes to accomodate it, pushing the buttons and scrollbar beyond the edge of
the window. Unfortunately the Prefs window is not resizable so you have to
resort to keystrokes. This is the same (mis)behaviour as in the mail/news header
pane, bug 91662

Comment 4

[Cees T.]

It also still occurs in trunk 2002080718 on WinME.

And the clipped Helper Apps prefs is
http://bugzilla.mozilla.org/show_bug.cgi?id=138680

Comment 5

[Cees T.]

Attachment: Looks like this

Still happens in trunk 2002090608 on Windows ME.

Steps to Reproduce:
1. Create some bogus Types till they don't all fit in the listbox.
2. Scroll the listbox and watch the line below it.

Comment 6

[Martyr]

This has been confirmed by THREE separate people -- why is it still marked
"unconfirmed"? What gives?

Comment 7

[sairuh (rarely reading bugmail)]

i can repro this using 2003.01.13.08 comm trunk on win2k and linux rh8 (both
modern and classic themes, fwiw). yep, the key is to create enough filetypes so
that the list becomes scrollable.

Comment 8

[Boris Zbarsky [:bzbarsky]]

This is a listbox bug, looks like...

Comment 9

[R.K.Aa.]

ping

Comment 10

[Jan Kozanek]

This is also happening in: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6)
Gecko/20040113

Exactly as specified by original reporter.

As an addition the erratic jump is caused also by a simple mouseclick on the
very last item in the listbox. Eventhough one manages to correctly scroll down
by pulling the scrollbar a click on the last item fires up the bug again.

Comment 11

[John T]

This looks related to bug #201117. As noted in a comment on that bug, I can see
only 7 helper apps listed, and am sure there are many more. A scrolling problem
is described too.

I like the idea mentioned in this bug of making the menu box resizeable---a
window, with space to show commandline arguments.

Comment 12

[[e-mail]]

OK, so I've tried this on 1.5 because it's the only build on which I have tons
of helper apps, it looks like scrolling the listbox causes a reflow of the
entire document, this is probably caused by an "optimization" of the listbox to
delete frames for rows that aren't visible. Now this document also contains a
<description> which doesn't reflow nicely, there are already reflow issues with
nsBlockToBoxAdapter. So unless somebody steps up to fix the reflow issues we'd
be better off converting this listbox to an rdfliner.

Comment 13

[Boris Zbarsky [:bzbarsky]]

Neil, what would be the pros and cons of converting to rdfliner?

If we do that, we should file a separate bug on the description reflow issue,
preferably with a minimal testcase...

Comment 14

[[e-mail]]

Attachment: Proposed patch

The listbox is just used to select an item from an RDF data source, for which
an RDFliner is just as suitable.

Anyway, don't we have enough bugs filed on the block to box adapter, such as
the radiobutton title not wrapping correctly when you change it?

Comment 15

[Boris Zbarsky [:bzbarsky]]

Yeah, we have existing bugs.  But that doesn't mean that fixing them would
address this problem... 

Comment 16

[Jan Varga [:janv]]

Comment on attachment 145021 [details] [diff] [review]
Proposed patch

man, that was quick
anyway, it looks ok
r=varga

Comment 17

[[e-mail]]

Comment on attachment 145021 [details] [diff] [review]
Proposed patch

Jan, it would have been quicker if I hadn't wasted time trying to convert it to
the simple template syntax which I prefer...

Comment 18

[Alec Flett]

Comment on attachment 145021 [details] [diff] [review]
Proposed patch

sr=alecf

Comment 19

[[e-mail]]

Comment on attachment 145021 [details] [diff] [review]
Proposed patch

FE-only stopgap for a strange interaction between lists and descriptions.

Comment 20

[Asa Dotzler [:asa]]

Comment on attachment 145021 [details] [diff] [review]
Proposed patch

a=asa (on behalf of drivers) for checkin to 1.7

Comment 21

[[e-mail]]

Marking fixed1.7 because the workaround is checked in but leaving bug open
pending testcase and investigation into the true cause of the bug.

Comment 22

[R.K.Aa.]

*** Bug 240537 has been marked as a duplicate of this bug. ***

Comment 23

[R.K.Aa.]

*** Bug 201117 has been marked as a duplicate of this bug. ***

Comment 24

[R.K.Aa.]

*** Bug 241180 has been marked as a duplicate of this bug. ***

Comment 25

[Christian :Biesinger (don't email me, ping me on IRC)]

*** Bug 221941 has been marked as a duplicate of this bug. ***

Comment 26

[BugBot [:suhaib / :marco/ :calixte]]

The severity field for this bug is relatively low, S4. However, the bug has 4 duplicates.
:neil, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Comment 27

[BugBot (nomail) [:suhaib / :marco/ :calixte]]

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Comment 28

[BugBot [:suhaib / :marco/ :calixte]]

The bug assignee is inactive on Bugzilla, so the assignee is being reset.