Ticket #2173 (closed New Feature: fixed)

Opened 21 months ago

Last modified 20 months ago

QuickUpload should show some info about the upload

Reported by: alfonsoml Owned by: alfonsoml
Priority: Normal Milestone: FCKeditor 2.6.1
Component: General Version: FCKeditor 2.6
Keywords: Review+ Cc:

Description

When the user presses the Upload button, there's no indicator that something is happening.

Reusing the dialog's throbber seems like an easy solution.

Attachments

2173.patch (3.1 KB) - added by alfonsoml 21 months ago.
Proposed SVN patch
2173_2.patch (4.3 KB) - added by alfonsoml 21 months ago.
Revised patch

Change History

Changed 21 months ago by alfonsoml

Proposed SVN patch

Changed 21 months ago by alfonsoml

  • keywords Review? added
  • status changed from new to assigned

The change is basically allowing the throbber parent to remain in the DOM and then calling it from the dialogs as needed.

Changed 21 months ago by martinkou

  • keywords Review- added; Review? removed

There are two problems with the current implementation:

  1. If the user tries to upload two or more files via the upload tab, then the throbber bar would get longer with each upload. This is actually my fault, though. It happens because the Throbber object in fckdialog.html is designed to show the throbber once only (note that throbber bar is added by DOMNode.appendChild() in the Show() function).
  2. While the throbber is being shown on the dialog, the "OK" button and the tabs should be disabled.

Changed 21 months ago by alfonsoml

Revised patch

Changed 21 months ago by alfonsoml

  • keywords Review? added; Review- removed

I've addressed both issues in the new patch.

Changed 20 months ago by martinkou

  • keywords Review+ added; Review? removed

Changed 20 months ago by alfonsoml

  • status changed from assigned to closed
  • resolution set to fixed
  • milestone set to FCKeditor 2.6.1

Fixed with [2043]

Note: See TracTickets for help on using tickets.