<div dir="ltr">No, I didn't mean directly connect to the slot in the editor. If you wanted to do that, btw, you could have passed a signal directly. I meant, after the KAction has been connected to the local slot in the LessonView, instead of emitting a signal to the editor, just call LessonView's inherited expandAll()/collapseAll() in that local slot. Hence, LessonView's slots would be calling the inherited methods directly.<div class="gmail_extra">
<br><br><div class="gmail_quote">On Fri, Mar 14, 2014 at 8:17 PM, Avinash Zebisky <span dir="ltr"><<a href="mailto:avinashzebisky@gmail.com" target="_blank">avinashzebisky@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi,<div><br></div><div>Thanks for review. I'll remove that <span style="font-size:8pt;line-height:14.933333396911621px;white-space:pre-wrap">m_wordTypeView</span><span style="font-size:8pt;line-height:14.933333396911621px;white-space:pre-wrap">-></span><span style="font-size:8pt;line-height:14.933333396911621px;white-space:pre-wrap">expandAll</span><span style="font-size:8pt;line-height:14.933333396911621px;white-space:pre-wrap">(); </span></div>


<div><br></div><div class="gmail_extra"><div class=""><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:Verdana,Arial,Helvetica,Sans-Serif">


<div><br>





 </div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">Good work, btw. Thanks for the patch. :) Here are a few points about it:

1. There isn't really a need to expand/collapse WordTypeView when the KAction is called. The user isn't expecting it and didn't call for it.
2. As the only work to be performed by the KAction is to call the LessonView's own methods (expandAll/collapseAll), there isn't a need for a signal to be emitted and then performing stuff in the Editor. This can all be done in LessonView itself. Let's keep things clean and tidy, this helps other code readers as well. :)</pre>



 </div></div></blockquote></div></div>I tried to connect KAction directly to slot in editor.cpp. But that didn't work. Moreover I don't see, how should I directly connect them :/.</div><div class="gmail_extra">Kindly, guide me :)</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">Regards, </div></div>
</blockquote></div><br></div></div>