Redmine

I've attached 3 screenshots with the implementation.

The patch improve consistency of UI. Setting target version to 3.4.0.

Please reassign when a solution that fits everyone's need is implemented.

Mischa The Evil wrote:

On #24720#note-2, I wrote the following about that:

[...] I've also explored the (re-)use of the contextual div class for this purpose (as such tightening the consistency of new links throughout the app in both the front- ánd back-end) but found that it led to a, for me unacceptable, increase of traveled mouse distance.

I read you reasons and I'm agree with you, but from my point of view is more important to have the links/buttons in the same place in all screens. In other words, I prefer the consistency instead of saving some mouse distance for some actions that are not made very often. But as you said, it is very much a matter of taste.

I understand. Can you look into incorporating my two notes into your proposed patch? With those covered, I'd say this is good enough so that this could be added still to 3.4.0 along with #24720 and #24776.

I've uploaded a new patch that incorporates your two notes.

Some technical notes:
- I tried to use the existing contextual class without adding new css class
- I moved the if conditions outside in order to not render empty contextual blocks.

As a overall conclusion, it is better than the actual implementation (with the bottom links), but I'm still not very happy with the new results (quite awkward to have a link top-left and one in top-right in the versions tab).

Initial patches can be removed.

Marius updated the patch. I am setting target version again.

Marius BALTEANU wrote:

It is a small change, I would suggest to include it in 4.0.0 to get rid of the buttons under the tables/lists.

Indeed.

But the attached patch cannot be applied cleanly against the current trunk. Could you update the patch? I will commit the new patch when it is ready.

Here it is.

Committed. Thank you.