Am 12.09.2017 um 11:19 schrieb LyX Ticket Tracker <t...@lyx.org>:
> 
> #10662: "LyX: Add BibTeX Database" dialog goes behind other dialog after 
> browse
> step
> ------------------------------+----------------------------
> Reporter:  jkulesza          |       Owner:  lasgouttes
>     Type:  defect            |      Status:  fixedinmaster
> Priority:  normal            |   Milestone:  2.2.4
> Component:  general           |     Version:  2.3.0dev
> Severity:  normal            |  Resolution:
> Keywords:  os=macosx, patch  |
> ------------------------------+----------------------------
> Changes (by spitz):
> 
> * cc: rgheck (added)
> * milestone:  2.3.0 => 2.2.4
> 
> 
> Comment:
> 
> Candidate for stable. The bug has also been reported for 2.2 (see #10396),
> and the patch is straightforward.

Wait a minute, please. I have some questions…

1. Why is QDialog::setModal(true) used instead of setModal(true)?

2. What is with the other setModal uses? 
Shouldn’t these followed by a call to setWindowModality(Qt::WindowModal) too?

3. What is with the call of setModal(Qt::WindowModal) in GuiCompareHistory.cpp?
Is this really correct?

Stephan

Reply via email to