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
| | |-+  Hide "advanced" theme options?

Poll

Should we hide advanced theme settings? (defaulting to yes hide)

yes
11 (68.8%)
no
5 (31.3%)

Total Members Voted: 15

« previous next »
  • Print
Pages: [1] 2

Author Topic: Hide "advanced" theme options?  (Read 6221 times)

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Hide "advanced" theme options?
« on: January 21, 2010, 02:47:49 PM »
This is a half idea to maybe fix some of the confusion with themes and settings.
If this option is added and enabled (so these settings are hidden) the only options under the theme settings menu would be load a .cfg. You would not be able to set colours, load a sbs or wps manually (from the browser or settings), ditto backdrops, etc.
This would completely eliminate confusion because all the settings which make up a theme would be hidden from the user.

With the option disabled it would be the same as now, complete with all the current confusion.
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Hide "advanced" theme options?
« Reply #1 on: January 21, 2010, 03:31:29 PM »
Where's the third option, "remove most of those settings permanently"?

I think loading .sbs, .wps, and .cfg manually are okay, but most of the extra settings themselves can be dropped.
Logged

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Hide "advanced" theme options?
« Reply #2 on: January 21, 2010, 03:40:53 PM »
I've tried having this discussion also, it ended pretty aggressively last time and I gave up.

The civil argument is it forces users who dont want fancy but do want colours to create a .sbs to do it.

IMO loading backdrop, sbs, wps, icons file (not in the menu though) are all useless and just add clutter. Ditto for colours and "ui viewport" (also not a menu-ed setting)
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Hide "advanced" theme options?
« Reply #3 on: January 21, 2010, 03:45:48 PM »
I think loading the individual files via context menu (not automatic behaviour via associated file types) might be nice for people who really want to mix and match themes.

But with the complexity of themes increasing vastly it might make sense to shunt everything to actual theme files.

Is there really that many people who say "oh, darn, man, I really, really need the screen to be purple with cyan text right now!" who couldn't set it up in advance.

I guess what I'm trying to say is I don't really understand what the user who needs a colour setting expects? Do they change colours often, or is it something else?
Logged

Offline AlexP

  • Global Moderator
  • Member
  • *
  • Posts: 3688
  • ex-BigBambi
Re: Hide "advanced" theme options?
« Reply #4 on: January 21, 2010, 03:47:07 PM »
I agree - it is time to move to loading pre-done themes (in the large part).
Logged
H140, F60, S120, e260, c240, Clip, Fuze v2, Connect, MP170, Meizu M3, Nano 1G, Android

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Hide "advanced" theme options?
« Reply #5 on: January 21, 2010, 03:52:03 PM »
I mean I wouldn't object to the possibility of a "customize this theme" context menu option later that loads a plugin capable of tweaking theme settings for the very advanced (including GUI preview of positioning viewports, basic image editing, font preview, etc) or anything, but that's head-in-the-clouds wishes.

I don't think the normal settings menu needs much of anything beyond a basic "load theme" especially given that RBUtil can help most users acquire themes very easily now.
Logged

Offline karashata

  • Rockbox Expert
  • Member
  • *
  • Posts: 603
  • Just a fluffy dragon with too much time on my paws
    • Alex Vanderpol (karashata) on Twitter
Re: Hide "advanced" theme options?
« Reply #6 on: January 21, 2010, 04:17:56 PM »
I'm so far agreeing to the idea of removing most of the theme settings from the menus, with the themes getting as complex as they are it would make more sense to let the config file do most of the work. Leaving the options to load a different .sbs or a different font along with the main "load theme" option though would be a good idea in case someone prefers a different status bar than the one used by the theme (rather than the theme's or none at all...), or wants to use a different font (though we already know the issues with that one, with themes for the most part being specifcally built around a certain size font...)
Logged
Welp, my iriver H10 20GB is out of commission again.

Offline saratoga

  • Developer
  • Member
  • *
  • Posts: 9004
Re: Hide "advanced" theme options?
« Reply #7 on: January 21, 2010, 04:38:34 PM »
I always thought editing themes on device made very little sense and would support removing some or all of those options, at least in principle.  We can better use the RAM for other things.
Logged

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Hide "advanced" theme options?
« Reply #8 on: January 21, 2010, 07:57:34 PM »
I've changed my mind, removing the menu items doesnt fix anything. the current system is broken and I want to fix the actual issue, which is setting priority.

refer to mailing list thread for details.
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline GodEater

  • Member
  • *
  • Posts: 2829
Re: Hide "advanced" theme options?
« Reply #9 on: January 22, 2010, 02:22:36 AM »
Quote from: JdGordon on January 21, 2010, 07:57:34 PM
I've changed my mind, removing the menu items doesnt fix anything.

It does, as saratoga has pointed out, get us back valuable RAM though...
Logged

Read The Manual Please

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Hide "advanced" theme options?
« Reply #10 on: January 22, 2010, 02:43:28 AM »
numbers from e200 with svn r24254

removing the entire theme menu from the build reclaims 1200bytes, (maybe another 400 for the strings... MAX)
removing the colour picker screen reclaims a further 2400 bytes

compare that with:
removing tagcache: 46KB
removing dircache + tagcache: 56KB
quickscreen: 4KB
album art: 5KB
morse input: 1200bytes
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline Llorean

  • Member
  • *
  • Posts: 12931
Re: Hide "advanced" theme options?
« Reply #11 on: January 22, 2010, 03:23:40 AM »
What that tells me is "remove theme settings AND quickscreen (to be replaced with a shortcut to a folder of cfg files :P ) for twice the savings!"
Logged

Offline karashata

  • Rockbox Expert
  • Member
  • *
  • Posts: 603
  • Just a fluffy dragon with too much time on my paws
    • Alex Vanderpol (karashata) on Twitter
Re: Hide "advanced" theme options?
« Reply #12 on: January 22, 2010, 12:26:03 PM »
So... ultimately it's not really worth it..? (I certainly hope no one's seriously considering removing the quickscreen, it's got its uses... It's faster than the WPS context menu, if only by a little bit...)
Logged
Welp, my iriver H10 20GB is out of commission again.

Offline JdGordon

  • Member
  • *
  • Posts: 1817
  • Constantly breaking stuff
Re: Hide "advanced" theme options?
« Reply #13 on: January 22, 2010, 12:56:40 PM »
yes, that's my point. being careful about ram is always good, but we are talking about the compressed audio buffer here. 100KB means 6 seconds of 128kbps audio.
Logged


Using PMs to annoy devs about bugs/patches is not a good way to have the issue looked at.

Offline karashata

  • Rockbox Expert
  • Member
  • *
  • Posts: 603
  • Just a fluffy dragon with too much time on my paws
    • Alex Vanderpol (karashata) on Twitter
Re: Hide "advanced" theme options?
« Reply #14 on: January 22, 2010, 01:04:14 PM »
Well... If you were to remove some the menu settings, at the least you'd clear up some potential clutter that's not really that useful anymore...

I still think it might be good to reduce the theme settings to a "load theme" option, as well as a font selector and a custom status bar selector, since that's really all I can see anyone really needing... (If anyone *really* needed to tweak any of the other settings, they could do so in the config file for whatever theme they want to tweak...)
Logged
Welp, my iriver H10 20GB is out of commission again.

  • Print
Pages: [1] 2
« previous next »
+  Rockbox Technical Forums
|-+  Support and General Use
| |-+  Theming and Appearance Customization
| | |-+  Hide "advanced" theme options?
 

  • SMF 2.0.18 | SMF © 2021, Simple Machines
  • Rockbox Privacy Policy
  • XHTML
  • RSS
  • WAP2

Page created in 0.14 seconds with 24 queries.