Possibility to override function used to determine current node #28

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@bguerin

bguerin commented May 30, 2012

No description provided.

@jzaefferer

This comment has been minimized.

Show comment Hide comment
@jzaefferer

jzaefferer May 31, 2012

Owner

I don't really work on this plugin, and don't want to merge this as-is. Adding more options is rarely a good idea. Could you explain your usecase, that is, what does your custom option look like in your app? Maybe there's a simple way to make the option-less approach work in both cases.

Owner

jzaefferer commented May 31, 2012

I don't really work on this plugin, and don't want to merge this as-is. Adding more options is rarely a good idea. Could you explain your usecase, that is, what does your custom option look like in your app? Maybe there's a simple way to make the option-less approach work in both cases.

@bguerin

This comment has been minimized.

Show comment Hide comment
@bguerin

bguerin May 31, 2012

I have a tree on the left of my page, and courbes with fields (dates and period)

When I click a link on the tree, I get a default courbe, with fields pre filled (last hour, and 10 seconds as period)
But I can change the dates and/or the period saying to show the last month with 1 hour as period

So my url contains parameters for the tree but also for my dates and period fields
The dates and period parameters are specific for each node of the tree, they do not match from one node to another.

So I do not want the tree to check for these parameters to determine the current node

bguerin commented May 31, 2012

I have a tree on the left of my page, and courbes with fields (dates and period)

When I click a link on the tree, I get a default courbe, with fields pre filled (last hour, and 10 seconds as period)
But I can change the dates and/or the period saying to show the last month with 1 hour as period

So my url contains parameters for the tree but also for my dates and period fields
The dates and period parameters are specific for each node of the tree, they do not match from one node to another.

So I do not want the tree to check for these parameters to determine the current node

@jzaefferer

This comment has been minimized.

Show comment Hide comment
@jzaefferer

jzaefferer May 20, 2013

Owner

This plugin isn't maintained anymore.

Owner

jzaefferer commented May 20, 2013

This plugin isn't maintained anymore.

@jzaefferer jzaefferer closed this May 20, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment