Patch #25853

Move left bottom links from project settings above

Added by Marius BALTEANU 3 months ago. Updated about 1 month ago.

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

0%

Category:UI
Target version:Candidate for next minor release

Description

This patch is related to #24720 where all "New item" links were been moved above the list.

There are three more links left under the list and I think it's better to have them all above the list:
1. "Close completed versions" from Versions tab
2. "Delete" from Wiki tab
3. "Reset" from Activities tab

I've attached 2 patches:

move_close_completed_versions_above.patch

  • adds the "icon" and "icon-lock" classes to the link (the same icon is used for button "Close project")
  • adds a class "left" for existing contextual class to let the icons aligned to left (according to the #24720)

move_delete_and_reset_above_the_list.patch

  • moves the "Delete" link from wiki tab on the same line with the wiki input field. I find it more natural to be there.
  • moves the "Reset" link from activities above the list.

I made this patch after I find out that our users didn't know about the button "Close completed version" which is very well hidden when the projects have many versions.

move_delete_and_reset_above_the_list.patch Magnifier (1.94 KB) Marius BALTEANU, 2017-05-15 21:33

move_close_completed_versions_above.patch Magnifier (2.12 KB) Marius BALTEANU, 2017-05-15 21:33

Screen Shot 2017-05-15 at 22.54.40.png (68.2 KB) Marius BALTEANU, 2017-05-15 21:56

Screen Shot 2017-05-15 at 22.54.48.png (76.3 KB) Marius BALTEANU, 2017-05-15 21:56

Screen Shot 2017-05-15 at 22.54.28.png (175 KB) Marius BALTEANU, 2017-05-15 21:56

manage_versions.png (61.1 KB) Marius BALTEANU, 2017-06-27 00:52

move_left_bottom_links_from_project_settings_above.patch Magnifier (3.29 KB) Marius BALTEANU, 2017-06-27 00:58

History

#1 Updated by Marius BALTEANU 3 months ago

I've attached 3 screenshots with the implementation.

#2 Updated by Go MAEDA 3 months ago

  • Target version set to 3.4.0

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

#3 Updated by Mischa The Evil 3 months ago

I've had a look at the screenshots and patches. My two cents on this proposal:
  • I find it more natural to have the 'Close completed versions' link rendered on the top-right (as it is a link that functions on all versions)
  • I also find it more natural to have the wiki 'Delete' link rendered aligned to the top-right, above the div.box.tabular (this in line with above; links that work on all the tab's items all on the top-right instead of on the bottom-right)
This is however:
  • more difficult to implement
  • very much a matter of taste, I think (making it a difficult thing to suit everyone's mileage)

#4 Updated by Marius BALTEANU 2 months ago

Thanks Mischa for reviewing this patch. If was only after me, I would move all the links to the top right for consistency with other screens. (even the New Item links). Getting back to your feedback, I'll be happy with any implementation that moves the links from the bottom to top.

#5 Updated by Mischa The Evil 2 months ago

Marius BALTEANU wrote:

If was only after me, I would move all the links to the top right for consistency with other screens. (even the New Item links).

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.

Marius BALTEANU wrote:

[...] I'll be happy with any implementation that moves the links from the bottom to top.

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.

#6 Updated by Jean-Philippe Lang 2 months ago

  • Target version changed from 3.4.0 to Candidate for next minor release

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

#7 Updated by Marius BALTEANU about 1 month ago

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.

Also available in: Atom PDF