stable
Clone or download
part of: story #22576 have multiple files uploaded with comments The drop zone is currently a combination of two drop zones: - One on the comment/attachments pane - One on the test buttons pane I could have wrapped the two panes in a parent pane, but this approach was breaking the layout, and given the multiple displayable panes sections, fixing them is a too big challenge (display flex + overflow auto === 💀). How to test: - Pick up a test - Click on [Edit comment] - Drag something over the drop zone --> It highlights --> A message "You can drop blablabla" is displayed - When the user has not the right to update the attachment field in the execution tracker --> Dnd is not available Change-Id: I4a8ec0b4430ef019d879fb67787a527e21539ed3
Modified Files
Name | ||||
---|---|---|---|---|
M | plugins/testmanagement/include/TestManagement/IndexPresenter.class.php | +7 | −0 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/po/fr.po | +7 | −4 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/src/app-controller.js | +6 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone-controller.js | +108 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone-controller.test.js | +226 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone-message.js | +63 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone-message.test.js | +82 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone-message.tpl.html | +6 | −0 | Go to diff View file |
A | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments-drop-zone.js | +31 | −0 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/src/execution/execution-attachments/execution-attachments.tpl.html | +1 | −1 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/src/execution/execution-detail.tpl.html | +37 | −31 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/src/execution/execution.js | +4 | −0 | Go to diff View file |
M | plugins/testmanagement/scripts/testmanagement/src/shared-properties/shared-properties-service.js | +11 | −0 | Go to diff View file |
M | plugins/testmanagement/templates/index.mustache | +1 | −0 | Go to diff View file |
A | plugins/testmanagement/themes/BurningParrot/css/includes/_attachments_dnd.scss | +60 | −0 | Go to diff View file |
M | plugins/testmanagement/themes/BurningParrot/css/includes/_current-execution.scss | +38 | −2 | Go to diff View file |