This is the bug/patch tracker for Rockbox. Click here for more information.
Quick links: Bugs · Patches · Rockbox frontpage
FS#9643 - some themes.rockbox changes..
Attached to Project:
Rockbox
Opened by Christoph Reiter (lazka) - Monday, 15 December 2008, 00:31 GMT+2
Last edited by Maurus Cuelenaere (mcuelenaere) - Thursday, 29 January 2009, 22:57 GMT+2
Opened by Christoph Reiter (lazka) - Monday, 15 December 2008, 00:31 GMT+2
Last edited by Maurus Cuelenaere (mcuelenaere) - Thursday, 29 January 2009, 22:57 GMT+2
|
Details- display the targets in a table like on the other pages
- make rewrite rules etc. better to have better fall back than 404 on wrong urls - add a upload link 
rb_themes_1.patch
(2.9 KiB)
trunk/PHP/tools.php | 14 +++++++++++--- trunk/PHP/.htaccess | 8 ++++++-- trunk/PHP/index.php | 9 ++++----- 3 files changed, 21 insertions(+), 10 deletions(-) |
This task depends upon
Closed by Maurus Cuelenaere (mcuelenaere)
Thursday, 29 January 2009, 22:57 GMT+2
Reason for closing: Accepted
Additional comments about closing: Thank you, parts of this are committed now.
Thursday, 29 January 2009, 22:57 GMT+2
Reason for closing: Accepted
Additional comments about closing: Thank you, parts of this are committed now.
I'm trying to rewrite the whole themes.rockbox site.
My php skills are a bit rusty.. so it will take a week or two.
Post here if you have any suggestions.
Read the README.
Admin panel isn't very nice (doesn't refresh) but more powerful.
I will continue if I get some feedback.
http://www.student.tugraz.at/christoph.reiter/files/tmp/themes.rockbox.zip
A few remarks:
- Theme site itself looks nice. But i dont think the bug-report link under the Themes is good here. We probably dont want Bugreports about user-themes in the tracker.
Perhaps it would be better to give to user a way to mark a theme as bad, which then can be inspected in the Admin panel.
- Admin-panel: I think this needs much more work to get it useable. It needes refreshes when you changes something. It should show the Theme when you click on the name and more. But it looks like a good start.
- I think one of the important things for this theme site is the automated checking when you upload a theme (maybe also rechecking in the admin interface). This should check all possible things (take a look at the upload/tools script in svn) and should also use checkwps to check the actual wps syntacs.
lets hope you continue you work, and we get a useable theme site fast :-)
greets
Lots of exams ahead, so, not much time.
1) Started
2) It gets the job done for now... the only good solution I see is using "ajax" and that's not worth the effort.
3) Noted
Thanks for your review Dominik.
not sure if I should add it here, but I did an update of Maurus Cuelenare's original theme site code.It works now for me for uploading and accepting themes. Some of the changes I made:
- added some more text in the index.php and put it in an additional file intro.php
- made links work in intro.php and added new links included for upload
- added new model handling in ini.php (model class)
- added some more automatic setup in ini.php
- made upload.php more generic
* support different unzip versions (with different output)
* support more zip mimetypes, browser can return both application/zip and application/x-zip-compressed.
* make filenames filesystem safe (instead of using html entities).
It can now be used to display and add themes.
What I would like to do:
- use a layout for displaying themes more like http://rbthemes.com/
- add support for sorting themes: alphabetically/popular/recently added/...
- add support for reporting problems (i.e. copyright infringment).
- add support in admin.php for removing accepted themes .
- make admin.php look a bit nicer.