webclient: list-items not sorted alphabetically in page panel

[Malte]

Currently, e.g. a list of associated Topics does not read as alphabetically sorted because the sort function in JavaScript? does not compare strings using toLowerCase(), so currently the term "audio" is listed after "Zoltan".

[Jörg Richter]

Webclient: case-insensitive topic sorting (#521).

The "what's related" topic listing in the detail panel is sorted case-insensitively.
Thanks, Malte, for reporting!

Close #521.

• Changeset: 2e801030e09172e9047d8912bd66a532d3255dfd
• URL: ​https://github.com/jri/deepamehta/commit/2e801030e09172e9047d8912bd66a532d3255dfd

[Jörg Richter]

Webclient: case-insensitive topic sorting (#521).

The "what's related" topic listing in the detail panel is sorted case-insensitively.
Thanks, Malte, for reporting!

Close #521.

• Changeset: 2e801030e09172e9047d8912bd66a532d3255dfd
• URL: ​https://github.com/jri/deepamehta/commit/2e801030e09172e9047d8912bd66a532d3255dfd

[jri]

Sorting does not always work. Sometimes a client-side error occurs and breaks the page panel rendering:

TypeError: 'undefined' is not a function (evaluating 'topic_1.value.toLowerCase()')
rest_client.js:257

I guess this happens with non-text child topics.

To force the error try revealing the DeepaMehta workspace (by selecting it from the Workspace menu).

[Malte]

Correct, I could reproduce that.
We could do cast the to be compared values to become a String in before comparison, like this:

var a_value = "" + topic_1.value
// and then compare that
a_value.toLowerCase()

Sorry, there might be better solutions but I don't got one at hand for this right now.

[Jörg Richter]

Webclient fix: sorting non-string values (#521).

Topic listings in the detail panel display properly also when non-string topic values are involved.

Note: for pagmatic reasons numeric values are sorted alphanumerically.

Close #521.

• Changeset: 85d1a2b0f363629b2bf1fa6b5c134f7aebbe02fe
• URL: ​https://github.com/jri/deepamehta/commit/85d1a2b0f363629b2bf1fa6b5c134f7aebbe02fe