Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Cannot delete too many files at once in new Resource Manager #5349

Closed
7 of 10 tasks
david-poindexter opened this issue Oct 21, 2022 · 1 comment · Fixed by #5359
Closed
7 of 10 tasks

Cannot delete too many files at once in new Resource Manager #5349

david-poindexter opened this issue Oct 21, 2022 · 1 comment · Fixed by #5359
Assignees
@david-poindexter
Labels
Area: Platform > Admin Modules Effort: Medium Priority: High Status: Ready for Development Type: Bug
Milestone
Future: Patch

Comments

@david-poindexter
Copy link
Contributor

Description of bug

When attempting to delete many files in the new Resource Manager, the UI does not allow for scrollable content.

Steps to reproduce

  1. Go to 'Site Assets' or Global Assets.
  2. Upload a lot of files into the 'Images' folder.
  3. Select a significant number of the files and click Delete.
  4. See UI problem in screenshot below.

Current behavior

The content is too tall for the page and the buttons are no longer viewable.

Expected behavior

Content should be scrollable within the modal and limited to a percentage of the view height, allowing room for buttons.

Screenshots

image

Error information

n/a

Additional context

n/a

Affected version

  • 10.00.00 alpha build
  • 09.11.00 release candidate
  • 09.10.02 release candidate
  • 09.10.01 latest supported release

Affected browser

  • Chrome
  • Firefox
  • Safari
  • Internet Explorer 11
  • Microsoft Edge (Classic)
  • Microsoft Edge Chromium
@jeremy-farrance
Copy link
Contributor

Noticed this in the DNN Community Forums, related.

Version 9.11 - vertical scroll in site assets changed
https://dnncommunity.org/forums/aft/3171

@david-poindexter david-poindexter self-assigned this Oct 22, 2022
@david-poindexter david-poindexter added this to the Future: Patch milestone Oct 22, 2022
valadas added a commit to valadas/Dnn.Platform that referenced this issue Oct 24, 2022
@valadas
Fixed an issue where resource maanger modal could not contain long lists
05de772 
Closes dnnsoftware#5349

This was fixed on the dnn-modal itself in DNNCommunity/dnn-elements#633 and this PR simply bumps our dependency on dnn-elements to bring in that fix.
@valadas valadas mentioned this issue Oct 24, 2022
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@david-poindexter david-poindexter

Labels
Area: Platform > Admin Modules Effort: Medium Priority: High Status: Ready for Development Type: Bug
Projects
None yet
Milestone
Future: Patch
Development

Successfully merging a pull request may close this issue.

Fixed an issue where resource manager modal could not contain long lists valadas/Dnn.Platform
2 participants
@jeremy-farrance @david-poindexter