Python console minor issues

src/appleseed.studio/CMakeLists.txt

@@ -218,6 +218,8 @@ source_group ("mainwindow\\project" FILES
 )
 
 set (mainwindow_pythonconsole_sources
+    mainwindow/pythonconsole/fontsizechangeable.cpp
+    mainwindow/pythonconsole/fontsizechangeable.h
     mainwindow/pythonconsole/linenumberarea.cpp
     mainwindow/pythonconsole/linenumberarea.h
     mainwindow/pythonconsole/outputredirector.cpp

src/appleseed.studio/mainwindow/pythonconsole/fontsizechangeable.cpp

@@ -0,0 +1,73 @@
+
+//
+// This source file is part of appleseed.
+// Visit http://appleseedhq.net/ for additional information and resources.
+//
+// This software is released under the MIT license.
+//
+// Copyright (c) 2017 Gleb Mishchenko, The appleseedhq Organization
+//
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal
+// in the Software without restriction, including without limitation the rights
+// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+// copies of the Software, and to permit persons to whom the Software is
+// furnished to do so, subject to the following conditions:
+//
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+// THE SOFTWARE.
+//
+
+// Interface header.
+#include "fontsizechangeable.h"
+
+namespace appleseed {
+namespace studio {
+
+FontSizeChangeable::FontSizeChangeable(QWidget* parent)
+  : QPlainTextEdit(parent)
+{
+}
+
+void FontSizeChangeable::keyPressEvent(QKeyEvent* event)
+{
+    if (event->modifiers() & Qt::ControlModifier &&
+        (event->key() == Qt::Key_Plus || event->key() == Qt::Key_Equal))
+        change_font_size(1);
+    else if (event->modifiers() & Qt::ControlModifier && event->key() == Qt::Key_Minus)
+        change_font_size(-1);
+    else if (event->modifiers() & Qt::ControlModifier && event->key() == Qt::Key_0)
+        change_font_size(QFont().pointSize() - font().pointSize());
+    else
+        QPlainTextEdit::keyPressEvent(event);
+}
+
+void FontSizeChangeable::wheelEvent(QWheelEvent* event)
+{
+    if (event->modifiers() & Qt::ControlModifier)
+        // Minus here because if you turn wheel up delta is negative
+        // while font should be incremented.
+        change_font_size(- event->delta() / 120);
+    else
+        QPlainTextEdit::wheelEvent(event);
+}
+
+void FontSizeChangeable::change_font_size(const int delta)
+{
+    int new_font_size = font().pointSize() + delta;
+    QFont new_font = font();
+    new_font.setPointSize(new_font_size);
+    setFont(new_font);
+    emit(fontChanged(new_font));
+}
+
+}   // namespace studio
+}   // namespace appleseed

src/appleseed.studio/mainwindow/pythonconsole/fontsizechangeable.h

@@ -0,0 +1,61 @@
+
+//
+// This source file is part of appleseed.
+// Visit http://appleseedhq.net/ for additional information and resources.
+//
+// This software is released under the MIT license.
+//
+// Copyright (c) 2017 Gleb Mishchenko, The appleseedhq Organization
+//
+// Permission is hereby granted, free of charge, to any person obtaining a copy
+// of this software and associated documentation files (the "Software"), to deal
+// in the Software without restriction, including without limitation the rights
+// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+// copies of the Software, and to permit persons to whom the Software is
+// furnished to do so, subject to the following conditions:
+//
+// The above copyright notice and this permission notice shall be included in
+// all copies or substantial portions of the Software.
+//
+// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+// THE SOFTWARE.
+//
+
+#ifndef APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_FONTSIZECHANGEABLE_H
+#define APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_FONTSIZECHANGEABLE_H
+
+// Qt headers.
+#include <QObject>
+#include <QPlainTextEdit>
+
+namespace appleseed {
+namespace studio {
+
+class FontSizeChangeable
+    : public QPlainTextEdit
+{
+    Q_OBJECT
+
+  public:
+    FontSizeChangeable(QWidget* parent);
+
+  protected:
+    void keyPressEvent(QKeyEvent* event);
+    void wheelEvent(QWheelEvent* event);
+
+  private:
+    void change_font_size(const int delta);
+
+  signals:
+    void fontChanged(QFont);
+};
+
+}   // namespace studio
+}   // namespace appleseed
+
+#endif // !APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_FONTSIZECHANGEABLE_H

src/appleseed.studio/mainwindow/pythonconsole/pythoneditor.cpp

@@ -44,7 +44,7 @@ namespace appleseed {
 namespace studio {
 
 PythonEditor::PythonEditor(QWidget* parent)
-  : QPlainTextEdit(parent)
+  : FontSizeChangeable(parent)
 {
     setObjectName("python_editor");
     setUndoRedoEnabled(true);
@@ -76,13 +76,11 @@ void PythonEditor::keyPressEvent(QKeyEvent* event)
 {
     if (event->key() == Qt::Key_Tab)
         insert_spaces(4);
-    else if (event->key() == Qt::Key_Plus || event->key() == Qt::Key_Equal)
-        change_font_size(1);
-    else if (event->key() == Qt::Key_Minus)
-        change_font_size(-1);
     else
     {
-        QPlainTextEdit::keyPressEvent(event);
+        if (event->key() == Qt::Key_Return || event->key() == Qt::Key_Enter)
+            event->setModifiers(event->modifiers() & ~Qt::ShiftModifier);
+        FontSizeChangeable::keyPressEvent(event);
         if (event->key() == Qt::Key_Return || event->key() == Qt::Key_Enter)
             indent();
     }
@@ -130,25 +128,6 @@ void PythonEditor::insert_spaces(const size_t count)
     insertPlainText(spaces.c_str());
 }
 
-void PythonEditor::wheelEvent(QWheelEvent* event)
-{
-    if (QApplication::keyboardModifiers().testFlag(Qt::ControlModifier))
-        // Minus here because if you turn wheel up delta is negative
-        // while font should be incremented.
-        change_font_size(- event->delta() / 120);
-    else
-        QPlainTextEdit::wheelEvent(event);
-}
-
-void PythonEditor::change_font_size(const int delta)
-{
-    int new_font_size = font().pointSize() + delta;
-    QFont new_font = font();
-    new_font.setPointSize(new_font_size);
-    setFont(new_font);
-    emit(fontChanged(new_font));
-}
-
 void PythonEditor::slot_highlight_current_line()
 {
     QTextEdit::ExtraSelection selection;

src/appleseed.studio/mainwindow/pythonconsole/pythoneditor.h

@@ -29,6 +29,9 @@
 #ifndef APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_PYTHONEDITOR_H
 #define APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_PYTHONEDITOR_H
 
+// appleseed.studio headers.
+#include "mainwindow/pythonconsole/fontsizechangeable.h"
+
 // Qt headers.
 #include <QObject>
 #include <QPlainTextEdit>
@@ -38,17 +41,14 @@
 #include <string>
 
 // Forward declarations.
-class QKeyEvent;
-class QResizeEvent;
-class QWheelEvent;
 class QWidget;
 namespace appleseed { namespace studio { class LineNumberArea; } }
 
 namespace appleseed {
 namespace studio {
 
 class PythonEditor
-  : public QPlainTextEdit
+  : public FontSizeChangeable
 {
     Q_OBJECT
 
@@ -59,15 +59,8 @@ class PythonEditor
     // Event used to update line number area.
     void resizeEvent(QResizeEvent* event);
 
-    // Event used for indentation and font size change.
     void keyPressEvent(QKeyEvent* event);
 
-    // Event used to change font size on wheel rotation.
-    void wheelEvent(QWheelEvent* event);
-
-  signals:
-    void fontChanged(QFont);
-
   private slots:
     void slot_highlight_current_line();
 
@@ -79,8 +72,6 @@ class PythonEditor
     void indent();
     void indent_like_previous(const std::string& previous);
     void insert_spaces(const size_t count);
-
-    void change_font_size(const int delta);
 };
 
 }       // namespace studio

src/appleseed.studio/mainwindow/pythonconsole/pythonoutput.cpp

@@ -29,14 +29,11 @@
 // Interface header.
 #include "pythonoutput.h"
 
-// Qt headers.
-#include <QApplication>
-
 namespace appleseed {
 namespace studio {
 
 PythonOutput::PythonOutput(QWidget* parent)
-  : QPlainTextEdit(parent)
+  : FontSizeChangeable(parent)
 {
     setObjectName("python_output");
     setUndoRedoEnabled(false);
@@ -47,33 +44,5 @@ PythonOutput::PythonOutput(QWidget* parent)
         Qt::TextSelectableByKeyboard);
 }
 
-void PythonOutput::keyPressEvent(QKeyEvent* event)
-{
-    if (event->key() == Qt::Key_Plus || event->key() == Qt::Key_Equal)
-        change_font_size(1);
-    else if (event->key() == Qt::Key_Minus)
-        change_font_size(-1);
-    else
-        QPlainTextEdit::keyPressEvent(event);
-}
-
-void PythonOutput::wheelEvent(QWheelEvent* event)
-{
-    if (QApplication::keyboardModifiers().testFlag(Qt::ControlModifier))
-        // Minus here because if you turn wheel up delta is negative
-        // while font should be incremented.
-        change_font_size(- event->delta() / 120);
-    else
-        QPlainTextEdit::wheelEvent(event);
-}
-
-void PythonOutput::change_font_size(const int delta)
-{
-    int new_font_size = font().pointSize() + delta;
-    QFont new_font = font();
-    new_font.setPointSize(new_font_size);
-    setFont(new_font);
-}
-
 }   // namespace studio
 }   // namespace appleseed

src/appleseed.studio/mainwindow/pythonconsole/pythonoutput.h

@@ -29,6 +29,9 @@
 #ifndef APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_PYTHONOUTPUT_H
 #define APPLESEED_STUDIO_MAINWINDOW_PYTHONCONSOLE_PYTHONOUTPUT_H
 
+// appleseed.studio headers.
+#include "mainwindow/pythonconsole/fontsizechangeable.h"
+
 // Qt headers.
 #include <QObject>
 #include <QPlainTextEdit>
@@ -40,22 +43,12 @@ namespace appleseed {
 namespace studio {
 
 class PythonOutput
-    : public QPlainTextEdit
+    : public FontSizeChangeable
 {
     Q_OBJECT
 
   public:
     explicit PythonOutput(QWidget* parent);
-
-  protected:
-    // Event used for indentation and font size change.
-    void keyPressEvent(QKeyEvent* event);
-
-    // Event used to change font size on wheel rotation.
-    void wheelEvent(QWheelEvent* event);
-
-  private:
-    void change_font_size(const int delta);
 };
 
 }   // namespace studio