Patch #28861

Add a date item to the context menu

Added by Mizuki ISHIKAWA 3 months ago. Updated 23 days ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:UI
Target version:Candidate for next major release

Description

The context menu is very convenient because you can change the information without making a screen transition.
I would like to change the date information by context menu.

By applying this patch you can open a modal to change the date.

screenshot2.png (156 KB) Mizuki ISHIKAWA, 2018-05-23 10:22

screenshot1.png (116 KB) Mizuki ISHIKAWA, 2018-05-23 10:22

Add-dates-on-context-menu.patch Magnifier (4.63 KB) Mizuki ISHIKAWA, 2018-05-23 10:22

datepicker_firefox.png (4.74 KB) Bernhard Rohloff, 2018-06-23 07:41

bulk_edit_selected_issues.png (124 KB) Mizuki ISHIKAWA, 2018-06-25 01:44

Add-dates-on-context-menu_v2.patch Magnifier (4.73 KB) Mizuki ISHIKAWA, 2018-07-30 06:56

History

#1 Updated by Yuuki NARA 3 months ago

+1

I am looking forward to releasing this function.

#2 Updated by Akipii Oga 3 months ago

+1

#3 Updated by Go MAEDA 2 months ago

  • Target version set to Candidate for next major release

#4 Updated by Marius BALTEANU 2 months ago

Interesting feature.

I'm writing down some thoughts that I've regarding this feature:
1. The "Date" label is not clear enough from my point of view. Maybe "Start/End dates" or just "Dates"?
2. I find it useful to support also the custom fields of type "Date".
3. Editing two attributes with one action is not usual for the Redmine contextual menu. I'm wondering if is not better to add the dates attributes as individual items in the contextual menu (Start date, End date, Custom Date etc) and in the submenu to add some dates shortcuts like "Tomorrow", "Next Week", "Next Month" and the date picker for selecting the date.

#5 Updated by Go MAEDA about 1 month ago

Marius BALTEANU wrote:

3. Editing two attributes with one action is not usual for the Redmine contextual menu. I'm wondering if is not better to add the dates attributes as individual items in the contextual menu (Start date, End date, Custom Date etc) and in the submenu to add some dates shortcuts like "Tomorrow", "Next Week", "Next Month" and the date picker for selecting the date.

Maybe project managers think start date and due date of an issue at the same time when they make a schedule, so I prefer having these two fields on the same dialog.

#6 Updated by Bernhard Rohloff about 1 month ago

Go MAEDA wrote:

Maybe project managers think start date and due date of an issue at the same time when they make a schedule, so I prefer having these two fields on the same dialog.

I agree with Go MAEDA on this point. I think it would be tedious to go through the context menu twice to set the start and due date separately. I also think the intention of the menu item "Date" would be more clear if we would rename it into something like "Schedule".

Marius BALTEANU wrote:

... to add some dates shortcuts like "Tomorrow", "Next Week", "Next Month" and the date picker for selecting the date.

This could also be very handy but I would integrate them in the modal view, too. In my experience project managers are quite volatile in making their decisions (especially at scheduling) so having all choices presented in a single view can be advantageous for this kind of users.

Referring to the proposed modal view, I find the checkboxes to reset the dates quite odd. As this functionality is also present in the browser based datepickers I would recommend to remove them from the view.

#7 Updated by Mizuki ISHIKAWA about 1 month ago

Thank you for much feedback.

Bernhard Rohloff wrote:

Referring to the proposed modal view, I find the checkboxes to reset the dates quite odd. As this functionality is also present in the browser based datepickers I would recommend to remove them from the view.

Emptying date_field and checking the clear checkbox work differently.
It is the same UI as existing bulk edit selected issues screen (views/issues/bulk_edit.html.erb)

  • If date_field is empty, it does not change the existing value of that field.
  • If the clear check box is checked, overwrite the value of that field with an empty string.

Many people think that the clear checkbox is odd just like you, but I have no other good idea.

#8 Updated by Mizuki ISHIKAWA 23 days ago

Thank you for much feedback.

I attached a patch to change label from "Date" to "Start date / Due date".
I set label to "Start date / Due date" because you do not have to increase the items in the en.yml file.

Also available in: Atom PDF