Skip to content

Commit

Permalink
UI|Client: Don't raise alerts when console history open
Browse files Browse the repository at this point in the history
The console history will show the alert anyway, so it is unnecessary
to alert the user about it redundantly.
  • Loading branch information
skyjake committed Apr 11, 2014
1 parent 947843f commit 773d7fd
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions doomsday/client/src/clientapp.cpp
Expand Up @@ -51,6 +51,7 @@
#include "ui/inputsystem.h"
#include "ui/clientwindowsystem.h"
#include "ui/clientwindow.h"
#include "ui/widgets/taskbarwidget.h"
#include "ui/dialogs/alertdialog.h"
#include "ui/styledlogsinkformatter.h"
#include "updater.h"
Expand Down Expand Up @@ -135,6 +136,14 @@ DENG2_PIMPL(ClientApp)
{
if(alertMask.shouldRaiseAlert(entry.metadata()))
{
// Don't raise alerts if the console history is open; the
// warning/error will be shown there.
if(ClientWindow::main().taskBar().isOpen() &&
ClientWindow::main().taskBar().console().isLogOpen())
{
return *this;
}

// We don't want to raise alerts about problems in id/Raven WADs,
// since these just have to be accepted by the user.
if((entry.metadata() & LogEntry::Map) &&
Expand Down

0 comments on commit 773d7fd

Please sign in to comment.