Closed Bug 482985 Opened 16 years ago Closed 9 years ago

Loading/busy/spinning mouse indicator/pointer/icon missing

Categories

(Firefox :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME
Tracking Status
status1.9.1 --- wontfix

People

(Reporter: pkc, Assigned: mstange)

References

(Depends on 1 open bug)

Details

(Keywords: common-issue-, regression, uiwanted, Whiteboard: [parity-chrome][parity-IE])

Attachments

(3 files, 1 obsolete file)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090305 Firefox/3.2a1pre Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090305 Firefox/3.2a1pre As indicated in Bug 481359, it has been arbitrarily decided that the loading cursor on data transfer was bad and it was altogether removed. However this was actually useful for all those that do not use the status bar and throbber(which are actually harder to notice, especially since usually when you click a link you have your eyes on the link you click, not on the throbber). It was in many ways very useful as it indicated actual network activity and hence that removal would need to be made an option or if it is thought to be in the wrong place, recoded. Reproducible: Always
Blocks: 481359
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression
Product: Firefox → Core
QA Contact: general → general
Version: unspecified → Trunk
Also note that the throbber is not longer default so users now have to figure out which tab has focus and then look at that tabs favicon. Granted it is only a very small ammount of time to do that but who knows for people with disabilities trying to differentiate between an active and non active tab.
Note a core issue. The decision in bug 481359 was that if an app wants the behavior it can set the cursor override on the window. Of course an extension can do the same. I have no idea why anyone would want to differentiate between active and inactive tabs, though....
Product: Core → Firefox
QA Contact: general → general
Usually to find out which pages have loaded and are ready to be looked at.
(In reply to comment #2) > Note a core issue. The decision in bug 481359 was that if an app wants the > behavior it can set the cursor override on the window. Bug 481359 comment 6 doesn't seem to support that.
Product: Firefox → Core
QA Contact: general → general
(In reply to comment #4) > (In reply to comment #2) > > Note a core issue. The decision in bug 481359 was that if an app wants the > > behavior it can set the cursor override on the window. > > Bug 481359 comment 6 doesn't seem to support that. Yet bug 481359 comment 21 indicates that there is a JS mechanism to activate this.
Changing the cursor is not the issue. I believe actually knowing when to do it is more of the issue here.
Yeah, there's knowing when to do it, and also doing it reliably while maintaining the correct hover behavior for links etc. I'm not sure that that works correctly.
(In reply to comment #5) > Yet bug 481359 comment 21 indicates that there is a JS mechanism to activate > this. That mechanism alters the page. There isn't one that doesn't alter the page, although I guess one could be added to nsIDOMWindowUtils.
Chrome has the semi-loading cursor as well. This also seems to be the os standard on Windows, when a new folder is loaded in Windows explorer or a new file opened in most applications, the cursor turns to the loading cursor. Seems like this is pretty much expected by most users. Just because it wasn't 100% doesn't mean it should be removed, IMHO, it only needs to be made more accurate.
Flags: wanted1.9.1?
Keywords: uiwanted
Whiteboard: [parity-chrome]
How do people feel about a removing the loading cursor only on Mac OS X? See also bug 362623 comment 5, which suggests changing cursor: progress there, too.
I personally think that since things were originally reported by OS X users, it should only be removed there.
In fullscreen mode, there is now no possibility to monitor the page load progress. There is an urgent need, to have hourglass as an indicator.
(In reply to comment #14) > In fullscreen mode, there is now no possibility to monitor the page load > progress. There is an urgent need, to have hourglass as an indicator. Great point Maksym!
Keywords: common-issue?
Summary: Removal of loading mouse cursor on data transfer → Loading/busy/spinning mouse indicator/pointer/icon missing
Getting this reverted on non-OSX-systems would be nice. You always think nothing happens if you click a link and the cursor doesn't change.
Is there any easy way (i.e. without recompiling) to re-enable the loading cursor manually?
Not as far as this thread and some others went. I personally have been forced to compile my own builds with the patch that causes this issue removed. Some have hinted that it might be possible to have an extension to do it, but so far it does not seem to be that easy(mostly because it's hard to known when things are loading) That's why I originally created this bug report, because I could not find any simple way to solve the issue.
I recompiled with this [http://hg.mozilla.org/releases/mozilla-1.9.1/rev/4ad59d117236] patch being reverted, but there's still no loading cursor. Is there anything else which needs to be changed?
Are there any plans to fix this on non-OSX systems in 3.5.1? Or make it at least configurable via about:config?
Flags: wanted1.9.1? → wanted1.9.1.x?
This is _not_ a feature for users who are stuck with dial-up. I do not have access to other types of connection where I live, and to me this is a deal-breaker. Firefox just becomes needlessly hard to use, and that is very demotivating, considering there is no reason to omit an about:config tweak to re-enable the cursor. I sincerely do not understand why this patch was silently slipped in inbetween betas 3 and 4, without consideration for the implications. It was a rushed decision, and quite frankly, a mistake.
Attached patch only remove on OS X (obsolete) — Splinter Review
We've got several options: 1. Wontfix this bug, 2. put the code back in place, #ifndef'd for OS X, 3. Add a pref 3.1 defaulting to off on all platforms, or 3.2 defaulting to off only on OS X. I'm not the right person to make the decision. This patch implements 2.
Attachment #388666 - Flags: ui-review?(beltzner)
I believe the correct option is: 4. Implement it as part of Browser code. The main problem with original throbber was that it was a Core code, and this is where it doesn't belong. Bringing that implementation back is not what anyone wants. See comment #2.
OK, but then somebody else will have to do that. I don't know how and I don't want to do it.
(In reply to comment #27) > The main problem with original throbber was that it was a Core code, and this > is where it doesn't belong. Bringing that implementation back is not what > anyone wants. See comment #2. The assumption in comment #2 (and bug 481359) was that an app can (reliably) set the cursor override on the window, which is not currently possible - at least not without changing core interfaces - if I understand comment #7 and #8 correctly. > Bringing that implementation back is not what anyone wants. I don't think that is true, but that's a matter of opinion.
> We've got several options: > 1. Wontfix this bug, > 2. put the code back in place, #ifndef'd for OS X, > 3. Add a pref > 3.1 defaulting to off on all platforms, or > 3.2 defaulting to off only on OS X. I think either 2 or 3.2 are the best solutions. 1 or 3.1 wouldn't be consistent with other windows programs - almost all programs (including explorer and IE) display a busy-cursor when loading new data or when you do something which is not finished immediately (e.g. clicking a link loading new content).
(In reply to comment #26) > 3. Add a pref > 3.1 defaulting to off on all platforms, or Of your options, this is the only one that solves the issue that there's no feedback at all in popups or in full-screen mode. There's nothing platform-specific about that. Another option would be to backout bug 481359. (In reply to comment #27) > I believe the correct option is: > 4. Implement it as part of Browser code. There's no concrete proposal how this could actually be implemented, as far as I can see. > The main problem with original throbber was that it was a Core code, This seems to be a pure invention on your side.
Flags: blocking1.9.2?
(In reply to comment #31) > (In reply to comment #26) > > 3. Add a pref > > 3.1 defaulting to off on all platforms, or > > Of your options, this is the only one that solves the issue that there's no > feedback at all in popups or in full-screen mode. Err, I misread your proposed solution. Only if the pref would default to 'on' would it solve the problem, as the use cases in question aren't uncommon in such a way that only advanced users would want to opt in to the old behavior.
Really not a core issue. If you need additional core APIs here, file bugs about them. But the app folks wanted to be freed from core forcing a cursor here, and they have that. Please don't move the bug to core again, ever.
Flags: blocking1.9.2?
Product: Core → Firefox
QA Contact: general → general
And to be even more clear, if you want to set the cursor in app code you can do that already. If you can't figure out _when_ to do that, that's more or less your problem; neither can the docshell code. App has strictly more information here than docshell does. If the issue is that you want to set the cursor only in some states and not others, then it sounds like we need a new magic cursor value for that; that would be the "additional core api" I mention in comment 36. If you want to just back out bug 481359 while you figure out the right way to fix this and then put that patch back in, I can probably buy that. But let's stop thrashing the core code here every time the Firefox UI wants changing. And just to be very clear, I am strongly opposed to any platform ifdefs in this code if they can be avoided. If we have to have per-platform behavior in the core (which I'm not seeing any need for!) it'll be a pref.
Attachment #388666 - Flags: ui-review?(beltzner) → review-
Flags: blocking-firefox3.6?
status1.9.1: --- → ?
Flags: wanted1.9.1.x?
(In reply to comment #43) > For me, visually disabled, this is a vital indicator. It is very hard for me to > find the "active" tab and try to see if the little "busy" icon is moving. The > "ANY" page indicator was larger and in a fixed location. One always knew what > was going on with the old functionality. Now we don't. That doesn't sound like you're talking about the busy _cursor_. Do you perhaps mean the throbber which used to be in the upper right corner? It was removed by default, but you can easily reenable it by right-clicking on the menu bar and choosing "Customize", then dragging it wherever you want it. This bug is about changing the cursor to inidicate activity, which many people (including myself) want back too.
This is a very simple addon that just adds a tiny amount of CSS to the old page (before the new page start loading) when a link is clicked (or such) which makes all elements on the page display the progress cursor. I have submitted it here because I dont think its a good solution in any way and could easily make page rendering problems. Feel free to take on the idea and submit a real addon.
(In reply to comment #46) > I have submitted it here because I dont think its a good solution in any way > and could easily make page rendering problems. Feel free to take on the idea > and submit a real addon. Works awesomely for my current needs (activity indicator while in full screen). Thanks!
> This hardly works as a real solution though. I know this is not a solution in any way. Interacting with the actual source of the webpages was not my original aim but I could not work out how to just apply the cursor change on the tab/browser object. And if the cursor setting is set in the element's style attribute then the cursor will not change anyway with this. > Plus it will not work properly on AJAX content. For the AJAX page activity I think that should be up to the actual creators of the AJAX page since they may not want the cursor flickering while their scripts load extra parts for display.
If bug 309547 ever got fixed, that would make it trivial for an extension to restore the loading mouse pointer, since it could just force it on in response to the same event that turns the throbber on and off.
Won't block 3.6 on this, and I don't think we want to back-port any change here. The cursor was removed because the cursor is meant to indicate that the system is busy loading a resource and the user can't interact with it. As others have mentioned, if we can fix our code so that we can understand at what point a user can interact with the loaded content, and only show the "loading" cursor between the point when a new resource is loaded and when the user can interact with it, I'm fine to put it back in. Otherwise, it's better out than in, I'm afraid.
Flags: blocking-firefox3.6? → blocking-firefox3.6-
(In reply to comment #55) > The cursor was removed because the cursor is meant to indicate that the system > is busy loading a resource and the user can't interact with it. CSS distinguishes between "wait" and "progress", both of which we implement. So if that was the motivation (this wasn't clear to me until now), then the cursor should have been replaced with one that doesn't indicate that the user can't interact with the app, rather than being removed.
Whiteboard: [parity-chrome] → [parity-chrome][parity-IE]
(In reply to comment #59) > Nevermind on the second part of my comment, just noticed it already uses this > cursor. In that case, "The cursor was removed because the cursor is meant to indicate that [...] the user can't interact with it" doesn't make sense.
(In reply to comment #56) > the cursor > should have been replaced with one that doesn't indicate that the user can't > interact with the app, rather than being removed. Indeed. As said before, the cursor is sometimes the only UI indication that something is going on. If the cursor doesn't change, then the obvious conclusion is that nothing is going on. So if I click on a link, and nothing is going on, what do I do? I click again. And again. Well, still nothing. Let's double-click then? Nope, sill nothing. Middle-click? Nope. Middle-double-click? And when the user leaves full-screen, there's five tabs of the same slow loading page (bonus points if the page loads some real-time report with lots of complicated math on massive datasets). I had to fight to make some of my users understand that the cursor tells them if their click succeeded, and there's no need to click five time on a link, because it won't make it load any faster. And now the cursor doesn't change anymore...
Attachment #388666 - Flags: ui-review?(beltzner)
Attachment #388666 - Flags: review?(bzbarsky)
Attachment #388666 - Flags: review-
Is this just backing out bug 481359 but adding ifdefs? If so, I'd prefer a pref, not ifdefs.
(In reply to comment #62) > Is this just backing out bug 481359 but adding ifdefs? Yes, since it seems that bug 481359 happened just because the cursor on OS X is too bold and misleading. > If so, I'd prefer a pref, not ifdefs. Markus, can you do that?
Comment on attachment 388666 [details] [diff] [review] only remove on OS X Per comment 37 (see the part about platform ifdefs there) and comment 62
Attachment #388666 - Flags: review?(bzbarsky) → review-
(In reply to comment #63) > > If so, I'd prefer a pref, not ifdefs. > > Markus, can you do that? OK. Any idea on a good name? ui.use_progress_cursor?
Assignee: nobody → mstange
ui.use_activity_cursor maybe?(since it's a cursor on active transfer rather than on progress) Either way, glad to see things moving.
Uses pref ui.use_activity_cursor, defaulting to false.
Attachment #388666 - Attachment is obsolete: true
Attachment #401141 - Flags: ui-review?(beltzner)
Attachment #401141 - Flags: review?(bzbarsky)
Attachment #388666 - Flags: ui-review?(beltzner)
Comment on attachment 401141 [details] [diff] [review] implement ui.use_activity_cursor (checked in) Looks good, but should there be an ifdef here in all.js enabling the cursor for non-mac?
Attachment #401141 - Flags: review?(bzbarsky) → review+
Let's have Beltzner make the decision. I don't have an opinion on it.
Why is this waiting for ui-review if it doesn't change behavior? This could land as is, as far as I can see, and either way we should get a patch up for ui-review that fixes this by default for Windows and Linux, given the increased indications that something went wrong in bug 481359.
You're right, this can land as-is. Feel free to do so.
Attachment #401141 - Attachment description: v2 → implement ui.use_activity_cursor (checked in)
Attachment #401141 - Flags: ui-review?(beltzner) → approval1.9.2?
Attachment #401212 - Flags: ui-review?(beltzner)
Attachment #401212 - Flags: review?(bzbarsky)
Attachment #401212 - Flags: review?(bzbarsky) → review+
Depends on: 517272
Depends on: 517768
(In reply to comment #79) > Hi, any chance this gets into 1.9.1? This has been WONTFIX'd on 1.9.1. So the only way people can have this on 1.9.1 is if somebody creates an extension. =/
Blocks: 517272
No longer depends on: 517272
Depends on: 534538
Comment on attachment 401141 [details] [diff] [review] implement ui.use_activity_cursor (checked in) Don't think we want to take this on the branch without bug 534538 addressed.
Attachment #401141 - Flags: approval1.9.2?
Comment on attachment 401212 [details] [diff] [review] adjust pref defaults With ui.use_activity_cursor set true this regresses bug 487382
Comment on attachment 401212 [details] [diff] [review] adjust pref defaults I don't suppose someone else exists who can do this ui-review more quickly?
It's not in 3.6, but it's in Minefield (3.7a1pre). So I guess 3.6 probably has been WONTFIX'd. We'll have to see what Lorentz brings or Firefox.next (4.0?) brings. Or somebody could make an extension to bring this function back for those not using Minefield.
There are other Bug Reports that get marked as a "DUPE" to here so to avoid a separate posting I will mention my NEWer "Cursor Bug" here: Up until yesterday I had updated Namoroka regularly (I have 'Nightly' 3.6.2pre, today's Update is awaiting a restart) and had noticed no Cursor problems. I visited this page for the first time ever: http://en.ergoline.de/content/index.php/4/1424/2189/design_elements_-_the_ergoline_look.htm which leads to a '3-D QuickTime Player page' here: http://en.ergoline.de/content/popup/qtvr/soltron.php ). I've used the QT Player before but never been to that particular Webpage. Now if I move my Cursor I get the "Regular Pointer" but when I let go of the mouse the Cursor changes to a "Target" (it looks similar to "(.)" except the circle closes and the dot is central, it seem's to be Quicktime's Cursor but Namoroka does not restore it's own over the "Webpage Area"). This only occurs while mousing over the 'Webpage Area' in Namoroka, when I leave that area either for the Tabs, Menus, or a different Application's Window then the Cursor is "Normal" (a Pointer, usually). Rob
Depends on: 552193
Blocks: 573385
Blocks: 537015
(In reply to comment #92) > It's not in 3.6, but it's in Minefield (3.7a1pre). So I guess 3.6 probably has > been WONTFIX'd. We'll have to see what Lorentz brings or Firefox.next (4.0?) > brings. > > Or somebody could make an extension to bring this function back for those not > using Minefield. Are there any chances this is back ported to 3.6? Sugar [1] and it's Browse activity [2] are using this as an important user feedback when loading pages. Even in new releases like Fedora 14 [3] we have 3.6. I already patched F11 for OLPC releases [4]. [1] http://sugarlabs.org/ [2] http://wiki.sugarlabs.org/go/File:Browse_0.86_palettes_improved.png [3] https://bugzilla.redhat.com/show_bug.cgi?id=640581 [4] http://dev.laptop.org/ticket/10383
Blocks: 603594
Isn't this a dupe of bug 481359?
(In reply to comment #95) > Isn't this a dupe of bug 481359? No...
(In reply to comment #96) As so this isn't adding back what that removed? Sorry was just attempting to clarify.
This is a considerably bigger problem in Firefox 4 builds. With the throbber gone by default (I think?) and the status bar outright removed, the only remaining indication that a page is loading is the throbber in a tab's favicon. And the tab isn't in a consistent place, so you have to hunt for it first. I've been on 4 full-time for a couple weeks now, and several times I've found myself confused as to whether Firefox is actually doing anything, even with FIOS.
No longer blocks: 603594
Attachment #401212 - Flags: ui-review?(mbeltzner) → ui-review?(faaborg)
(In reply to comment #103) > I heard this was added back with a preference (ui.use_activity_cursor) but I > tried and it doesn't work in the final Firefox 4.0 version! Big > disappointment. Sounds like somebody isn't using a clean profile, cause the activity cursor preference in about:config works for me. Please go to SUMO to get help on your problem: http://support.mozilla.com/
The low-level functionality seems to be restored, but hasn't been enabled by default for Firefox. I don't think this issue is resolved until it's again enabled by default. This isn't esoteric behavior that only a few people want; this is basic user feedback, at least on Windows. You can enable it via "ui.use_activity_cursor" in about:config. I don't know if there are still issues preventing it from being used by default; it seems to be working for me, for what it's worth.
Depends on: 659573
(In reply to comment #106) > Vova, you should file a new Firefox enhancement request then. This bug is > for the missing mouse activity cursor from Firefox 3.5 and 3.6. I filed a new request for FF4 for this issue: https://bugzilla.mozilla.org/show_bug.cgi?id=659353
Blocks: 659573
No longer depends on: 659573
Comment on attachment 401212 [details] [diff] [review] adjust pref defaults While feedback is really important, I (and the rest of the UX team) feel that changing the cursor based on network state is overly noisy and distracting, and makes the UI less zen. I'm fine with this being a hidden pref, but we want the default UI to be calm and sleek (and of course still provide great feedback in a consistent location).
Attachment #401212 - Flags: ui-review?(faaborg) → ui-review-
This bug has been marked for Regression and or Closure Version 46.0.1 Build ID 20160502172042 ui.use_activity_cursor (T/F) works for me Considering this I will close this as Resolved-WFM Feel free to reopen if issue is still a concern.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
Duplicate of this bug: 531058
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: