Expansion Sample Maps -Sorted Naturally by default now?

[d.healey]

@danh I'm not seeing any commits specific to that function so this change must be a side-effect of some other change that was made. Try some different commits until you find the one where the change occurred.

[DanH]

@d-healey Alphabetical is great, as long as it stays that way for the future!!

[d.healey]

@danh You can sort the list yourself with the sort function to make sure it does ;)

[DanH]

@d-healey I did try that, and the list in the combo box went alphabetical, but it seemed it still loaded the sample maps as before. I couldn't figure it out!

[d.healey]

@danh Did you only sort the list as it was written to the combo box items list or did sort the array that contains the sample maps and the combo box items?

[DanH]

@d-healey I did this:

sampleMaps.sortNatural();

[d.healey]

@danh And that array is also used to populate the combo box?

[DanH]

@d-healey aye:

SAMPLEBOX.set("items", "");
SAMPLEBOX.set("items", sampleMaps.join("\n"));

[d.healey]

@danh You don't need to clear the combo box before populating any more, that bug was fixed :)

[ulrik]

@d-healey said in Expansion Sample Maps -Sorted Naturally by default now?:

You don't need to clear the combo box before populating any more, that bug was fixed

[DanH]

@d-healey good to know. So the code looks ok? Maybe I'll have another go - although maybe impossible to tell now

[d.healey]

@danh said in Expansion Sample Maps -Sorted Naturally by default now?:

So the code looks ok?

From the tiny amount you've posted, yes.