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

[full-ci] Drag & drop on Breadcrumb parent folder #9052

Merged
merged 30 commits into from May 31, 2023

Conversation

lookacat
Copy link
Contributor

@lookacat lookacat commented May 16, 2023

!Please review first (as this PR is based on that):#8984

Description

See #9043

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

Open tasks:

@lookacat lookacat marked this pull request as ready for review May 17, 2023 10:10
@lookacat lookacat force-pushed the drag-drop-parent-folder branch 2 times, most recently from 86dbc77 to b937913 Compare May 26, 2023 08:01
@ownclouders
Copy link
Contributor

ownclouders commented May 26, 2023

Results for e2e-tests oC10 https://drone.owncloud.com/owncloud/web/36161/11/1

💥 To see the trace, please open the link in the console ...

npx playwright show-trace https://cache.owncloud.com/public/owncloud/web/36161/tracing/copy-and-move-resources-in-personal-space-alice-2023-5-30-09-55-03.zip

@lookacat lookacat changed the title Drag & drop on Breadcrumb parent folder [full-ci] Drag & drop on Breadcrumb parent folder May 30, 2023
@lookacat lookacat requested a review from kulmann May 30, 2023 09:22
Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nitty gritty detail: could you look into why the styling is getting cut off by the drop shadow of the left sidebar?

Screenshot 2023-05-30 at 15 39 25

@lookacat lookacat requested a review from kulmann May 30, 2023 14:32
@sonarcloud
Copy link

sonarcloud bot commented May 31, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

53.7% 53.7% Coverage
0.0% 0.0% Duplication

@lookacat lookacat requested a review from JammingBen May 31, 2023 10:49
Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳

@sonarcloud
Copy link

sonarcloud bot commented May 31, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 3 Code Smells

56.7% 56.7% Coverage
0.0% 0.0% Duplication

@lookacat lookacat merged commit 16d2d20 into master May 31, 2023
4 checks passed
@delete-merged-branch delete-merged-branch bot deleted the drag-drop-parent-folder branch May 31, 2023 13:10
ownclouders pushed a commit that referenced this pull request May 31, 2023
* WIP

* WIP

* Moving folders works

* Refactor breadcrumb dropped method

* Implement drop hover effect

* Implement edge case error handling: same folder name

* Remove dev leftover

* Implement experimental styling

* remove border, undo experimental breadcrumb styling

* Make drop on root work

* Add unittests for edge case

* Fix rebase errors

* Fix rebase issues

* Fix disable drag/drop on current folder

* Remove unused import / linting

* Implement e2e tests

* Rename dragover class

* Add changelog

* Address PR issues

* Fix unittest

* Fix e2e test for oc10

* Address PR issue

* Set overflow visible

* Fix buttons having different font feature settings

* Address PR issue

* Refactor AppBar drop logic into GenericSpace

* Remove unused import

* Address PR issues

* remove unused resize event listener

* Make linter happy
@micbar micbar mentioned this pull request Jul 24, 2023
68 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[web] drag and drop to parent Move files in parent folder via Drag&Drop not possible
4 participants