Ticket #3613 (closed feature: wontfix)

Opened 5 years ago

Last modified 19 months ago

Accordion: cookie persistence

Reported by: joern.zaefferer Owned by:
Priority: major Milestone: 1.6
Component: ui.accordion Version: 1.6rc2
Keywords: Cc:
Blocking: Blocked by:

Description (last modified by joern.zaefferer) (diff)

Here is an example of adding external cookie persistence to an accordion:

var accordion = $("#accordion");
var index = $.cookie("accordion");
var active;
if (index !== null) {
	active = accordion.find("h3:eq(" + index + ")");
} else {
	active = 0
}
accordion.accordion({
	header: "h3",
	event: "click hoverintent",
	active: active,
	change: function(event, ui) {
		var index = $(this).find("h3").index ( ui.newHeader[0] );
		$.cookie("accordion", index, {
			path: "/"
		});
	},
	autoHeight: false
});

This should be integrated into the plugin, with the option to enable it (requires cookie plugin) and the option to customize the cookie settings, eg. for a different path.

Until the change-option is added, that callback must be used like this (currently untested):

{{{{ $("...").accordion(...).bind("accordionchange", function(event, ui) {

var index = $(this).find("h3").index ( ui.newHeader[0] ); $.cookie("accordion", index, {

path: "/"

});

}); }}}

Change History

comment:1 Changed 5 years ago by joern.zaefferer

  • Milestone changed from TBD to 1.next

comment:2 Changed 5 years ago by joern.zaefferer

  • Description modified (diff)

comment:3 Changed 4 years ago by scott.gonzalez

  • Type changed from bug to feature
  • Summary changed from accordion: cookie persistence to Accordion: cookie persistence

comment:4 Changed 4 years ago by scott.gonzalez

  • Priority changed from minor to major

comment:5 Changed 4 years ago by scott.gonzalez

  • Status changed from new to closed
  • Resolution set to wontfix

This shouldn't be built into the core of the accordion.

comment:6 Changed 19 months ago by scott.gonzalez

  • Milestone changed from 1.next to 1.6
Note: See TracTickets for help on using tickets.