Quick update: I found the region while fixing the offset issue (forgot to call 
ui_window_to_block()). Calling ED_region_tag_redraw(data->region) does indeed 
fix the issue. I'd still be curious to know as to why RNA_property_update() 
isn't working as expected for future reference, but no worries if you have 
other things to do, at least I got my little wishlist working. =P

Thanks again!
-Patrick

> From: [email protected]
> To: [email protected]
> Date: Thu, 25 Apr 2013 22:48:02 +0200
> Subject: Re: [Bf-committers] Question regarding RNA_property_update().
> 
> Hey Campbell,
> 
> Thanks for your quick response. I'm not sure where I should get the region 
> from, but I uploaded a patch[1]. The line of interest is 53 in the diff, 5473 
> in the applied C file.
> 
> Thanks!
> 
> [1]: http://www.pasteall.org/41731/diff
> 
> > Date: Fri, 26 Apr 2013 06:37:11 +1000
> > From: [email protected]
> > To: [email protected]
> > Subject: Re: [Bf-committers] Question regarding RNA_property_update().
> > 
> > On Fri, Apr 26, 2013 at 6:07 AM, patrick boelens <[email protected]> wrote:
> > > Hey everyone,
> > >
> > > I figured I'd add in some quick hotkeys for the Colorband in Blender such 
> > > as Shift+D to duplicate an element, X to remove one, etc. I got this 
> > > mostly working, except for a weird offset at start when dragging and, my 
> > > main issue I want to ask about, updating the RNA to immediately show the 
> > > changes without having to mouse-out of the area.
> > >
> > > I have added the actions in interface_handlers.c ->
> > > ui_do_button() like so. The problem lies with the RNA_property_update() 
> > > call, though I'm not sure why. If anyone can point me in the right 
> > > direction here it'd be much appreciated!
> > >
> > > Thanks in advance,
> > > Patrick
> > 
> > Does tagging the region for redraw help?
> > ED_region_tag_redraw(ar);
> > 
> > Even if it does it shouldn't be required, the update function should
> > send notifiers so views on this data are refreshed.
> > 
> > Its hard to guess though so suggest to post a patch.
> > _______________________________________________
> > Bf-committers mailing list
> > [email protected]
> > http://lists.blender.org/mailman/listinfo/bf-committers
>                                         
> _______________________________________________
> Bf-committers mailing list
> [email protected]
> http://lists.blender.org/mailman/listinfo/bf-committers
                                          
_______________________________________________
Bf-committers mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-committers

Reply via email to