Closed
Bug 511075
Opened 15 years ago
Closed 15 years ago
The arrows of the scrollbars don't work in hotmail (event.preventDefault() causes clicks on scrollbars to be ignored)
Categories
(Core :: DOM: Events, defect)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
People
(Reporter: elguerreroesp, Assigned: smaug)
References
(Depends on 1 open bug)
Details
(Keywords: regression, testcase)
Attachments
(3 files, 2 obsolete files)
(deleted),
text/html
|
Details | |
(deleted),
image/gif
|
Details | |
(deleted),
patch
|
enndeakin
:
review+
neil
:
superreview+
christian
:
approval1.9.2.4+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a2pre) Gecko/20090817 Namoroka/3.6a2pre (.NET CLR 3.5.30729)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a2pre) Gecko/20090817 Namoroka/3.6a2pre (.NET CLR 3.5.30729)
When I sign in hotmail the arrows of the scrollbar in the inbox don´t work. This happens only int the name of the mails. Sorry for my English, I´m Spanish.
Reproducible: Always
Steps to Reproduce:
1.Sign in hotmail.
2.Go to inbox.
3.The arrows in scrollbar don´t work.
The arrows don't work in 3.6 beta 5 as well. They work fine in 3.5.x.
Comment 2•15 years ago
|
||
This worked for me on w32, Juan, can you take a look and renominate if you can reproduce?
Flags: blocking-firefox3.6?
Comment 3•15 years ago
|
||
It might be worth trying Spanish localizations (of Hotmail, the browser, and the OS) if you can't reproduce in English.
I'm in the US and can confirm this on 2 separate installations - Windows XP (32-bit) and Windows 7 (64-bit). When I log into Hotmail, click on Inbox, the scroll arrows in the upper pane (to scroll down the messages pane) does not work for the arrows. The scroll arrows in the lower pane work fine. Kindly confirm that your upper pane scroll arrows work.
Flags: blocking-firefox3.6?
Comment 5•15 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; es-MX; rv:1.9.2b5) Gecko/20091204 Firefox/3.6b5 (.NET CLR 3.5.30729) Latest version of Flash.
I tried that on es-mx, es-es, on a VM as well as in hardware on XP (OS installation in Spanish); I clicked on the scrollbars' arrows for the page. Same in individual messages long enough to display scrollbars. Both horizontal and vertical scrollbars. Works for me.
Reporter, are you running any(which) addons? When I use NoScript, for example, my experience in Hotmail is pretty unpleasant.
Are there different versions of Hotmail these days? IE and FX are giving me the same experience on these machines, so at least I get consistent results on my personal account.
Updated•15 years ago
|
Flags: blocking-firefox3.6?
Problem: Hotmail top pane scroll arrows do not work.
Add-ons: Firegestures, Flashblock. I tried uninstalling both of them and got the same result.
Do the plugins make a difference? I have 3.5.6 installed and installed 3.6 b5 using the same profile. Hotmail is the only e-mail provider which which I have a problem and that is just with 3.6. 3.5 does not have the issue.
Comment 7•15 years ago
|
||
To reproduce you have to use the non-default setting on hotmail to use "panes", which is probably why other people haven't been able to reproduce.
It is caused by bug 311053.
Comment 8•15 years ago
|
||
So is the page just calling preventDefault() and not expecting that to have an effect?
Component: General → XUL
Product: Firefox → Core
QA Contact: general → xptoolkit.widgets
Comment 9•15 years ago
|
||
As mentioned in comment# 7, this is easily reproduced if you click on the Options link which then shows a drop-down menu where you can change the "Reading pane settings" from Off to Right or Bottom. It also happens at least on Mac, and it has been like that for a very long time. I'll renominate to get it triaged.
Flags: blocking1.9.2?
Comment 10•15 years ago
|
||
Feels like a tech evangelism bug based on comment 7 and comment 8, no? I don't think it's a release blocker based on the fact that it's a non-standard option for Hotmail, but I'll add a relnote.
Keywords: regression,
relnote
Updated•15 years ago
|
Flags: blocking1.9.2? → blocking1.9.2-
Comment 11•15 years ago
|
||
True, it may be a non-standard option in Hotmail, but why was it working in 3.5.x and is broken in the new version? The argument can be made that this is currently _known_ to be broken only in Hotmail, but the bug can easily affect other sites as well. It is a newly introduced bug in 3.6 and should be tracked separately. Have we confirmed that bug 311053 is the cause? If so, the status of that bug shows up as fixed. If that bug (or a new one) is not reopened, do we know this issue will be fixed?
Comment 12•15 years ago
|
||
Note that the same issue is reported in bug 530237 for the Excel Web App. I'm trying to get a simple repro to help track down the root issue, but I do know that there's nothing in our app that is swallowing those mouse messages for the scrollbars. Note that this is a regression in 3.6 since everything works fine in 3.5.
Comment 13•15 years ago
|
||
This is also being reported in the forum (http://support.mozilla.com/nl/forum/1/550043) as it also affects http://www.snopes.com/katrina/soapbox/snowfall.asp.
Seems like this should be investigated.
Comment 14•15 years ago
|
||
Based on the snopes link. It seems bug 311053 allowed js to override clicks to scrollbars.
Comment 15•15 years ago
|
||
We run into this problem in our JavaScript framework qooxdoo as well <http://bugzilla.qooxdoo.org/show_bug.cgi?id=3357>. We hope to be able to fix it in the framework but all existing application will have to be updated. For this reason this is a pretty severe bug for us.
Comment 16•15 years ago
|
||
This is also being reported in the forum
http://support.mozilla.com/en-US/forum/1/564719
Comment 17•15 years ago
|
||
bug 524401 is a duplicate. How did a major release go out with bug 311053?
Comment 19•15 years ago
|
||
(In reply to comment #17)
> How did a major release go out with bug 311053?
Bug 311053 was landed eleven months ago, so unless you are saying that this is a regression *because of* that bug, I don't see the logic.
Comment 20•15 years ago
|
||
(In reply to comment #19)
> Bug 311053 was landed eleven months ago, so unless you are saying that this is
> a regression *because of* that bug, I don't see the logic.
That is what I was saying. What I should have said was, "How did a major release go out with bug 511075?"
Comment 21•15 years ago
|
||
Possibly related with bug 520501...? (Sorry if this sounds like noise, given that the bug is actually around the scrollbar thumb.)
Comment 22•15 years ago
|
||
(In reply to comment #10)
> Feels like a tech evangelism bug based on comment 7 and comment 8, no?
This is not a tech evangelism bug based on comment 14.
Keywords: testcase
Comment 23•15 years ago
|
||
The OpenExchange interface is affected by this, too.
What is weird here is that only the scrollbars for the folder panel (top left) are affected, all the other ones do work fine.
Comment 24•15 years ago
|
||
I see this bug as well with WinXP Prof SP3 English and FF 3.6 using hotmail. (btw Reading panes used to be the default and I have inherited them through the ages.)
Comment 25•15 years ago
|
||
Hi, this is my first post ever on here so I hope you don't feel this post is in the wrong spot but I thought it was appropriate to put here as this is the closest issue to mine. I use Yahoo Mail, and I am having the same problem with Yahoo Mail (I think). I can't use the scroll feature with my mouse. I use a reading pane as well. Whether I'm in the body of the email, or in the main screen with the reading pane I can't scroll up or down (using the middle roller button on my mouse). If I click the arrows on the right to go up and down they work fine... Is this at all relevant to this post? If not, can someone direct me to where I can present this issue perhaps? I just want to help make the community aware of the problem so it can be fixed for other users. Also, If there's anything I can do to help I'd be more than willing. Thanks!
Updated•15 years ago
|
blocking2.0: --- → ?
Comment 27•15 years ago
|
||
possibly related to Bug 489667?
Comment 28•15 years ago
|
||
Yeah, that is the same bug.
Comment 29•15 years ago
|
||
NOT FIXED IN 3.6.2
Scrolling through emails in hotmail does not work properly in Firefox 3.6.2 on Mac. Dragging blue scroll blob up or down DOES work, but use of arrows or clicking below or above scroll blob DOES NOT work. Safari works fine, so this is clearly a firefox issue. By the way, it worked fine in earlier versions, just appeared in the last 1 or 2 version upgrades.
Updated•15 years ago
|
Summary: The arrows of the scrollbars don´t work in hotmail. → The arrows of the scrollbars don't work in hotmail.
Updated•15 years ago
|
Severity: minor → normal
Component: XUL → DOM: Events
QA Contact: xptoolkit.widgets → events
Summary: The arrows of the scrollbars don't work in hotmail. → The arrows of the scrollbars don't work in hotmail (event.preventDefault() causes clicks on scrollbars to be ignored)
Comment 32•15 years ago
|
||
Per the letter of the DOM spec, this is probably the right behavior... if the default click behavior is to move the scrollbar and you call preventDefault(), the scrollbar shouldn't move. I assume no other browser actually does that, though?
Assignee | ||
Comment 33•15 years ago
|
||
Yeah, as far as I see this is the correct behavior. And IIRC we have always
had this behavior if event propagation and default handling has been stopped
early enough (in capture phase).
So this could be evang bug, or we need to change native anonymous scrollbar handling. Perhaps the latter to get better backwards compatibility.
Taking.
Assignee: nobody → Olli.Pettay
Status: NEW → ASSIGNED
Assignee | ||
Comment 34•15 years ago
|
||
(In reply to comment #33)
> And IIRC we have always
> had this behavior if event propagation and default handling has been stopped
> early enough (in capture phase).
Actually, I'm probably wrong about this.
Anyway, the current behavior is right, but perhaps we can't keep it.
Patch coming soon.
Assignee | ||
Comment 35•15 years ago
|
||
(In reply to comment #25)
> Hi, this is my first post ever on here so I hope you don't feel this post is in
> the wrong spot but I thought it was appropriate to put here as this is the
> closest issue to mine. I use Yahoo Mail, and I am having the same problem with
> Yahoo Mail (I think). I can't use the scroll feature with my mouse. I use a
> reading pane as well. Whether I'm in the body of the email, or in the main
> screen with the reading pane I can't scroll up or down (using the middle
> roller button on my mouse). If I click the arrows on the right to go up and
> down they work fine... Is this at all relevant to this post? If not, can
> someone direct me to where I can present this issue perhaps? I just want to
> help make the community aware of the problem so it can be fixed for other
> users. Also, If there's anything I can do to help I'd be more than willing.
> Thanks!
So how is this related to this bug? Isn't this bug about not being able to scroll using arrows, and seems like here the arrows work but something else doesn't.
Assignee | ||
Comment 36•15 years ago
|
||
This fixes the testcase.
I uploaded the patch to tryserver and will post link to the builds once the
builds are ready.
Assignee | ||
Comment 37•15 years ago
|
||
Comment 38•15 years ago
|
||
Comment 39•15 years ago
|
||
Thanks for the build; I have just tested it using hotmail and it fixes the issue regarding the scroll arrow; but if you click on the scrollbar space it still does not scroll down (see screenshot). I suspect this is a second (probably related) issue -- do you want me to put it in a fresh bug?
Comment 40•15 years ago
|
||
One correction, the right hand scroll-bar does work as expected, but the left-hand one doesn't. Both can be grabbed and dragged.
Assignee | ||
Comment 41•15 years ago
|
||
Sorry, I don't understand what does "right hand scroll-bar" mean?
Comment 42•15 years ago
|
||
If you look at the attached image you see that I circled the spaces within two scroll-bars. The left-hand scroll-bar is next to the list of emails and if you click within that space it should scroll down to where you click but doesn't. The right hand scroll-bar is next to the mail message and does scroll down when you click in the space. Hope that's a bit clearer; if not mail me directly and I'll chat to you on IRC or whatever. Cheers.
Assignee | ||
Comment 43•15 years ago
|
||
I'll post this to tryserver.
Attachment #438259 -
Attachment is obsolete: true
Assignee | ||
Comment 44•15 years ago
|
||
Look for scrollarrow_2 in https://build.mozilla.org/tryserver-builds/?C=M;O=D
in an hour or two
Comment 45•15 years ago
|
||
Excellent; that fixed the issue, both sliders work as expected. Will this fix also be landed on 1.9.2? Thanks, much appreciated.
Assignee | ||
Comment 46•15 years ago
|
||
Comment on attachment 438280 [details] [diff] [review]
v2
Neil, what do you think of this?
And yes, this should land on 1.9.2.x, and that was the reason why I took this pretty safe approach here.
I'll write tests for this.
Attachment #438280 -
Flags: review?(enndeakin)
Comment 47•15 years ago
|
||
Thanks!
Assignee | ||
Comment 48•15 years ago
|
||
The tests pass at least on Linux and OSX, and I posted the patch to
tryserver to verify that it works also on Windows.
Attachment #438280 -
Attachment is obsolete: true
Attachment #438357 -
Flags: superreview?(neil)
Attachment #438357 -
Flags: review?(enndeakin)
Attachment #438280 -
Flags: review?(enndeakin)
Updated•15 years ago
|
Attachment #438357 -
Flags: superreview?(neil) → superreview+
Comment 49•15 years ago
|
||
I can confirm testing with Win XP Pro SP3 and the build from opettay@mozilla.com-scrollarrow2. It passed all the tests I did with the hotmail email web page. If you want me to do any other testing let me know.
Comment 50•15 years ago
|
||
Just tested with the build at https://build.mozilla.org/tryserver-builds/opettay@mozilla.com-scrollarrow2_with_tests/ using XP SP3 and this also works using hotmail.com.
Comment 51•15 years ago
|
||
Updated•15 years ago
|
Attachment #438357 -
Flags: review?(enndeakin) → review+
Comment 52•15 years ago
|
||
Comment on attachment 438357 [details] [diff] [review]
+tests
>+ function() {
>+ ok(true, "Setting location.hash should scroll.");
>+ nextTest();
>+ // Click the top scroll arrow.
>+ var x = scroller.getBoundingClientRect().width - 5;
>+ synthesizeMouse(scroller, x, 5, { type : "mousedown" }, window);
>+ synthesizeMouse(scroller, x, 5, { type: "mouseup" }, window);
You could just call synthesizeMouse once and leave out the type.
I suppose the test happens to work on Mac, even when the scroll up button is at the bottom of the scroll bar?
Assignee | ||
Comment 53•15 years ago
|
||
(In reply to comment #52)
> You could just call synthesizeMouse once and leave out the type.
I know. I just wanted to be more clear what is actually tested here.
> I suppose the test happens to work on Mac, even when the scroll up button is at
> the bottom of the scroll bar?
Uh, right. Well it just happens to work :)
Assignee | ||
Comment 54•15 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/1fd66a6b0129
Will nominate for 1.9.2.x if the tests pass also on trunk tbox.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•15 years ago
|
Attachment #438357 -
Flags: approval1.9.2.4?
Comment 55•15 years ago
|
||
Comment on attachment 438357 [details] [diff] [review]
+tests
a=LegNeato for 1.9.2.4 as the patch doesn't look too risky and this is a 3.6 regression
Attachment #438357 -
Flags: approval1.9.2.4? → approval1.9.2.4+
status1.9.2:
--- → wanted
Assignee | ||
Comment 56•15 years ago
|
||
Comment 57•15 years ago
|
||
Can confirm this is working for hotmail for me on XP in v.3.6.4 Beta (Build 1)
Updated•14 years ago
|
blocking2.0: ? → final+
You need to log in
before you can comment on or make changes to this bug.
Description
•