Project

General

Profile

Actions
Copy link

Bug #2647

closed

cursor in directives selection in rule creation should not be the text selector.

Added by Arthur ANGLADE over 12 years ago. Updated over 12 years ago.

Status:
Released
Priority:
N/A
Assignee:
Jean VILVER
Category:
Web - Config management
Target version:
2.4.0~beta3
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

It should be the "hand" one, and it should not be a "hand" when hovering the categories.

Actions
Copy link
 #1

Updated by Jonathan CLARKE over 12 years ago

  • Target version set to 2.4.0~beta3
Actions
Copy link
 #2

Updated by Jean VILVER over 12 years ago

  • Status changed from New to In progress
  • Assignee changed from Arthur ANGLADE to Jean VILVER
Actions
Copy link
 #3

Updated by Jean VILVER over 12 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100
Actions
Copy link
 #4

Updated by Jean VILVER over 12 years ago

  • Status changed from Pending technical review to In progress
Actions
Copy link
 #5

Updated by Jean VILVER over 12 years ago

In 5fc55531db2e3b0874872faca0426367fe753900, I changed the tree click/hover behavior:

- A hand appear when hovering a category

- When clicking on a category or a template it expands the category

No need to click on the small arrow in front of a category/template to expand it now.

Note : This commit only affect the tree in directive management

Actions
Copy link
 #6

Updated by Jonathan CLARKE over 12 years ago

Jean VILVER wrote:

In 5fc55531db2e3b0874872faca0426367fe753900, I changed the tree click/hover behavior:

- A hand appear when hovering a category

- When clicking on a category or a template it expands the category

No need to click on the small arrow in front of a category/template to expand it now.

Awwww, this sounds awesome!!! Thank you Jean!

Actions
Copy link
 #7

Updated by Jean VILVER over 12 years ago

Jonathan CLARKE wrote:

[...]
Awwww, this sounds awesome!!! Thank you Jean!

I know :p

Jean VILVER wrote:

[...]
Note : This commit only affect the tree in directive management.

682907b04a54b2a01532431a09bf14866d49dcb7 is affecting trees in Rule Management.

286577da7dd42314b0530545d9fccbf7e1fd1792 is affecting trees in Technique Management.

Actions
Copy link
 #8

Updated by Jean VILVER over 12 years ago

  • Status changed from In progress to Pending technical review
Actions
Copy link
 #9

Updated by François ARMAND over 12 years ago

  • Status changed from Pending technical review to Released

I'm not a big fan of all the repeated:

function(e) {
  this.toggle_node(e); 
  return true;
}

But I'm not sure that a call to a method would be shorter, so let it like that.

Actions
Copy link

Also available in: Atom PDF