InDesign CS3 Win: Playing with PictureIcon...

Question

Hello Folks:<br /><br />   I have a rare moment to actually explore the SDK a bit on my own without deadline worries or similar impediments to creativity.  I thus decided to play with the sample plugin called PictureIcon.  I compiled it and ran it without a problem, although the writers of the plugin did little more than show how to implement an event handler to change the image used in a picture button when that icon is clicked.  <br /><br />   The problem is that the range of behavior implemented is very, very limited.  Clicking the one picturre button implemented toggles between two images when there are eight included in the project.  So I decided as an exercise for the student, to rewrite the project to cycle through all eight images.  <br /><br />   The problem is that if I click the picture button without a breakpoint included in the event handler, the picture button does not change, as expected.  If I do include a breakpoint, the picture button does change.  I even went ahead and included an Update call based upon comments made in the event handler header, to no effect.  Here's the changed code (I left the old code in for reference purposes):<br /><br />bool16 PicIcoPictureWidgetEH::LButtonDn(IEvent* e)<br />{<br />//DebugMessage("I am inside the event handler.");<br />     do<br />    {<br />          int32 rsrcID = 0;<br /><br />          InterfacePtr<IBooleanControlData> controlData(this, IID_IBOOLEANCONTROLDATA); // no kDefaultIID<br />          if(controlData==nil) <br />        {<br />               ASSERT_FAIL("PicIcoPictureWidgetEH::LButtonDn - No IBooleanControlData found on this boss.");<br />               break;<br />          }<br /><br />          InterfacePtr<IUIFontSpec> fontSpec(this, IID_IUIFONTSPEC); // no kDefaultIID<br />          if(fontSpec==nil) <br />        {<br />               ASSERT_FAIL("PicIcoPictureWidgetEH::LButtonDn - No IUIFontSpec found on this boss.");<br />               break;<br />          }<br /><br />        /*<br />         *  This is the meat of the issue.  If the <br />         *  correct button is clicked, the value of <br />         *  rsrcID is changed to a font number id.  <br />         * <br />         *  It is this font number id which is used to <br />         *  decide which picture will be displayed with <br />         *  each click.  One does not relate to the other;<br />         *  the programmer used this value as a matter<br />         *  of convenience.  Iam going to try something <br />         *  else here, just for fun.<br />         */<br />          //bool16 isPressedIn = controlData -> IsSelected();<br />          //if(isPressedIn == kTrue) <br />        {<br />               //controlData       ->       Deselect();     <br />               rsrcID = Resource;<br /><br />            Resource++;<br />            if(Resource > kPicIcoCenterCompPictureRsrcID)<br />                Resource = kPicIcoTopOutsidePictureRsrcID;<br /><br />            //isPressedIn = controlData -> IsSelected();<br />        }<br /><br />#ifdef OldCode<br />          if(isPressedIn == kTrue) <br />        {<br />               controlData       ->       Deselect();     <br />               rsrcID = fontSpec ->      GetFontID();<br />          } <br />        else <br />        {<br />               controlData       ->          Select();<br />               rsrcID = fontSpec -> GetHiliteFontID();<br />          }<br />#endif<br />          InterfacePtr<IControlView> myView(this, UseDefaultIID());<br />          if(myView == nil) <br />        {<br />               ASSERT_FAIL("PicIcoPictureWidgetEH::LButtonDn - No IControlView found on this boss.");<br />               break;<br />          }<br /><br />          if(rsrcID != 0) <br />        {<br />               myView -> SetRsrcID(rsrcID);<br />            Update(e);     <br />          }<br />          return kTrue;<br />        /*<br />           * early return as here we have already<br />           * handled the event and do not need to<br />           * delegate any processing to the base class.<br />         */<br /><br />     } while(0);<br /><br />    /*<br />     *  Of we get this far then it means that the program <br />     *  should perform the default <br />     */<br />     return CEventHandler::LButtonDn(e);<br />}<br /><br />Does anyone see what I am doing wrong?<br /><br />TIA!<br /><br />John

Answer 1

Hello John,
I think you should start by modifying your *WidgetEH boss definition in your framework resources - replacing the IID_IBOOLEANCONTROLDATA it aggregates by an IID_IINTEGERCONTROLDATA - then you'll have to follow this boolean field down deep into your *WidgetEH implementation to replace it to any flavour of integer value you are used to.
Doing it that way you'll be able to store a state value from 0 to 7 - by example: that would be much closer from the behaviour you are looking for than a boolean toggle I'll guess.
Is this making sense to you?
Best regards
Patrick Perroud