Hmm, perhaps you haven't fetched latest changes? Tried the createOnEvent solution in various places but with no success. Finally, i had to remove the refreshView() call and created my own mechanism for refreshing. The relevant changes are here,

https://github.com/chrispetsos/prefsEditors/commit/76469b3994559189a9e79ee5c23a438ecc3fa9af


Date: Thu, 24 Oct 2013 09:45:27 -0700
From: notifications@github.com
To: prefsEditors@noreply.github.com
CC: chrispetsos@msn.com
Subject: Re: [prefsEditors] GPII-187, GPII-188, GPII-225, GPII-227 contributions (#7)

In src/pmt/js/PMT.js:

> +                },
> +                resetButtonText: {
> +                    expander: {
> +                        func: "gpii.uiOptions.pmt.lookupMsg",
> +                        args: ["{uiOptionsLoader}.msgBundle", "resetButtonText"]
> +                    }
> +                },
> +                cancelButtonText: {
> +                    expander: {
> +                        func: "gpii.uiOptions.pmt.lookupMsg",
> +                        args: ["{uiOptionsLoader}.msgBundle", "cancelButtonText"]
> +                    }
> +                }
> +            },
> +            components: {
> +                increaseSizeExtraAdjustersComponent: {

Yes, the duplication's unfortunate and a pain to work with. Cindy and I are almost ready with the composite panel. We're basically just fixing up unit tests at this point and will have it sent off for code review afterwards.


I looked into your issue for the UI breaking. For me the plus/minus buttons don't work at all and the contrast doesn't seem to preview everywhere. However, what breaks for me on save is just the slider for cursor size. I believe this is because the subcomponent it uses. When you refresh view, the linkage it has to the markup is broken. You can try using a createOnEvent for it, which is triggered both by the initial point when it should be created and after a call to refreshView.


—
Reply to this email directly or view it on GitHub.