"Content -> Files" accessibility links show up when attempting to drag and drop
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Mahara |
Fix Released
|
High
|
Aaron Wells | ||
| 1.10 |
Fix Released
|
High
|
Aaron Wells | ||
| 15.04 |
Fix Released
|
High
|
Aaron Wells | ||
| 15.10 |
Fix Released
|
High
|
Aaron Wells | ||
Bug Description
In 1.9 for accessibility purposes we added a feature to the "Content -> Files" screen, which makes it so that if you click on a thumbnail of a file, it shows you a list of locations you could move the file to. This is an accessible alternative to the drag-and-drop system for moving your files to different folders.
There are a couple of problems with this currently.
First, the patch that we added to make it work in IE, has the side effect that in Firefox and Chromium, the accessible alternative shows up when you start dragging and dropping, as opposed to when you just click on it. Also, the thumbnail shifts up and away from the mouse pointer at the same time, meaning you need to click on it, it pops open, then you have to move your mouse back over the thumbnail's new location.
To replicate:
1. Go to Content -> Files
2. Upload at least one file
3. Create a folder called "Test"
3. Move your mouse pointer over the thumbnail's icon and try to drag and drop the thumbnail icon onto the Test folder.
Expected result: You drag it onto the Test folder and drop it onto the Test folder. You do not see the accessibility options.
Actual result: When you depress the mouse button, the accessibility option shows up, and the thumbnail moves away out from under your pointer.
Using git bisect, I see this started with this patch, for IE support:
| tags: | added: regression |
| summary: |
- "Content -> Files" accessiblity links for moving files, get in the way - of drag-and-drop + "Content -> Files" accessibility links show up when attempting to drag + and drop |
| Mahara Bot (dev-mahara) wrote A patch has been submitted for review | #1 |
| Mahara Bot (dev-mahara) wrote A change has been merged | #3 |
| Aaron Wells (u-aaronw) wrote | #2 |
| Mahara Bot (dev-mahara) wrote A patch has been submitted for review | #4 |
| Mahara Bot (dev-mahara) wrote | #5 |
| Mahara Bot (dev-mahara) wrote A change has been merged | #6 |
| Mahara Bot (dev-mahara) wrote | #7 |
| Changed in mahara: | |
| status: | Fix Committed → Fix Released |
| tags: | added: behat-needed |
Patch for "master" branch: https:/