Project

General

Profile

Actions
Copy link

User story #4208

closed

Make visible if a Directive is not in use by any Rule

Added by Dennis Cabooter almost 11 years ago. Updated almost 11 years ago.

Status:
Released
Priority:
3
Assignee:
-
Category:
Web - Config management
Target version:
2.9.0~rc1
Pull Request:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Make visible if a directive is not in use by a rule. If I delete a rule but I forget to delete the directive used by that rule, the directive will be there forever.

Proposal: mark a directive red if it's not in use by any rule.

Files

Download all files
Rudder - Directives Management - 2013-12-11_11.51.25.png (19.2 KB) Rudder - Directives Management - 2013-12-11_11.51.25.png François ARMAND, 2013-12-11 11:54
Rudder - Directives Management - 2013-12-11_12.06.29.png (18.4 KB) Rudder - Directives Management - 2013-12-11_12.06.29.png François ARMAND, 2013-12-11 12:10
Rudder - Directives Management - 2013-12-11_17.20.58.png (11.1 KB) Rudder - Directives Management - 2013-12-11_17.20.58.png François ARMAND, 2013-12-11 17:26

Subtasks 1 (0 open — 1 closed)

User story #4219: Add the logic to change directive without rule display ReleasedNicolas CHARLES2013-12-06Actions
Actions
Copy link
 #1

Updated by François ARMAND almost 11 years ago

  • Assignee set to Jonathan CLARKE

That's an interesting idea, and does not seems very hard to do.

Jon, what do you thing about it? It could nicelly complete the "directive <-> rule" link enhancement going on with 2.9 ?

Actions
Copy link
 #2

Updated by François ARMAND almost 11 years ago

For now, the idea is to make the directive looks like disabled directive, and have in the tooltop the reason why it's greased.

Actions
Copy link
 #3

Updated by François ARMAND almost 11 years ago

Updated look:

- disable directive are greyed
- not used one are in italic.

See !Rudder - Directives Management - 2013-12-11_11.51.25.png!

Actions
Copy link
 #4

Updated by François ARMAND almost 11 years ago

Well, my personnal feeling is that italic and normal don't differentiate much.
Here is a version with "bold" in place of "normal" weight for assigned and enabled directive.

Actions
Copy link
 #5

Updated by Dennis Cabooter almost 11 years ago

I agree that italic and normal look too much the same. Another idea would be to use <del>strikethrough</del> for unused directives.

Actions
Copy link
 #6

Updated by François ARMAND almost 11 years ago

Well, even with bold, the diff is not really clear. Let's test with an "warning" icon.

Actions
Copy link
 #7

Updated by Jonathan CLARKE almost 11 years ago

  • Status changed from New to Discussion
  • Assignee changed from Jonathan CLARKE to Dennis Cabooter

How about adding a little yellow warning triangle to Directives not used by any Rules? Something like http://www.clker.com/cliparts/1/5/5/7/13065475412041287699yellow-warning-sign.jpg.

Actions
Copy link
 #8

Updated by François ARMAND almost 11 years ago

  • File Rudder - Directives Management - 2013-12-11_17.20.58.png added

And a new version with "warning" logo.

That seems a little scary in my demo, because almost no directive are used, but it could be nice in a real Rudder installation.

Actions
Copy link
 #9

Updated by François ARMAND almost 11 years ago

  • File deleted (Rudder - Directives Management - 2013-12-11_17.20.58.png)
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ almost 11 years ago

  • Subject changed from Make visible if a directive is not in use by a rule to Make visible if a Directive is not in use by any Rule
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ almost 11 years ago

  • Target version set to 2.9.0~rc1
Actions
Copy link
 #13

Updated by Vincent MEMBRÉ almost 11 years ago

  • Status changed from Discussion to 10
Actions
Copy link
 #14

Updated by Vincent MEMBRÉ almost 11 years ago

  • Category set to Web - Config management
Actions
Copy link
 #15

Updated by François ARMAND almost 11 years ago

  • Status changed from 10 to Pending release
  • Assignee deleted (Dennis Cabooter)
Actions
Copy link
 #16

Updated by Vincent MEMBRÉ almost 11 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.9.0~rc1, which was released on Friday 13/12/2013.
Check out:

Actions
Copy link

Also available in: Atom PDF