Bug 875126
| Summary: | "Take screenshot" dialog should add .png to the saved file name and ignore all common image extensions typed | ||
|---|---|---|---|
| Product: | Red Hat Enterprise Linux 6 | Reporter: | David Jaša <djasa> |
| Component: | virt-viewer | Assignee: | Daniel Berrangé <berrange> |
| Status: | CLOSED ERRATA | QA Contact: | Virtualization Bugs <virt-bugs> |
| Severity: | low | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 6.4 | CC: | cfergeau, dallan, dblechte, desktop-qa-list, dyasny, gkong, marcandre.lureau, mjenner, mzhan, rwu, tlavigne, yupzhang |
| Target Milestone: | rc | ||
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | virt-viewer-0.5.2-18.el6 | Doc Type: | Bug Fix |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 875124 | Environment: | |
| Last Closed: | 2013-02-21 09:34:11 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | |||
| Bug Blocks: | 886216 | ||
|
Description
David Jaša
2012-11-09 15:20:17 UTC
patch are on the ML commit 8a7aa6a1e15dfc2adbb9627ec5e914e445f434ab
Author: Christophe Fergeau <cfergeau>
Date: Tue Nov 13 12:52:31 2012 +0100
Make sure png screenshots have a .png extensions
When falling back to saving to .png, the filename might not
end with .png. This commit appends the .png extension to the
screenshot filename if it's missing.
commit f9935c832b84a31bc91f5fcdfdf459ee3a0913fe
Author: Christophe Fergeau <cfergeau>
Date: Tue Nov 13 12:50:10 2012 +0100
Allow to save to other formats than png
Currently, the screenshots can only be saved to png. This commit
checks if the file extension is a known one, and will save to this
format if it is. Otherwise it will fallback to saving to png.
commit 45a791082ed07b4dfaa347ed434f29324dbd6815
Author: Christophe Fergeau <cfergeau>
Date: Tue Nov 13 12:47:50 2012 +0100
Set a default dir/name for screenshots
It makes sense for the screenshots to be saved in ~/Images,
especially as otherwise the filechooser will display
'recent documents' to which we cannot save. This commit also sets
the default screenshot name to 'Screenshot'.
I can reproduce this issue with virt-viewer-0.5.2-17.el6.x86_64. Verified this issue with virt-viewer-0.5.2-18.el6.x86_64. 1.Use virt-viewer to open a guest. 2.Click File->Screenshot,the "Save screenshot" diaglog will pop up.The default name is "Screenshot".The default path is ~/Pictures. 3.Select a directory and then click "Save". 4.Check the file,the file format is PNG and .png has been added to the file. # file Screenshot.png Screenshot.png: PNG image data, 800 x 600, 8-bit/color RGB, non-interlaced 5.Repeat the step 2,and save the screenshot with name: Screenshot.jpg , Screenshot.bmp and Screenshot.sdgfsdfawsdf. Screenshot.bmp: PC bitmap, Windows 3.x format, 800 x 600 x 24 Screenshot.jpg: JPEG image data, JFIF standard 1.01 Screenshot.png: PNG image data, 800 x 600, 8-bit/color RGB, non-interlaced Screenshot.sdgfsdfawsdf.png: PNG image data, 1024 x 768, 8-bit/color RGB, non-interlaced According to comment 3,change the bug status to VERIFIED. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2013-0361.html |