Theme Editor Feedback

[mc2739]

Thanks for the explanation!  I had seen that setting on the Misc. tab but I didn't realize that is what it did.

[Bieber]

Yeah, I know it's not entirely intuitive to have it there, but it should at least be useful when creating the SBS document, so you can see how it will look when rendered for the various screens.  Hopefully the auto-setting will confuse people a little less

[ixo4vita]

Pair of ideas, in an attempt to make it more WYSIWYG:
1]
how feasible would it be to code a wizard to build bitmap strip strings for playback/repeat etc tags.

the wizard could be a form questionnaire asking
width/height, and total parts (for the custom ones like volume/battery);
tag to apply it to
then according to answers provided, load a bitmap, and have it show in parts what each part will correspond to (preview function) and the tag string ready for copy/paste

the export project function would also package the bitmap in the appropriate folder when it zips.


2]
was the intention for the parser window something along the lines of having the parser pull descriptive tag definitions from a database-type file, the parser would show output similar to --debugwps from the simulator? if not, then should it?

[JdGordon]

Pair of ideas, in an attempt to make it more WYSIWYG:
1]
how feasible would it be to code a wizard to build bitmap strip strings for playback/repeat etc tags.

slight side issue, are you asking because %?mp<%xd(Aa)|%xd(Ab)|%xd(Ac)|%xd(Ad)|...> is too complicated? not understandable? error prone? long winded?

[ixo4vita]

Pair of ideas, in an attempt to make it more WYSIWYG:
1]
how feasible would it be to code a wizard to build bitmap strip strings for playback/repeat etc tags.

slight side issue, are you asking because %?mp<%xd(Aa)|%xd(Ab)|%xd(Ac)|%xd(Ad)|...> is too complicated? not understandable? error prone? long winded?

Just saying the editor should have an ability to show where these parts go, and as well be able to display it into the preview window as it currently doesn't load the images listed in %xl tags unless within a project

If you are pointing out the fact i am still working on the theme with that tag usage - i've already got it straight, but changed other things.

[JdGordon]

no, never mind. I was hoping you would say yes that forma to display the images is hard to understand (initially)

[ew]

Congratulations on the great work.

One minor idea, how about showing the x,y position while hovering (or clicking) in the skin preview area?

[bluebrother]

I've just updated the binaries of the Theme Editor I'm providing (at http://www.alice-dsl.net/dominik.riebeling/rockbox/themeeditor/) and for the first time added a NSIS based installer for Windows. This is my first attempt at an installer so please expect it to be anything but final. However, there are some things I'm not completely sure about:

- should the installer offer creating a desktop icon?
- should the installer download the fonts? Right now it does (if the option is enabled), but given that the Theme Editor itself can download the fonts too I'm not sure if it makes sense having an option to download the fonts in the installer at all.
- anything else important that's missing?
- and last: is there interest in having an installer for Windows at all? If yes please test and let me know of any issues with the installer

[[Saint]]

bluebrother,

A Start Menu entry would be nice, I have a habit of automatically selecting "no" for desktop shortcuts...(I hate them), and I was surprised to find I had to go digging to launch the app as it wasn't in my Start Menu.

re: Downloading the Font Pack, no, I don't believe the installer should do this personally.


[St.]

EDIT:

*Gah!* Ignore the part about the Strat Menu shortcut...I somehow read it as "Desktop Shortcut" in the installer :/

[Bieber]

Congratulations on the great work.

One minor idea, how about showing the x,y position while hovering (or clicking) in the skin preview area?

I think I actually intended to do that before, and it fell by the wayside.  I'll probably throw that in sometime next week after pencils-down (that's Monday).  Right now I'm trying to get in as much as I can on the GUI editing front before Monday, and then after that I'll devote some time to the little features that I've wanted to add along the way, I think.

Edit - Actually, I just implemented that.  I've been working on some graphical editing features: at the moment, you should be able to move and resize viewports, images, album art, and progress bars (although obviously resizing an image won't have any permanent effect).  Clicking the little gear button at the left of the preview window will update the code with the changes you've made, and clicking the arrow button will undo any changes you've made.  The coordinates should show up in the middle of the skin preview panel at the bottom.  I'd love to hear feedback on any of this, right now I'm trying to decide how to work in the capability to add new viewports/images/progress bars...

[expos1994]

Theme editor is coming along nicely.  Great work.

Here's my feature suggestion:

Something to allow toggling different conditions.  Maybe a control panel where the user can set the battery level, the volume, the play/rw/ff/stop, etc.  Then based on that setting the preview window will update accordingly.

Another way to do this would be to change the preview window when the user hovers over the code.

IE:

Code: [Select]

%?mp<||%xd(Pa)|%xd(Pb)|%xd(Pc)>

Hovering over the %xd(Pa) part of this statement would show the Pa image in the preview window.

A toggling control panel would probably be the easiest to implement and also use. 
Another thing that would be nice would be a few text boxes to set the various tags (artist, album, track title, etc)  That way the user could see how different strings would be displayed and perhaps test the scrolling.

Keep up the great work!

[[Saint]]

Here's my feature suggestion:

Something to allow toggling different conditions.  Maybe a control panel where the user can set the battery level, the volume, the play/rw/ff/stop, etc.  Then based on that setting the preview window will update accordingly.

You might want to go exploring in the menu of the themeeditor, all this is already implemented.
Try navigation to "View - Device Configuration" (or simply press "Ctrl+D").

Another thing that would be nice would be a few text boxes to set the various tags (artist, album, track title, etc)  That way the user could see how different strings would be displayed and perhaps test the scrolling.

Again, this is in the Device Configuration menu (Ctrl+D) under the tab "ID3 Info".

Scrolling and anything else that requires a timeout can can be viewed by using the "Timer" function in "View - Timer" (or simply press Ctrl+T).


Hope this helps,


[St.]

[expos1994]

Don't I feel stupid!

You're right, this is all implemented... and implemented well.

[ew]

Thanks for implementing the x,y positional indicator.  It is great.

One further suggestion and a possible bug:

• Suggestion - could there be an option for word wrap (for display of) of long lines (in the editor)
• Could there be a bug in the way a progress bar is displayed in the preview window?  Possibly between 27667 and 27700. 
  
  The following was displayed as I expected in 27667 and the simulator
  

  %Vl(Info_Menu_NoAA,0,199,168,2,0)
  %?mp<|%pb(8,-,-,-,linebarpb.bmp)>

  
  In 27700 the progress bar is justified to the left margin
  
  Edit:Could it be that x&y got somehow reversed?

Thanks again for your work.

[ixo4vita]

It feels like the last two builds have started to double up the directory names when loading images within projects? i've tried a pair of projects and get the projectname/projectname/wps/wps/projectname/imagename.bmp is not found error