Make tree arrow clickable in admin UI to show/hide details [#1327614]

Comment	File	Size	Author
#7	metatag-n1327614-7.patch	1.26 KB	damienmckenna
#1	tree_arrow_clickable_1327614_1.patch	1.31 KB	andremolnar

Comment #1

andremolnar commented 31 October 2011 at 19:59

Status	File	Size
new	tree_arrow_clickable_1327614_1.patch	1.31 KB

Attached patch for that.

Log in or register to post comments

Comment #2

marcoka commented 1 June 2012 at 20:21

Status:

Active

» Needs review

i am not sure of this. this does not fit with other modules handling (backend consistentcy)

Log in or register to post comments

Comment #3

andremolnar commented 5 June 2012 at 01:25

This is going a while back now, but if I recall correctly the motivation was the UX expectation that the arrows be clickable.

This patch provides an overall better user experience with little to no drawbacks (that I can think of). If there are other modules that do not follow this pattern today, perhaps they will follow this lead.

Log in or register to post comments

Comment #4

marcoka commented 5 June 2012 at 07:42

arrows are great, agree. but dint you mention that the whole div area is clickable too?

Log in or register to post comments

Comment #5

andremolnar commented 8 June 2012 at 07:50

Yes, its all clickable. The arrows are implemented as background images on the div - so its the div that must be clickable (or else re-write the entire display and add a ton of tag soup).
Have you tried it out to see it in action? It works quite nicely if I do say so myself :)

Log in or register to post comments

Comment #6

marcoka commented 8 June 2012 at 08:52

ok, i know what you mean. i think in the end it depends on the maintainers. i am so familiar with ui stuff that i would not have problems with the current or your suggestion.