Bug 8218 – factory test menu items being sorted in CustomizeHomeApplet incorrectly

Bug 8218 - factory test menu items being sorted in CustomizeHomeApplet incorrectly

Summary:

factory test menu items being sorted in CustomizeHomeApplet incorrectly

Status:	NEW

Product:	SB Controller
Classification:	Unclassified
Component:	UI
Version:	unspecified
Platform:	PC Other

Importance:	P2 normal (vote)
Target Milestone:	Future
Assigned To:	Ben Klaas

URL:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2008-05-22 20:53 UTC by Ben Klaas
Modified:	2008-05-23 15:18 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Ben Klaas 2008-05-22 20:53:33 UTC

factory test menu items aren't sorted by weight, so that might be it.
also need to look at whether the complex weight sort works for trees that deep.

Comment 1 Ben Klaas 2008-05-23 14:56:18 UTC

There are many items in Settings->Advanced with a weight of 100, including Factory Test, which has subitems. The new complexWeight sort algorithm doesn't discriminate on the weights of these items, therefore sorts the subitems of Factory test below the lowest 100 weight item, which is "Wireless Region".

The quick workaround for this would be to set the weight of Factory Test to 101, which would drop it down to the bottom of the menu next to Factory Reset.

But the bug still remains...really not sure how this can be solved.

Comment 2 Ben Klaas 2008-05-23 15:18:05 UTC

r2525 works around the immediate issue by weighting Factory Test so its lower than the other 100-weight items in Advanced Settings.

leaving open, targetted to future for fixing the underlying complexWeight sort issue as described in comment #1