プロジェクト

全般

プロフィール

Vote #81256

完了

"Copy link" feature for issue and issue journal

Admin Redmine さんが4年以上前に追加. 4年以上前に更新.

ステータス:
Closed
優先度:
通常
担当者:
-
カテゴリ:
UI_10
対象バージョン:
開始日:
2022/05/09
期日:
進捗率:

0%

予定工数:
category_id:
10
version_id:
152
issue_org_id:
34703
author_id:
259873
assigned_to_id:
259873
comments:
19
status_id:
5
tracker_id:
2
plus1:
0
affected_version:
closed_on:
affected_version_id:
ステータス-->[Closed]

説明

When you are looking at a issue with Redmine and want to share the URL, if you copy the URL with a browser, it will be a long URL with many unnecessary parameters such as issue_count and prev_issue_id.

I think it would be convenient to be able to copy a simple URL from the view.

  • URL of the issue to be copied: @https:///issues/1@
  • URL of the issue journal to be copied: @http:///issues/1#note-1@

journals

When I added the "Copy link" icon, it was crowded with many icons displayed on the view.
So this patch hides infrequently used action links in actions-dropdown, just like wiki pages.

!screenshot_2021-02-10_16.15.22.png!
!screenshot_2021-02-10_16.14.35.png!

If it is better to separate the patch that adds the function to copy the URL and the patch that combines the Action icons, it will be supported.
This patch contains an icon image, so apply it with the git apply command.
--------------------------------------------------------------------------------
For your reference, I've attached a patch with a pattern that doesn't combine the "Copy link" icon into actions-dropdown. (I don't like this UI very much)
--------------------------------------------------------------------------------
I really like the ideea of moving the Delete icons under the actions dropdown, I think we should extract it from your patch and added a separat tichet. From my point of view, this solution is a good fix for #31278.

Also, having the copy link it's a nice and useful feature.
--------------------------------------------------------------------------------
Marius BALTEANU wrote:
> I really like the ideea of moving the Delete icons under the actions dropdown, I think we should extract it from your patch and added a separat tichet. From my point of view, this solution is a good fix for #31278.

Good point there.

> Also, having the copy link it's a nice and useful feature.

I like the feature, too. Sharing a link in a chat is a bit tedious at the moment.

I've tried your patch and it worked quite well. The only thing I've missed was some kind of feedback after copying the URL to the clipboard. On all web applications which allowed me to copy stuff to the clipboard I got a toast or little message, that the job has finished. So I felt a bit awkward after clicking the copy link without anything happening.
--------------------------------------------------------------------------------
Thank you for your feedback.

Marius BALTEANU wrote:
> I really like the ideea of moving the Delete icons under the actions dropdown, I think we should extract it from your patch and added a separat tichet. From my point of view, this solution is a good fix for #31278.

I split the patch and attached it to the #34714 issue.

Bernhard Rohloff wrote:
> I've tried your patch and it worked quite well. The only thing I've missed was some kind of feedback after copying the URL to the clipboard. On all web applications which allowed me to copy stuff to the clipboard I got a toast or little message, that the job has finished. So I felt a bit awkward after clicking the copy link without anything happening.

I think if I put the "Copy link" link inside the Actions dropdown, pressing the copy button will close the Dropdown as feedback that the work is done.
GitHub and Gitlab also put a "Copy link" in the dropdown and do not show the message that the work is completed.
However, if you accidentally click outside the dropdown, the dropdown will close, which may be incomplete feedback in that you don't know if the copy is really complete.

In the patch for #34703#note-2, I have implemented a mechanism to display a message after the copy is completed, so I will try if I can incorporate that into the patch for #34703#note-1.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Since #34714 was committed, I modified feature-34703-1.patch accordingly.
I also added a System test.
--------------------------------------------------------------------------------
Bernhard Rohloff wrote:
> The only thing I've missed was some kind of feedback after copying the URL to the clipboard. On all web applications which allowed me to copy stuff to the clipboard I got a toast or little message, that the job has finished. So I felt a bit awkward after clicking the copy link without anything happening.

I agree that it would be even better to have feedback when the link is copied.

But how about committing the current patch first, and improving the feedback display in the future? I think the feature is useful even without feedback. And as Mizuki already wrote, the equivalent feature of GitHub and Gitlab also don't show feedback when a link is copied.
--------------------------------------------------------------------------------
Go MAEDA wrote:
> But how about committing the current patch first, and improving the feedback display in the future? I think the feature is useful even without feedback. And as Mizuki already wrote, the equivalent feature of GitHub and Gitlab also don't show feedback when a link is copied.

Of course! It's a useful feature and we should add it to Redmine. A bit of feedback would just be the icing on the cake. :-)

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
I changed the icon to an image created by combining a copy icon and a link icon.

!screenshot-2021-03-16-11.20.14.png!
--------------------------------------------------------------------------------
Bernhard Rohloff wrote:
> Go MAEDA wrote:
> > But how about committing the current patch first, and improving the feedback display in the future? I think the feature is useful even without feedback. And as Mizuki already wrote, the equivalent feature of GitHub and Gitlab also don't show feedback when a link is copied.
>
> Of course! It's a useful feature and we should add it to Redmine. A bit of feedback would just be the icing on the cake. :-)

I've also written a patch that adds feedback, but I'm a little confused about how to implement it, so I'm thinking of making another issue after this feature is committed.
--------------------------------------------------------------------------------
Mizuki ISHIKAWA wrote:
> I changed the icon to an image created by combining a copy icon and a link icon.

I accidentally attached an old patch.

--------------------------------------------------------------------------------
Setting the target version to 4.2.0.
--------------------------------------------------------------------------------
@test/system/copy_to_clipboard_test.rb@ fails on macOS. Could you fix the test to support macOS? Probably source:trunk/test/system/keyboard_shortcuts_test.rb@20729#L125 is helpful.

<pre>
[Screenshot]: tmp/screenshots/failures_test_copy_issue_url_to_clipboard.png
F

Failure:
CopyToClipboardSystemTest#test_copy_issue_url_to_clipboard [test/system/copy_to_clipboard_test.rb:40]:
Expected false to be truthy.

bin/rails test test/system/copy_to_clipboard_test.rb:29

[Screenshot]: tmp/screenshots/failures_test_copy_issue_journal_url_to_clipboard.png
F

Failure:
CopyToClipboardSystemTest#test_copy_issue_journal_url_to_clipboard [test/system/copy_to_clipboard_test.rb:54]:
Expected false to be truthy.

bin/rails test test/system/copy_to_clipboard_test.rb:43
</pre>
--------------------------------------------------------------------------------
Go MAEDA wrote:
> @test/system/copy_to_clipboard_test.rb@ fails on macOS. Could you fix the test to support macOS? Probably source:trunk/test/system/keyboard_shortcuts_test.rb@20729#L125 is helpful.
>
> [...]

I think it will be fixed by applying this patch in addition to the 0001 patch.
--------------------------------------------------------------------------------
Committed the patch. Thank you for your contribution.

Users can copy a URL of an issue or an issue journal by clicking "Copy link" in the action menu.
--------------------------------------------------------------------------------

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------


related_issues

relates,Closed,34714,Move delete button for issues and journals to the dropdown menu
relates,Closed,34932,"Copy link" feature for issues list

Admin Redmine さんが4年以上前に更新

  • カテゴリUI_10 にセット
  • 対象バージョン4.2.0_152 にセット

他の形式にエクスポート: Atom PDF

いいね!0
いいね!0