Feature #34714

Move delete button for issues and journals to the dropdown menu

Added by Mizuki ISHIKAWA 9 months ago. Updated 7 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Go MAEDA% Done:

0%

Category:UI
Target version:4.2.0
Resolution:Fixed

Description

I would like to include infrequently used link such as Delete and Copy in Actions Dropdown.
This is a patch that is based on the current features of the wiki.

The reason for making this issue:
  • There are many icons on the view, so I would like to make it a little simpler.
  • I am trying to increase the number of "copy links" links in #34703. Then there will be more icons than now.
  • Journal action links only show icons, but there are some features that are difficult to understand with just the icons.

Concern:
The same icon is used for the context menu of related issues on the page, and it may be misunderstood that the link added this time also opens the context menu.

screenshot_2021-02-12_15.41.37.png (88.8 KB) Mizuki ISHIKAWA, 2021-02-12 07:52

screenshot2.png (93.8 KB) Mizuki ISHIKAWA, 2021-02-12 07:54

screenshot1.png (126 KB) Mizuki ISHIKAWA, 2021-02-12 07:54

34714.patch Magnifier (6.25 KB) Mizuki ISHIKAWA, 2021-02-12 07:54

34714-v2.patch Magnifier (5.98 KB) Mizuki ISHIKAWA, 2021-02-22 01:48

screenshot_2021-02-22_9.49.20.png (14.1 KB) Mizuki ISHIKAWA, 2021-02-22 01:51

34714-v3.patch Magnifier (5.61 KB) Mizuki ISHIKAWA, 2021-03-03 01:47

screenshot_2021-03-03_9.54.32.png (11 KB) Mizuki ISHIKAWA, 2021-03-03 01:54

34714-move-copy.patch Magnifier (2.51 KB) Mizuki ISHIKAWA, 2021-03-22 09:22

screenshot_2021-03-22_17.02.20.png (21.3 KB) Mizuki ISHIKAWA, 2021-03-22 09:23

34714-after.png (23.1 KB) Go MAEDA, 2021-03-25 05:25


Related issues

Related to Redmine - Feature #34703: "Copy link" feature for issue and issue journal Closed

Associated revisions

Revision 20765
Added by Go MAEDA 8 months ago

Move some action links for issues and journals to the dropdown menu (#34714).

Patch by Mizuki ISHIKAWA.

Revision 20805
Added by Go MAEDA 7 months ago

Fix a failed system test due to r20765 (#34714).

Patch by Mizuki ISHIKAWA.

Revision 20842
Added by Go MAEDA 7 months ago

Restore Copy button to its original location (#34714).

Patch by Mizuki ISHIKAWA.

History

#1 Updated by Mizuki ISHIKAWA 9 months ago

I have attached a patch.

The following image is the issues / show page after applying the patch.

#2 Updated by Go MAEDA 8 months ago

I think it is better to put "Edit" in the menu instead of "Quote"

  • In my opinion, "Quote" is a more frequently used feature than "Edit"
  • "Edit" is displayed only for users who have appropriate permission. If the current user does not have permission, no button is displayed there

#3 Updated by Mizuki ISHIKAWA 8 months ago

Go MAEDA wrote:

I think it is better to put "Edit" in the menu instead of "Quote"

  • In my opinion, "Quote" is a more frequently used feature than "Edit"
  • "Edit" is displayed only for users who have appropriate permission. If the current user does not have permission, no button is displayed there

Thank you for your feedback.
I agree with your opinion and modified the patch as shown in the image below.

#4 Updated by Marius BALTEANU 8 months ago

I use quite a lot the "Edit" link, mostly to edit my own comments (eg: fix typos). I think we should hide under the dropdown only the "Delete" link.

#5 Updated by Mizuki ISHIKAWA 8 months ago

It seems that both "Quote" and "Edit" are used frequently, so I made a patch that puts only Delete in the dropdown.
It may seem that you don't need to change to the dropdown as there are only "Delete" dropdown items, but once this patch is committed, you want to put the "CopyLink" under development in #34703 into the dropdown. think.

#7 Updated by Marius BALTEANU 8 months ago

  • Related to Feature #34703: "Copy link" feature for issue and issue journal added

#8 Updated by Marius BALTEANU 8 months ago

  • Target version set to 4.2.0

The patch looks good to me with one exception, there are 3 Rubocop offences that should be fixed.

I'm assigning this to 4.2.0, it's an useful change.

#9 Updated by Go MAEDA 8 months ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch after fixing RuboCop offenses. Thank you.

#10 Updated by Mizuki ISHIKAWA 8 months ago

Thank you for your code review and commit :)

#11 Updated by Marius BALTEANU 7 months ago

  • Status changed from Closed to Reopened

A system test fails:


IssuesSystemTest#test_removing_issue_shows_confirm_dialog:
Capybara::ExpectationNotMet: expected to find visible css "#content a.icon-del" at least 1 time but there were no matches. Also found "", "", which matched the selector but not all filters. 
    test/system/issues_test.rb:241:in `block (2 levels) in <class:IssuesSystemTest>'
    test/system/issues_test.rb:240:in `block in <class:IssuesSystemTest>'
bin/rails test test/system/issues_test.rb:237
.[Screenshot]: tmp/screenshots/failures_test_bulk_edit.png

#12 Updated by Mizuki ISHIKAWA 7 months ago

Marius BALTEANU wrote:

A system test fails:

I missed the result of the system test. Thank you for reporting.
The test will work by adding the following line.

diff --git a/test/system/issues_test.rb b/test/system/issues_test.rb
index f1f355d0d..0e37ae77d 100644
--- a/test/system/issues_test.rb
+++ b/test/system/issues_test.rb
@@ -238,6 +238,7 @@ class IssuesSystemTest < ApplicationSystemTestCase
     log_user('jsmith', 'jsmith')
     visit '/issues/1'
     page.accept_confirm /Are you sure/ do
+      first('#content span.icon-actions').click
       first('#content a.icon-del').click
     end
   end

#13 Updated by Go MAEDA 7 months ago

  • Status changed from Reopened to Closed

Mizuki ISHIKAWA wrote:

Marius BALTEANU wrote:

A system test fails:

I missed the result of the system test. Thank you for reporting.
The test will work by adding the following line.

[...]

Committed the fix in r20805.

#14 Updated by Mizuki ISHIKAWA 7 months ago

This is a feature that has already been committed, but I thought it would be better if the issue copy link wasn't in the dropdown.
If you move the copy link out of the dropdown, the content of the dropdown will match between the journal and the issue for a unified look.

#15 Updated by Robert Korulczyk 7 months ago

If you move the copy link out of the dropdown, the content of the dropdown will match between the journal and the issue for a unified look.

I don't think that it is valid reason to move this action outside of the dropdown. Dropdown should contain less frequently used items, it should be the last thing that you want to keep unified across different views.

#16 Updated by Go MAEDA 7 months ago

Mizuki ISHIKAWA wrote:

This is a feature that has already been committed, but I thought it would be better if the issue copy link wasn't in the dropdown.
If you move the copy link out of the dropdown, the content of the dropdown will match between the journal and the issue for a unified look.

+1

After applying r20765 to my company's production server, team members started to complain that accessing the "Copy" button requires one more click is very inconvenient because they use the feature frequently.

#17 Updated by Go MAEDA 7 months ago

  • File 34714-after.png added
  • Subject changed from Move some action links for issues and journals to the dropdown menu to Move delete button for issues and journals to the dropdown menu
  • Status changed from Reopened to Closed

Committed the fix #34714#note-14 in r20842.

Before (pre r20765):

[Edit] [Log time] [Watch] [Copy] [Delete] 

After (r20805):

[Edit] [Log time] [Watch] [Copy] […]
                                 [Copy link]
                                 [Delete]

Also available in: Atom PDF