Bug Tracker

#2360 closed bug (fixed)

Opened February 18, 2008 03:48PM UTC

Closed March 20, 2008 12:39PM UTC

Last modified March 15, 2012 04:26PM UTC

[treeview] Branches with checkboxes

Reported by: morrijr Owned by: joern
Priority: major Milestone:
Component: plugin Version:
Keywords: treeview Cc:
Blocked by: Blocking:
Description

Hi Jörn

The problem I've got is that I need to be able to have a checkbox on each

branch/node, but clicking the checkbox of a branch in addition to toggling

the checkbox expands/collapses the branch.

Can you help? I can't see what's causing this. This is a regression from 1.3.

Thanks,

J.

Attachments (1)
  • experimental.zip (44.5 KB) - added by morrijr February 18, 2008 03:49PM UTC.

    Test case

Change History (3)

Changed February 19, 2008 02:51PM UTC by morrijr comment:1

The change that is causing this is in the applyClasses function.

From (1.3)

this.filter(":has(>ul):not(:has(>a))").find(">span").click(function(event) {
    if ( this == event.target ) {
        toggler.apply($(this).next());
    }
}).add( $("a", this) ).hoverClass();

to (1.4)

this.filter(":has(>ul):not(:has(>a))").find(">span").click(function(event) {
    toggler.apply($(this).next());
}).add( $("a", this) ).hoverClass();

Was there a reason this if was removed?

Changed February 19, 2008 08:30PM UTC by joern comment:2

keywords: treeview branch checkbox → treeview
milestone: 1.2.4
need: Review → Commit
owner: → joern
summary: [TreeView] Branches with checkboxes → [treeview] Branches with checkboxes
version: 1.2.3

Note: Add comment why the check is necessary.

Changed March 20, 2008 12:39PM UTC by joern comment:3

resolution: → fixed
status: new → closed

Fixed in [5100].