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

[9.2] Adding a button to the detailed view where the webdav url can be copi… #25757

Closed
wants to merge 2 commits into from

Conversation

DeepDiver1975
Copy link
Member

…ed to the clipboard

bildschirmfoto von 2016-08-10 14-59-34

The webdav url to a folder can now easily be copied to the clipboard using this new button on the detailed view.

This is valuable for users who want to mount a specific folder in e.g. their windows explorer, nautilus file manager and so on ....

@DeepDiver1975 DeepDiver1975 added this to the 9.2 milestone Aug 10, 2016
@cdamken
Copy link
Contributor

cdamken commented Aug 10, 2016

Thanks @DeepDiver1975 !

@PVince81
Copy link
Contributor

  • add some padding !
  • add tooltip, as a user discovering this I have no idea what it is about
  • optionally: change the "Copied" tooltip to "Copied the stuffz" (replace stuffz with whatever it is I copied)

Having these two buttons so close here doesn't feel comfortable and rather confusing. I wonder if we should put them both under the link part, basically it would expand with two fields instead of one. Might look ugly though.

@@ -18,6 +18,7 @@
'<span class="icon icon-public"></span>' +
'<span class="hidden-visually">{{permalinkTitle}}</span>' +
'</a>' +
'<a class="{{#unless showWebDavCopyButton}}hidden-visually{{/unless}} clipboardButton icon icon-external" data-clipboard-text="{{webDavUrl}}"></a>' +
Copy link
Contributor

Choose a reason for hiding this comment

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

it should be "hidden", not "hidden-visually".

"hidden-visually" is used for screen readers so they can still read an icon's label even when we don't want to show it visually.

- use hidden instead of hidden-visually
- add tooltip
- use notifications on copy of the dav url- tooltip is used for the description of the button
@PVince81
Copy link
Contributor

PVince81 commented Oct 5, 2016

I think it would be best to have both links in the dropdown from the permalink to avoid cluttering the UI.

@felixheidecke felixheidecke self-assigned this Feb 28, 2017
@PVince81 PVince81 modified the milestones: triage, 10.0 Jun 29, 2017
@PVince81
Copy link
Contributor

PVince81 commented Jul 4, 2017

@DeepDiver1975 update or close ?

conflicts detected. also see comments above

@PVince81
Copy link
Contributor

PVince81 commented Jul 4, 2017

considering that most users use the desktop client, I don't see much value in exposing the current Webdav path in such an exposed location. The Webdav path is rather something internal to ownCloud and would qualify as "advanced usage" so should maybe be put somewhere else.

@DeepDiver1975
Copy link
Member Author

considering that most users use the desktop client, I don't see much value in exposing the current Webdav path in such an exposed location.

the use case behind this is that a user can easily copy the web dav path to mount the folder in the operating system's file manager.

seems to be a legitimate use case - maybe? @pmaier1

originally requested by @MTRichards

@PVince81
Copy link
Contributor

PVince81 commented Jul 5, 2017

now one idea to keep both could be to do like github used to do and have a "tab button" on top of the link that says "Private link" and "Webdav link". It would show "Private link" by default but you can switch to "Webdav link" if you want the Webdav format of it. It's better than two separate fields or two separate buttons.

Apparently Github now changed the git URL and have it with a link instead to switch to "https" mode:
github-clone

@DeepDiver1975 DeepDiver1975 deleted the wevdav-link-in-details branch July 19, 2017 11:51
@felixboehm felixboehm removed this from the triage milestone Apr 10, 2018
@lock
Copy link

lock bot commented Jul 31, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jul 31, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants