ViewScene bug report button #1997
Conversation
…ieved through qt and opengl
| @@ -46,13 +46,13 @@ namespace SCIRun | |||
| public: | |||
| explicit Screenshot(QGLWidget *glwidget, QObject *parent = nullptr); | |||
There was a problem hiding this comment.
This won't merge easily with https://github.com/SCIInstitute/SCIRun/pull/1979/files#diff-09b4cb29ce5bcc63aba5130a741d0f1a. I guess we can wait and see after #1979 is merged into qt5-dev.
There was a problem hiding this comment.
I think the function for grabbing frame buffer is still the same.
|
do we want this option on the main vs window? what about in the configure window? |
|
@RubioJr9 This needs a proper merge from master--the method naming changed on the new opengl widget. |
|
It's merged properly now |
|
@RubioJr9 will move the button to the new developer tab. |
| // Temporarily save screenshot so that it can be sent over email | ||
| takeScreenshot(); | ||
| QImage image = screenshotTaker_->getScreenshot(); | ||
| QString location = QDir::homePath() % QLatin1String("/scirun5screenshots/scirun_bug.png"); |
There was a problem hiding this comment.
@RubioJr9 I tested this and it works great. One request though. It would be nice to make unique screenshot files for people that test frequently. It's easy to append a timestamp to the file name and then copy that unique name here. Could you add that too? Thanks.
|
@RubioJr9 I merged this one at long last and just created another issue to continue my requests (#2022) |
Added a button in the view scene toolbar that saves a screenshot and pulls up an email template to send to @dcwhite.
Opens through the default mail client.
Requires qt5 to get system info.
Issues:
#1989
Edit:
Tested on MacOS 10.13 and Arch Linux
This won't pass travis until all builds are qt5