On 07/06/2016 21:01, Mike Kerner wrote:
never mind, Ali responded on one of the bug reports...
FWIW, http://quality.livecode.com/show_bug.cgi?id=17813#c3 Here's Ali's comment in full:
Here is the pull request: https://github.com/livecode/livecode-ide/pull/1192 Essentially the problem was that the 'frame behavior', which unifies all of the palette header and footer handling, assumes that there is a 1-1 mapping between preference menu items and preferences. The original implementation of PB ordering used two preferences for each menu item (sort type and sort order for each of stack, card and control), which meant that the frame behavior couldn't set the checkmarks correctly. Hence the fix was to combine each pair of prefs into one (pb_stackSortOrder & pb_stackSortType -> pb_stackSort), add a mini API for setting and getting the preference to make sure it was in a consistent form, and then changing all the places where these pref names were used. Note I missed one instance, hence bug 17769, which will be fixed in 8.0.2 rc 2 (but not 8.0.2 rc 1 and possibly not 8.1.0 dp 2). The fix for that is here: https://github.com/livecode/livecode-ide/pull/1222 I also took the opportunity there to add tests for each of the ordering types.
Peter -- Dr Peter Brett <peter.br...@livecode.com> LiveCode Technical Project Manager LiveCode 2016 Conference https://livecode.com/edinburgh-2016/ _______________________________________________ use-livecode mailing list use-livecode@lists.runrev.com Please visit this url to subscribe, unsubscribe and manage your subscription preferences: http://lists.runrev.com/mailman/listinfo/use-livecode