[Development] QGuiApplication::topLevelWindows()

[Anselmo L. S. Melo]

On Fri, Jan 27, 2012 at 8:15 AM, Anselmo L. S. Melo
<anselmo.melo at openbossa.org> wrote:
> On Fri, Jan 27, 2012 at 4:51 AM, Samuel Rødal <samuel.rodal at nokia.com> wrote:
>> On 01/26/2012 08:21 PM, ext Anselmo L. S. Melo wrote:
>>> Hi,
>>>
>>> A window is a top level window if it has no parent, correct? Code and comments in qwindow.cpp also confirm that.
>>>
>>> With that in mind, it seems reasonable that QGuiApplication::topLevelWindows() should return a list of QWindows with parent()==0.
>>> However, the returned list contains all QWindows created, including the ones that don't respect this condition.
>>>
>>> So, it seems we need to either adjust QGuiApplication::topLevelWindows() or control which windows are added to
>>> QGuiApplicationPrivate::window_list - currently, all windows are prepended to this list in QWindow's ctors.
>>
>> Agreed, maybe just make QGuiApplication::topLevelWindows() filter the
>> window_list, since it would take more work to keep track of a
>> top_level_window_list in qwindow.cpp.
>>
>>> Also, would it be useful to have QGuiApplication::allWindows()?
>>
>> Yep, it might be.
>
> Ok, I'll work on that when I arrive at the office.

This patch was submitted here: http://codereview.qt-project.org/#change,14492

>
> Regards,
> Anselmo
>
>
> --
> Anselmo L. S. Melo
> openBossa / INdT
> http://www.indt.org



-- 
--
--
Anselmo L. S. Melo
openBossa / INdT
http://www.indt.org
http://www.anselmolsm.org