Patch #24776

UI inconsistencies on /enumerations/index view

Added by Mischa The Evil 6 months ago. Updated 5 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:UI
Target version:3.4.0

Description

Coming out of and a follow-up to #24720. I spotted three things that could benefit improvement:
  • re-position the new enumeration links to above the item tables (in line with #24720 changes)
  • add 'icon' and 'icon-add' classes to the new enumeration links (like most, if not all, new ... links in both the front- and the back-end) to make them stand-out more
  • handle cases of none existing enumerations using the regular no-data message blocks

I attach three patches implementing these changes. This patch serial, against current source:/trunk@16156, is produced using git format-patch which makes the individual patches apply-able using "patch -p1 < 0001-...".

before.png (19.2 KB) Mischa The Evil, 2017-01-07 18:57

after.png (27.4 KB) Mischa The Evil, 2017-01-07 18:58

before-add-margin.png (20 KB) Go MAEDA, 2017-01-09 13:12

after-add-margin.png (21.2 KB) Go MAEDA, 2017-01-09 13:12

rare-case-indifferent-distance.png (33.7 KB) Mischa The Evil, 2017-01-10 06:56

extreme-case.png (12.9 KB) Mischa The Evil, 2017-01-10 06:56

0001-Move-New-enumeration-links-to-above-the-item-tables.patch Magnifier (1.01 KB) Mischa The Evil, 2017-01-10 07:30

0002-Add-icon-and-icon-add-classes-to-new-enum-links.patch Magnifier (893 Bytes) Mischa The Evil, 2017-01-10 07:30

0003-Handle-the-case-of-none-existing-enumerations-using-.patch Magnifier (699 Bytes) Mischa The Evil, 2017-01-10 07:30

0004-Add-margin-between-tables-and-the-following-headings.patch Magnifier (1.04 KB) Mischa The Evil, 2017-01-10 07:30


Related issues

Related to Redmine - Feature #24720: Move all 'new item' links in project settings to above th... Closed

Associated revisions

Revision 16186
Added by Jean-Philippe Lang 5 months ago

Move 'New enumeration' links to above the item tables (#24776).

Patch by Mischa The Evil.

Revision 16187
Added by Jean-Philippe Lang 5 months ago

Add 'icon' and 'icon-add' classes to new enum links (#24776).

Patch by Mischa The Evil.

Revision 16188
Added by Jean-Philippe Lang 5 months ago

Handle the case of none existing enumerations using no-data blocks (#24776).

Patch by Mischa The Evil.

Revision 16189
Added by Jean-Philippe Lang 5 months ago

Add margin between tables and the following headings (#24776).

Patch by Mischa The Evil.

History

#1 Updated by Mischa The Evil 6 months ago

Some comparative screenshots.

Before:

After:

#2 Updated by Mischa The Evil 5 months ago

  • Related to Feature #24720: Move all 'new item' links in project settings to above the item tables added

#3 Updated by Go MAEDA 5 months ago

Nice improvement.

But I think it would be better to set more margin between the table and the next heading. Please see the following screenshots.

current patch:

set more margin:

#4 Updated by Mischa The Evil 5 months ago

Go MAEDA wrote:

But I think it would be better to set more margin between the table and the next heading. Please see the following screenshots.

I totally agree. Thanks for reviewing and your feedback. I'll leave an updated, rebased patch set — against source:/trunk@16171 — adding a fourth patch containing the CSS rule you've proposed. The first three patches didn't change by content, only its metadata.

Regarding the CSS rule: while it fixes practical all cases1, it does leave a difference between the space between the tables and the following headers ánd the space between the paragraphs and the following headers. See the following two screenshots:

  • Remaining difference made visual:
  • Extreme case without any enumerations at all:

This cannot be fixed easily (and in a clean way) due to the current structure of the page. Though, given the rarity of the case, I think I can live with this minor drawback/trade-off. But, YMMV...

1 I've almost never seen Redmine instances in production without enumerations, only just installed ones where no default data was loaded yet.

#5 Updated by Mischa The Evil 5 months ago

  • File deleted (0001-Move-New-enumeration-links-to-above-the-item-tables.patch)

#6 Updated by Mischa The Evil 5 months ago

  • File deleted (0002-Add-icon-and-icon-add-classes-to-new-enum-links.patch)

#7 Updated by Mischa The Evil 5 months ago

  • File deleted (0003-Handle-the-case-of-none-existing-enumerations-using-.patch)

#8 Updated by Go MAEDA 5 months ago

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

Mischa The Evil wrote:

Regarding the CSS rule: while it fixes practical all cases1, it does leave a difference between the space between the tables and the following headers ánd the space between the paragraphs and the following headers. See the following two screenshots:

Looks good to me. Thanks!
I think this issue is now ready to be merged. Setting target version to 3.4.0.

1 I've almost never seen Redmine instances in production without enumerations, only just installed ones where no default data was loaded yet.

Absolutely agree.

#9 Updated by Jean-Philippe Lang 5 months ago

  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Patch committed, thanks Mischa.

Also available in: Atom PDF