Hi,<br><br>I think you should have done this at the more global-level in anim_filter.c instead of modifying each function where this might apply. The whole goal of this animation filtering system was to avoid having to do hacks like that all over the place (see the old 2.4x branches/trunk, especially the NLA Editor for an example of how nasty the old filtering stuff was). Keep in mind also that if this option is broken for the Graph Editor, it is also broken for the DopeSheet (and soon the NLA too).<br>

<br>What you've done here is really to restrict many select tools to operating only on curves that are selected. This doesn't really match what that setting is supposed to be used for. Also, you've only 'fixed' this selection, while there are also editing tools which should get affected by the setting in the exact same way.<br>

<br><div class="gmail_quote">On Fri, May 29, 2009 at 3:32 PM, Jess Balint <span dir="ltr"><<a href="mailto:jbalint@gmail.com" target="_blank">jbalint@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

Hi,<br>
I've attached a patch to get "Only display selected Objects" option<br>
working in the graph editor. I think it makes sense, unless there's some<br>
other global way to enforce this constraint.<br>
<br>
However, the fact that border select (and some others), don't select the<br>
associated channel when the control points are selected, means you can<br>
still have some keys selected (and invisible) that would be translated<br>
with a visible key selection. Would it make sense for all selection<br>
operations to sync the channel selection to match that of the keys?<br>
(and likewise for deselection, eg deselect a channel when none of its<br>
keys are selected)</blockquote><div>Yes. There are some syncing issues here that need to be cleaned up.<br> <br><br>Joshua<br></div></div><br>