Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

replace "Yes, close all" button text with 'Quit'

The explanation is already in the body of the message, and the 'close all'
is confusing for `%qtconsole` launched consoles, which don't halt
their kernels.
  • Loading branch information...
commit 05cbf58d253dbbc4319f109958e139b7ec0a52c7 1 parent bc36c47
Min RK authored

Showing 1 changed file with 2 additions and 2 deletions. Show diff stats Hide diff stats

  1. +2 2 IPython/frontend/qt/console/mainwindow.py
4 IPython/frontend/qt/console/mainwindow.py
@@ -880,8 +880,8 @@ def closeEvent(self, event):
880 880 else:
881 881 msg = "Close console, stop kernel, and Quit?"
882 882 info = "Kernels not started here (e.g. notebooks) will be left alone."
883   - closeall = QtGui.QPushButton("&Yes, quit everything", self)
884   - closeall.setShortcut('Y')
  883 + closeall = QtGui.QPushButton("&Quit", self)
  884 + closeall.setShortcut('Q')
885 885 box = QtGui.QMessageBox(QtGui.QMessageBox.Question,
886 886 title, msg)
887 887 box.setInformativeText(info)

0 comments on commit 05cbf58

Please sign in to comment.
Something went wrong with that request. Please try again.