Project

General

Profile

Actions

Feature #24123

closed

Loading indicator just displays if module is changed

Added by Kay Strobach about 14 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2010-11-18
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:
Needs Decision

Description

He guys,

changing the module shows the new loading indicator (nice).
But doing the following actions doesn't display a loading indicator (even if the page loads):

- hitting edit button
- hitting save
- initiating http requests from within the webmodule (other modules)

PS: I do like the idea of the loading indicator.
(issue imported from #M16464)

Actions #1

Updated by Steffen Gebert about 14 years ago

I hate the idea of the loading indicator ;-)

It's only shown, when a frame is changed through the API, not when a simple link is clicked. Think it should be no problem, to make it consistent for sb. with ExtJS knowledge (you? g). Can you provide a patch?

Actions #2

Updated by Kay Strobach about 14 years ago

can be solved using the managed iframes in extjs:

http://www.sencha.com/learn/Extension:ManagedIframe

I'm really busy at this point, sry - but i can't spend enough time to dig into the complete backend extjs and find the point for the fix. I have a lot projects which are in the last phase before release. (Very stressfull part - you know).

Actions #3

Updated by Chris topher about 14 years ago

Saying I hate the loading indicator only is a personal opinion. Although I think the same, it alone is only a matter of taste, but no reason to remove it.

But there in fact are good reasons which should make us remove or at least drastically modify this indicator again.
- Some people say this indicator looks nice. But "looking nice" is not the most important thing a CMS should do. We are not composing a picture here, but something you must be able to work with in a reasonable fashion.
- You might argue that this indicator tells you that there is something happening in the background and that this is the reason why you have to wait. But that is not convincing. You can already see the loading progress in the status bar. We don't need another loading indicator, less than ever one which fills the whole screen and makes the system unusable when it is displayed. And if the loading in the background does not continue (e.g. because your network connection shortly breaks in between), you still see the indicator although nothing is happening at all. In that case the loading indicator only pretends that something is happening while in fact you are waiting for nothing while you maybe could already use the BE (because maybe only a picture is missing). You are waiting for nothing.
- The loading indicator makes the backend feel slow, much slower than the BE of current versions feels. It is quite boring to see always this turning icon while nothing is happening. There already where people in the newsgroups who legitimately complained about that. The indicator always again and again reminds you that you have to wait and that you cannot do anything but wait. Being told that you must wait is annoying and must be avoided.
- But the most important reason to remove the loading indicator is that it makes TYPO3 objectively slower. It does not only feel slower, but working with TYPO3 really becomes slower. Before you could at least have a look at the loaded module and you could much more easily get a first orientation on where to click next and begin moving the mouse there. With the indicator that has become much harder. You do not see the module content until the disappearing of the indicator allows you to.

That is why we have to remove (or at least drastically improve) this indicator again.

Actions #4

Updated by Chris topher about 14 years ago

There also has already been a discussion in Core List (#23614) where people agreed to modify the loading indicator.
The idea to solve this problem was to put a small loading indicator in a top corner of the reloading frame. This prevents all the negative effects listed above while it still gives you the information where something is loaded. I think that really is a good idea.
This has not yet been fixed because Ext JS does not yet offer the needed options itself. Steffen K wanted to ask the developers of Ext JS and maybe write the functions himself, if needed.

Actions #5

Updated by Chris topher about 14 years ago

Kay, could you please also have an eye on #23614 and think of it before the final version is released? That would be great!

Actions #6

Updated by Kay Strobach about 14 years ago

ofcourse.

@Loading indicator:
I'm not sure if i like / dislike the loading indicator (Therefor we have UI specialists.). But in my opinion it must be consistent if there is one or not.

#0015811 is about removing the indicator that would be ok for me too.

Actions #7

Updated by Chris topher about 14 years ago

Yes, the discussion went to another direction (showing a small indicator in a corner of the BE). I can live with that, too.

But as you say: It should be consistent: Everywhere or nowhere.
(Usabilitywise: Please first make it small and THEN introduce it everywhere, not the other way around. ;-) )

Actions #8

Updated by Alexander Opitz over 11 years ago

  • Target version deleted (0)

What is the state of this issue? ExtJS isn't updated and nothing changed here IMHO, but the idea sounds good. :-D

Actions #9

Updated by Kay Strobach over 10 years ago

don't know, close it for now, as it is ignored anyway?

Actions #10

Updated by Alexander Opitz over 10 years ago

  • Tracker changed from Bug to Feature
  • Category set to Backend User Interface
  • Status changed from Needs Feedback to New

I mark this as feature, but this may need to wait for switching away from ExtJS

Actions #11

Updated by Susanne Moog over 9 years ago

  • Sprint Focus set to PRC
Actions #12

Updated by Christian Kuhn about 9 years ago

  • Status changed from New to Closed

I think we can safely close this issue now. The loading spinner was substituted with at most places already in 7.

Actions #13

Updated by Benni Mack over 4 years ago

  • Sprint Focus changed from PRC to Needs Decision
Actions

Also available in: Atom PDF