Rockbox.org home
Downloads
Release release
Dev builds dev builds
Extras extras
themes themes
Documentation
Manual manual
Wiki wiki
Device Status device status
Support
Forums forums
Mailing lists mailing lists
IRC IRC
Development
Bugs bugs
Patches patches
Dev Guide dev guide
Search



Donate

Rockbox Technical Forums


Login with username, password and session length
Home Help Search Staff List Login Register
News:

Thank You for your continued support and contributions!

+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Theming and Appearance Customization
| | |-+  ANNOUNCEMENT: Album Art now committed to svn
« previous next »
  • Print
Pages: [1] 2

Author Topic: ANNOUNCEMENT: Album Art now committed to svn  (Read 5395 times)

Offline stripwax

  • Developer
  • Member
  • *
  • Posts: 84
ANNOUNCEMENT: Album Art now committed to svn
« on: November 11, 2007, 02:02:20 PM »
http://www.rockbox.org/twiki/bin/view/Main/AlbumArt

 :)
Logged

Offline psycho_maniac

  • Member
  • *
  • Posts: 814
    • MyWebPage
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #1 on: November 11, 2007, 02:11:41 PM »
it seems that no themes will work because most of the AA themes that i saw on the wiki need the scrolling margins patch also.
Logged
Please SEARCH the wiki | Please read the Forum Guidelines | Please Read the Manual
I Own A Gigabeat F80

Offline NicolasP

  • Developer
  • Member
  • *
  • Posts: 195
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #2 on: November 11, 2007, 02:45:37 PM »
The scrolling margins patch (FS#2954) should be dropped in favour of linuxstb's simpler version (FS#8135), which has far better chances of being committed.
Logged

Offline yapper

  • Rockbox Expert
  • Member
  • *
  • Posts: 794
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #3 on: November 11, 2007, 03:14:55 PM »
Excellent - Album Art :)   Thanks to all who worked on this!
Logged
G2 iPod 20GB / Sansa c240 v1 + 2GB microSD / Sansa c250 v1 / Sansa e250 v1 + 8GB microSDHC / Sansa Fuze v2 + 32GB microSDHC

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #4 on: November 11, 2007, 03:35:54 PM »
Please respect the posting guidelines. The forums are for questions and discussion. The major changes page exists for a reason and posts like this simply dilute search results.
Logged

Offline Mikerman

  • Artist
  • Member
  • *
  • Posts: 721
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #5 on: November 12, 2007, 12:35:08 PM »
This is a wonderful development in the Rockbox experience--many thanks!  It offers the opportunity for a great deal of creativity and the need for creativity, especially for lcd display and lcd real estate limited devices such as the iRiver H1xx.  I look forward to people's creative efforts here!

:)

A question about the syntax:  with the syntax of

 Â    %Cl|x|y|[l|c|r][d|i|s]maxwidth|[t|c|b][d|i|s]maxheight|,

what do the variables [d|i|s] represent?  Perhaps I missed that--I couldn't find it at the cover art wiki page.

Also, am I correct that right now with committed (and unpatched) code, it is not possible to set left (or right) margins, for scrolling text or not, so as, for example, to have album information appear to the right of the album art?  I assume that to be the case (and did not see any set margin commands at the WPS commands page); I take it this is the topic being considered with the patches listed above, but I just wanted to make sure.

Finally, I wonder if consideration has been given to the possibility of alternating WPS displays, such as album art for 5-10 seconds followed by album text information for 5-10 seconds.  Although, thinking about it some, I guess that this currently could be accomplished by using conditionals and alternating sublines throughout current WPS code, under which if there was album art, the album art would display for a period of time during which the current listed text/variables would not appear.  An interesting possibility, and the makings for many hours of challenging fun!
« Last Edit: November 12, 2007, 04:39:35 PM by Mikerman »
Logged

Offline Mikerman

  • Artist
  • Member
  • *
  • Posts: 721
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #6 on: November 13, 2007, 12:22:33 AM »
Well, and now as of later today, we have a left margin setting command for scrolling text ...
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #7 on: November 13, 2007, 12:25:24 AM »
Considering I already mentioned the guidelines earlier in this thread, I thought it would be unlikely that a second guideline would be broken in it.

Please be so kind as to do what you agreed to do and read them.
Logged

Offline Mikerman

  • Artist
  • Member
  • *
  • Posts: 721
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #8 on: November 13, 2007, 09:58:12 AM »
Sorry, dude, I don't get you:  are you saying that this thread is not for questions and discussion?

Quote
The forums are for questions and discussion.

My second post simply modifies my first to clarify it as to the development, so that people don't reply that the margin can indeed be set  or otherwise think that the margin can't be set.  Perhaps I should have edited my first post, but I sometimes find that confusing as a reader who already has read through the posts.

Please feel free to pm me with any issue, so as to avoid diluting the threads.  Am looking forward to responses to my inquiries in the original post--thanks!
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #9 on: November 13, 2007, 10:01:43 AM »
Whether you find it confusing or not to edit the first post, the rules explicitly say not to post twice in a row.

So, while yes, your post is of a topic that would be better suited to a new thread (how is anyone to know from the topic of this thread that it contains questions and possibly answers about album art syntax: it would've been better to start a new thread) it isn't strictly off-topic, and what I was commenting on was most certainly the fact that you posted twice in a row rather than using edit, as we frequently remind people of the guidelines that relates to this. Unfortunately, you don't get to decide to ignore the guidelines because you think they make things more confusing.

Editing your existing post when nobody has responded yet will still mark the thread with a [new] icon for other users, so they know it has changed.
« Last Edit: November 13, 2007, 10:09:15 AM by Llorean »
Logged

Offline Mikerman

  • Artist
  • Member
  • *
  • Posts: 721
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #10 on: November 13, 2007, 10:10:45 AM »
Thank you for the helpful comments and I certainly will re-edit in the future, so as to avoid causing issue (altho in my humble belief, it still causes confusion as to matters and what has been changed--and I did not proceed so as to intentionally ignore the rules).

Again, and getting back to the topic (which I thought was broad enough to cover my inquiry), am looking forward to responses to my original post, for the development of WPSs with this great feature!  (I also am happy to start a separate thread to cover them, should that be preferred.)
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #11 on: November 13, 2007, 10:13:01 AM »
In regard to those extra letters, I have no clue, and wondered about them myself, but I think they were accidentally added or left on the page from the unsupported patch, as they aren't there any more.
Logged

Offline pixelma

  • Rockbox Expert
  • Member
  • *
  • Posts: 645
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #12 on: November 14, 2007, 03:24:00 AM »
I asked Nico_P in IRC what the [d|i|s] means and he said it had to do with the scaling that was in the original patch. As a result he removed it from the wiki because this is not implemented in the official version yet which the wiki is supposed to describe.
Logged

Offline tdtooke

  • Member
  • *
  • Posts: 151
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #13 on: November 14, 2007, 05:51:40 AM »
Quote from: NicolasP on November 11, 2007, 02:45:37 PM
The scrolling margins patch (FS#2954) should be dropped in favour of linuxstb's simpler version (FS#8135), which has far better chances of being committed.
I'm in 100% agreement with you, I just don't want 2954 closed.  As it stands now (as of 10 minutes before this post) it extends the recent %m commit to function as you committed it and also as the old scroll-margins worked.  For the time being I think it fills a void as there will always be people who want to define a right margin.  And on a happier note I must say I'm completely thrilled with your latest commits!  Amiconn commiting ipod brightness control was pretty cool too!
Logged

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: ANNOUNCEMENT: Album Art now committed to svn
« Reply #14 on: November 14, 2007, 02:51:14 PM »
The whole point of the patch tracker is for patches seeking inclusion in Rockbox.

If a patch will never, ever be included in Rockbox, why should it be left open to encourage people to work on something that is unwanted, rather than closed with the (somewhat slim) hope that the people wanting that feature will instead go about doing it a way that can be included?

To me at least, leaving a patch open on the tracker that we don't want specifically makes the tracker a means of supporting unsupported builds. It encourages people to post any patch, even "NoDo patches", that users may want to use, so that they can be developed for these unsupported builds, and any patch that is closed as "rejected" can become a controversy because of inconsistent policies on when a patch is rejected if it's not simply "we have decided not to include it in the official build."

For consistency's sake, if a patch isn't to be included, it should be rejected. It's the Rockbox tracker, not the "Everyone's build's patch tracker."
« Last Edit: November 14, 2007, 02:54:12 PM by Llorean »
Logged

  • Print
Pages: [1] 2
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Theming and Appearance Customization
| | |-+  ANNOUNCEMENT: Album Art now committed to svn
 

  • SMF 2.0.17 | SMF © 2019, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.12 seconds with 14 queries.