FS#5458 - Corrections to screen shots in the Ondio manual

Attached to Project: Rockbox
Opened by Michael DiFebbo (medifebbo) - Tuesday, 30 May 2006, 00:28 GMT
Last edited by Peter D'Hoye (petur) - Thursday, 01 June 2006, 09:16 GMT
Task Type Bugs
Category Manual
Status Closed
Assigned To No-one
Operating System Archos Ondio
Severity Low
Priority Normal
Reported Version
Due in Version Version 3.0
Due Date Undecided
Percent Complete 100%
Votes 0
Private No


These errors and suggestions noted by amiconn on IRC on Friday, 26 May 2006:

(1) A number of screenshots shows a button bar, i.e. they were taken from recorder. The Ondio has no button bar b/c there are no soft buttons
(2) Many many screenshots also show the clock in the status bar. The Ondio has no clock, and hence no clock display. This is also true for the Ondio FM
(3) The 'bounce' screenshot is also the recorder version, with the clock in the background
This task depends upon

Closed by  Dominik Riebeling (bluebrother)
Sunday, 04 June 2006, 11:33 GMT
Reason for closing:  Fixed
Additional comments about closing:  fixed by adding the ability to have device specific screenshots. If there are still wrong screenshots used for the ondio please report.
Comment by Mark Bright (Redbreva) - Tuesday, 30 May 2006, 18:41 GMT

basic problem is that there are several obvious visual differences between the Archos Recorder and Ondio screenshots... Button Bars & RTC on the Recorder for example, and the current method of patching screenshots to the manual is based purely on the screen resolution... this is just one case where that method will fail. Others include H340 & iPod Photo, H140 & iPod 4G Greyscale etc.

To overcome this problem, I propose changing from resolution to platform based naming of the images, I realise that there will inevitably be duplication of some screens in CVS, but at least the manuals will reflect the target accurately - important for new users wanting to get familier with rockbox.

I have attached a set of screenshots (Taken with an Ondio FM) and the are all named according to: ss-foo-ondio.png

Also attached is a patch to amment the platform file for Ondio to match this change...
Comment by Dominik Riebeling (bluebrother) - Friday, 02 June 2006, 18:19 GMT
another option would be to expand the screenshot macro to first search for a device specific screenshot, and if none is found look for a resolution specific one. So we wouldn't have much duplicates in cvs and also automatically use the resolution screenshots if the device shot hasn't been commited -- I think it's better to have a similar screenshot in the manual than nont. (Similar is already done for displaying the "image not found" message if images are missing).

The only topic we should agree about would then be how to name device specific screenshots. I'd suggest using a <resolution>-<device> for device specific screenshots. Comments?