Patch #28602

[Gantt] Move edit and delete buttons for queries to the buttons section

Added by Bernhard Rohloff about 1 month ago. Updated 14 days ago.

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

0%

Category:Gantt
Target version:4.1.0

Description

Except in the gantts view, in all query based views the edit and delete buttons are placed inside the buttons section (apply/clear/save) under the filters and options.
I think the gantts view should be aligned to the common UI pattern, as well.

In the submitted patch the yet empty contextual area is left inside the view as it breaks the tests and I don't know if it should be left to make further modifications easier.

Current view

Proposed view

gantt_before_patch.png (7.1 KB) Bernhard Rohloff, 2018-04-22 08:46

gantt_after_patch.png (8.99 KB) Bernhard Rohloff, 2018-04-22 08:46

0001-align-query-buttons-with-other-query-based-views.patch Magnifier (1.35 KB) Bernhard Rohloff, 2018-04-22 08:59


Related issues

Related to Redmine - Patch #26125: Unify form#query_form on calendar/gantt views with issues... Closed

History

#1 Updated by Marius BALTEANU about 1 month ago

  • Related to Patch #26125: Unify form#query_form on calendar/gantt views with issues/spent time views added

#2 Updated by Marius BALTEANU about 1 month ago

It'll be nice to deliver this change along with #26125.

#3 Updated by Go MAEDA 28 days ago

  • Target version set to Candidate for next major release

+1

The buttons on issues list page were moved in r15344. To make the UI consistent, I think also the buttons on gantt page should be moved to below the filters.

#4 Updated by Marius BALTEANU 24 days ago

Bernhard Rohloff wrote:

In the submitted patch the yet empty contextual area is left inside the view as it breaks the tests and I don't know if it should be left to make further modifications easier.

I'm in favour of removing the empty contextual element and update the tests accordingly.

#5 Updated by Go MAEDA 24 days ago

Marius BALTEANU wrote:

Bernhard Rohloff wrote:

In the submitted patch the yet empty contextual area is left inside the view as it breaks the tests and I don't know if it should be left to make further modifications easier.

I'm in favour of removing the empty contextual element and update the tests accordingly.

I am concerned that removing the div.contextual may affect some plugins which insert custom tags into the area.

#6 Updated by Go MAEDA 14 days ago

  • Target version changed from Candidate for next major release to 4.1.0

Setting target version to 4.1.0.

Also available in: Atom PDF