[Feature Request] add toggle to diable "expand image" feature and restore the pervious behaviors

Environment data

Version: 1.35.1 (user setup) Commit: c7d83e57cd18f18026a8162d042843bda1bcf21f Date: 2019-06-12T14:30:02.622Z Electron: 3.1.8 Chrome: 66.0.3359.181 Node.js: 10.2.0 V8: 6.6.346.32 OS: Windows_NT x64 10.0.17763 Python Extension: 2019.6.22090

Issue:

The new release includes a new feature to interact with the plots:

This implement has caused the following issues:

1. users can no longer drag the plots to wherever they want. In fact, it seems like many users save their plots this way: https://github.com/microsoft/vscode-python/issues/5201 https://github.com/microsoft/vscode-python/issues/4132

2. When saving a plot as a .png file in the new Plots windows, the image got cut off for some reasons.

import matplotlib.pyplot as plt
import numpy as np
import geopandas as gpd

# load dataset
path = gpd.datasets.get_path('naturalearth_lowres')
gdf = gpd.read_file(path)
# generate a random column
gdf['random_col'] = np.random.normal(100, 10, len(gdf))

# plot quantiles map
fig, ax = plt.subplots(figsize=(10, 10))
gdf.plot(column='random_col', scheme='quantiles', k=5, cmap='Blues',
         legend=True, legend_kwds=dict(loc=6), ax=ax)

1. the DPI of the plots in the Plots windows seem very high, but the png image file saved via “export to different formats” has a much lower DPI.

This new “expand image” feature completely change the behavior of the vs code. I understand this feature might be convenient for some users, but I’d like to use code to explicitly set the dpi and save plots.

Please consider adding a toggle for it or at least disable the double-click and restore the “drag to desktop” behavior.

[rchiodo]

This has already been implemented in the insider’s build. We generally add the ‘sign-off’ label when something moves to insider’s but hasn’t shipped in the general release yet.

Issues with the ‘data-science parking lot’ or ‘parking-lot’ are in limbo. Issues with the ‘backlog’ label are stuff we’re working on for the next release.

We close issues when the ship in the general release.

I also reopened the other issue.

[rchiodo]

No that was a release for some specific bug fixes. It doesn’t have the enablePlotViewer bits in it. That’s only in the insider’s build.

You can see what was fixed in the changelog for the release branch: https://github.com/microsoft/vscode-python/blob/release/CHANGELOG.md

Looks like a fix for a/b testing.