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

Moved private link icon to "links" section #2496

Merged
merged 1 commit into from
Nov 26, 2019

Conversation

Julian1998
Copy link
Contributor

Removed private links icon next to breadcrumb as well as the file destails and added it instead in… the links section

Description

  • remove private link from breadcrumb
  • remove private link from the top of the files sidebar
  • add private link icon in the "Links" section

Related Issue

How Has This Been Tested?

  • tested on chrome, firefox and small devices
  • as I didn't change any functionality, everything should work as expected

Screenshots (if appropriate):

Bildschirmfoto von »2019-11-19 17-12-26«

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:

  • ...

@Julian1998 Julian1998 added the Status:Needs-Review Needs review from a maintainer label Nov 19, 2019
@Julian1998 Julian1998 self-assigned this Nov 19, 2019
@PVince81
Copy link
Contributor

expecting some acceptance test failures to happen as the element has moved

@ownclouders
Copy link
Contributor

💥 Acceptance tests PrivateLinks failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6559/

20191119-163352-509.png
20191119-163352-722.png
20191119-163352-945.png
20191119-163353-227.png
20191119-163353-573.png
20191119-163434-029.png
20191119-163434-397.png
20191119-163434-628.png
20191119-163435-055.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests XGAPortrait failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6559/

20191119-163748-058.png
20191119-163748-164.png
20191119-163748-288.png
20191119-163748-424.png
20191119-163748-521.png
20191119-163821-758.png
20191119-163821-896.png
20191119-163822-127.png
20191119-163822-262.png
20191119-163822-373.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests iPhone failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6559/

20191119-164048-635.png
20191119-164048-718.png
20191119-164048-799.png
20191119-164048-897.png
20191119-164049-022.png
20191119-164049-138.png
20191119-164124-375.png
20191119-164124-445.png
20191119-164124-587.png
20191119-164124-677.png
20191119-164124-782.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests Files failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6559/

20191119-163325-941.png
20191119-163405-381.png
20191119-163444-172.png
20191119-163600-354.png

@PVince81
Copy link
Contributor

@Julian1998 please adjust the tests

@Julian1998 Julian1998 force-pushed the feature/private-link-in-links-section branch from 98da625 to f0f555f Compare November 25, 2019 16:19
@ownclouders
Copy link
Contributor

💥 Acceptance tests PrivateLinks failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6764/

20191125-162831-338.png
20191125-162831-562.png
20191125-162831-768.png
20191125-162831-995.png
20191125-162832-206.png
20191125-162908-389.png
20191125-162908-617.png
20191125-162908-845.png
20191125-162909-088.png

@Julian1998 Julian1998 force-pushed the feature/private-link-in-links-section branch from f0f555f to a72e1d0 Compare November 25, 2019 16:40
@ownclouders
Copy link
Contributor

💥 Acceptance tests PrivateLinks failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6765/

20191125-165232-311.png
20191125-165232-963.png
20191125-165233-564.png
20191125-165233-911.png
20191125-165234-365.png
20191125-165325-483.png
20191125-165325-895.png
20191125-165326-389.png
20191125-165326-768.png

@Julian1998 Julian1998 force-pushed the feature/private-link-in-links-section branch from a72e1d0 to 8bffa78 Compare November 25, 2019 17:03
@ownclouders
Copy link
Contributor

💥 Acceptance tests Files failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6767/

20191125-171101-966.png
20191125-171137-770.png
20191125-171232-129.png
20191125-171326-357.png

@ownclouders
Copy link
Contributor

💥 Acceptance tests Trashbin failed. Please find the screenshots inside ...

https://drone.owncloud.com/owncloud/phoenix/6767/

20191125-173926-907.png
20191125-174039-576.png

@Julian1998 Julian1998 force-pushed the feature/private-link-in-links-section branch from 8bffa78 to 207b36a Compare November 25, 2019 18:29
@Julian1998
Copy link
Contributor Author

@PVince81 Tests passed. Please review again.

Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 6b69915 into master Nov 26, 2019
@delete-merged-branch delete-merged-branch bot deleted the feature/private-link-in-links-section branch November 26, 2019 13:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Needs-Review Needs review from a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Private link icon in "Links" section
3 participants