Repository: Qiskit/qiskit

Issue:
interactive=True has no effect with default drawer output ('text')
<!-- ⚠️ If you do not respect this template, your issue will be closed -->
<!-- ⚠️ Make sure to browse the opened and closed issues to confirm this idea does not exist. -->

### What is the expected enhancement?
I think we either convert the text output to an image and make it pop out, or we clearly document that interactive only works when output is 'latex' or 'mpl'. Not the default 'text'




Discussion:
So the interactive flag is documented as opening the visualization in a new window. That really wasn't an option with text. Did you have a way in mind for doing it? I can update the docstring though. When I wrote the docstring for the circuit_drawer() the text drawer didn't exist.
Updating the docstring (maybe a 'warning' that interactive doesn't work with this `output` arg?) sounds good. I originally thought converting the text output with a bounding box to an image and showing that, but that's probably too crazy.

Resolving patch:
diff --git a/qiskit/circuit/quantumcircuit.py b/qiskit/circuit/quantumcircuit.py
--- a/qiskit/circuit/quantumcircuit.py
+++ b/qiskit/circuit/quantumcircuit.py
@@ -320,8 +320,10 @@ def draw(self, scale=0.7, filename=None, style=None, output='text',
                 circuit. Valid choices are `text`, `latex`, `latex_source`,
                 `mpl`.
             interactive (bool): when set true show the circuit in a new window
-                (cannot inline in Jupyter). Note when used with the
-                latex_source output type this has no effect
+                (for `mpl` this depends on the matplotlib backend being used
+                supporting this). Note when used with either the `text` or the
+                `latex_source` output type this has no effect and will be
+                silently ignored.
             line_length (int): sets the length of the lines generated by `text`
             reverse_bits (bool): When set to True reverse the bit order inside
                 registers for the output visualization.
diff --git a/qiskit/tools/visualization/_circuit_visualization.py b/qiskit/tools/visualization/_circuit_visualization.py
--- a/qiskit/tools/visualization/_circuit_visualization.py
+++ b/qiskit/tools/visualization/_circuit_visualization.py
@@ -77,8 +77,10 @@ def circuit_drawer(circuit,
             to mpl. However this behavior is deprecated and in a future release
             the default will change.
         interactive (bool): when set true show the circuit in a new window
-            (cannot inline in Jupyter). Note when used with the latex_source
-            output type this has no effect.
+            (for `mpl` this depends on the matplotlib backend being used
+            supporting this). Note when used with either the `text` or the
+            `latex_source` output type this has no effect and will be silently
+            ignored.
         line_length (int): Sets the length of the lines generated by `text`
             output type. This useful when the drawing does not fit in the
             console. If None (default), it will try to guess the console width


Test patch: