Xscreensaver photopile hack configuration cryptic, images off screen

Bug #1007862 reported by D Tangman on 2012-06-02
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
xscreensaver (Ubuntu)
Undecided
Unassigned

Bug Description

In the settings for the xscreensaver photopile hack, it is not obvious that "Clip images" applies only when "Simulate instant film" is enabled. Further, the distinction between "Animation speed" and "Frame rate" is opaque, particularly since the photopile man page talks about the "-speed" and "-delay" options without relating them to the settings.

In addition, the hack places the selected images up to 50% offscreen. The effect is more aesthetically pleasing if the images are confined to the bounds of the screen.

The attached patch changes the labels in the photopile settings from "Clip images" to "Clip to square (for instant film)", "Animation speed" to "Animation speed (Slow: More frames)", and "Frame rate" to "Animation frame rate"; the latter is to make it clear that the last two settings affect the same aspect of the hack's behavior. It also modifies the random image placement so that prior to applying the random tilt the image lies entirely within the bounds of the screen.

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: xscreensaver-gl-extra 5.15-2ubuntu1 [modified: usr/lib/xscreensaver/photopile usr/share/xscreensaver/config/photopile.xml]
ProcVersionSignature: Ubuntu 3.2.0-24.39-generic 3.2.16
Uname: Linux 3.2.0-24-generic x86_64
NonfreeKernelModules: nvidia
ApportVersion: 2.0.1-0ubuntu8
Architecture: amd64
Date: Sat Jun 2 12:22:12 2012
SourcePackage: xscreensaver
UpgradeStatus: No upgrade log present (probably fresh install)

D Tangman (dgtangman) wrote :

The attachment "photopile.diff" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Mamoru TASAKA (mtasaka) wrote :

> The attached patch changes

It is highly appreciated if you would send your patch to the upstream developer.

D Tangman (dgtangman) wrote :

> It is highly appreciated if you would send your patch to the upstream developer.

Done.

Mamoru TASAKA (mtasaka) wrote :

> photopile.diff

While the comment changes in this photopile.diff was not applied, the changes in hacks/glx/photopile.c is already included in xscreensaver 5.16.

Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in xscreensaver (Ubuntu):
status: New → Confirmed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers