Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[4] Uploading and overwriting media generates blank space instead of messages #30880

Closed
PhilETaylor opened this issue Oct 2, 2020 · 3 comments

Comments

@PhilETaylor
Copy link
Contributor

Steps to reproduce the issue

Joomla 4 admin
Media Manager

one way:

Drag 4 files from your desktop to drop in the media manager
Note they upload ok
Drag the same 4 files from your desktop to drop in the media manager
Note you get asked if you want to overwrite - say ok 4 times
note the huge blank space at the top of the page now

Another one way:

Select 4 items (holding down shift to multi select them)
Click delete
Click delete in warning popup
Note flashing as messages are added
Note blank space above "Item Deleted" message
Note that if you select 40 items then the space is then 10x the size

Video

Too large for Github. view here: https://screenshot.phil-taylor.com/2Nuj2Kgz

Expected result

Any messages should be shown
if there are 10 items deleted then message should say "10 items deleted" and not show 10 x "Item deleted"
"blank space" should not be there,

Actual result

As explained above.

screenshot with highlighted dragger the space to revel blank area divs:

Screenshot 2020-10-02 at 19 26 42

System information (as much as possible)

Additional comments

In source, there are a messages in that "blank space" just not visible

Screenshot 2020-10-02 at 19 23 06

@brianteeman
Copy link
Contributor

I can confirm the bug. The problem is that instead of the alert being destroyed it is set to display:none

It doesnt happen all the time which is even more weird

@Quy
Copy link
Contributor

Quy commented Feb 7, 2021

Related #29152

@Quy
Copy link
Contributor

Quy commented Mar 25, 2021

Please test PR #32868

@Quy Quy closed this as completed Mar 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants