Since the LESS settings are minimal, I think it would make sense to move the settings into a fieldset or something within the admin/settings/performance page. This would eliminate yet another "configuration" menu item, plus (to me at least) makes logical sense.

Does anyone else agree/disagree?

CommentFileSizeAuthor
#6 move-settings-to-performance-1160174-6.patch2.14 KBbocaj

Comments

bocaj’s picture

bocaj commented

Also, I would be willing to work on a patch for this if others are interested in this...and corey.aufang is willing to make the change.

patrickharris’s picture

patrickharris commented

Yes - this makes a lot of sense to me.

davidfells81@gmail.com’s picture

davidfells81@gmail.com commented

Great idea!

Headache’s picture

Headache commented

Yes, would be nice.

timoti’s picture

timoti commented

makes sense to me.

bocaj’s picture

bocaj commented
Version: 6.x-3.x-dev » 6.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new move-settings-to-performance-1160174-6.patch2.14 KB

Since there were a number of people interested in this I decided to create a patch. This is rolled against 2.x-dev.

Also, the less.admin.inc file can be removed as it is no longer used.

You will find the settings at the bottom of admin/settings/performance.

corey.aufang’s picture

corey.aufang commented
Status: Needs review » Active

While the functionality on the settings page is similar to what is available @ admin/settings/performance, I find it is more helpful that the LESS settings are clearly available.

I have more often been extremely frustrated trying to find where a module's configuration options are than being annoyed by another admin/settings/* page.

jox’s picture

jox commented

I agree somewhat with #7.

I think minimalness must not necessarily be a reason to eliminate something. Usability should come first.

Maybe thematically it does not fit 100% as well. While the Less settings have impact on performance, it is not the primary subject. It is a penalty.

On the other hand it would be helpful to have an overview of all the performance affecting options.

But then it could/should also include the "Rebuild theme registry on every page." setting, "hidden" in the Apprearance configuration (admin/appearance/settings/theme). EDIT: This is even more misplaced, since it is a development setting that can not be found in the development section.

bocaj’s picture

bocaj commented

I do agree with #7 and #8. I personally just hate a configuration page that has a couple of configurations. Then, depending on the modules you enable, the Site Configuration menu easily becomes a mile long.

I agree that the performance page might not be the best place for this, since it really isn't a performance tool, I just don't think that it needs to have it's own page.

Thankfully Drupal 7 has the configuration section segmented a little more.

I think that the LESS settings could be better placed somewhere else, I'm just not sure exactly where :)

bocaj’s picture

bocaj commented
Status: Active » Closed (won't fix)

Now that I've thought about this some more, I don't know that there's a better place for the settings than on it's own page. Since Drupal 7 does a lot better job of breaking out the Configuration menu this probably isn't much of an issue anymore.

Thanks for all your input!