- Status Closed
- Percent Complete
- Task Type Patches
- Category User Interface → Themes
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#4871 - peakmeter margins and height
The following patch extends the possibility for the peakmeter WPS tag (%pm)
It enables the same margin functionality as it already exists for the progressbar.
So the new extended syntax is as follows:
%pm|<height>|<start>|<end>|
height is the height of the peakmeter; minimum height is 4 - default is 12.
start is the X position where the peakmeter starts
end is the X position where the peakmeter ends
All of the additional parameters are optional and may also be empty
e.g. %pm||100|200|
This produces a peakmeter starting at 100, ending at 200 with the default height of 12.
Of course the old format of just %pm is still possible… –> if you don’t use the new parameters, it works as it did =⇒ 100% compatible to current code!
The patch should work at all platforms where the peakmeter is already supported…
Please tell me if there are issues - I’ll fix them ASAP!
I hope the functionality will find its way to the CVS for v3.0 ;) !
2008-05-28 06:30
Reason for closing: Out of Date
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Viewports are now done - so this patch is no longer necessary…
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
Synced patch with today’s CVS (2006-03-26)
hi Matthias,
i just start trying to use your patch (great!)
it seems there is a bug:
my irock wps (carbon or white) displays covert art from X=1 to X=101, then on the right i programmed peakmeter from X=105 to X=217
⇒ when no cover art is available, i can see blinking dots (pixels) on the line between two peakmeter bars
can you help ?
precision: in place of cover art…
I also noticed that - it has nothing to do with cover art.
Even if you don’t add the album art patches it will show you
strange spots between the lines and all over the line
(does not matter which margins you choose - it’s always over the whole line)
And I have no idea where the spots come from.
I assume this is a bug of the internal API function peak_meter_draw which I indirectly use.
But I don’t know for sure…
O.K. the dots between the lines are not a bug - they are a feature :)
They show the scaling for the peakmeter bars.
But they have a bug:
They don’t take respect of the x offset as the peakmeter bars itself do.
I fixed this bug here:
So you need first to add the bugfix-patch and then the patch above
for the peakmeter margins…
you’re toooooooooo much !
great!
did you ear of that : http://www.rockbox.org/tracker/task/1644/edit
may it be possible that you integrate it in your patch, as it seems that the owner of the original did nothing since 01/2004…
The mentioned bugfix (task #4999) of the dots has found its way to CVS.
–> it’s no longer needed to apply it…
About the “redzone”: in principle I like the idea - the patch itself
would work for grayscale devices, but I think much more work should be
done to make it correctly work and also to make it work for coloured
devices.
And I assume that after v3.0 the whole peakmeter code needs to be
Here’s an update for CVS 20060405. I hope I didn’t break anything.
Another fix that uses the correct display width instead of LCD_WIDTH.
Thanks for synching it.
but I think, display→height and display→width should be used
instead of LCD_WIDTH and LCD_HEIGHT (this should make it possibe
to use it at remote screens, too)
I produced a new patch which changes this…
Ooops - I didn’t see your fix…
But you still missed a LCD_WIDTH, so mine is more complete ;)
Here’s an update for today’s CVS (20060424).
Another sync to today’s CVS…