Bug 1036397 - When bulk uploading images, the messages is missing the list of ids
Summary: When bulk uploading images, the messages is missing the list of ids
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: Web-UI
Version: 1.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.3
Assignee: Matthew Casperson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-12-01 20:26 UTC by Matthew Casperson
Modified: 2014-08-04 22:27 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-03 22:16:24 UTC


Attachments (Terms of Use)
Wrapped Overflow Example (49.33 KB, image/png)
2013-12-02 00:46 UTC, Lee Newson
no flags Details
Overflow Example (6.35 KB, image/png)
2013-12-02 00:46 UTC, Lee Newson
no flags Details

Description Matthew Casperson 2013-12-01 20:26:40 UTC
When bulk uploading a number of images, the success message should list the image ids. At the moment, that list of missing.

Comment 1 Matthew Casperson 2013-12-01 22:16:19 UTC
This should now be fixed.

Comment 2 Lee Newson 2013-12-02 00:39:59 UTC
Failing QA on this because if the image id list is too long and then the ids disappear out side of the div and eventually off the screen. Also if it did wrap then the okay button messes up the screen as it forces the div to be larger than the window.

Screenshots to come.

Comment 3 Lee Newson 2013-12-02 00:46:06 UTC
Created attachment 831343 [details]
Wrapped Overflow Example

Note: I didn't upload the large amount of images but simulated the output based on the current code.

Comment 4 Lee Newson 2013-12-02 00:46:35 UTC
Created attachment 831344 [details]
Overflow Example

Comment 5 Matthew Casperson 2013-12-02 03:02:15 UTC
Fixed in 201312021241

Comment 6 Lee Newson 2013-12-02 07:15:49 UTC
Verified in the above build. The alert box now has a max size and when it overflows, it shows a scrollbar to be able to see the extra content.


Note You need to log in before you can comment on or make changes to this bug.