Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
As suggested by Factory Joe.
You can auto-disable certain blocks during high server load to reduce stress by enabling the "throttle" option. The option is available if the throttle module is enabled.
Comment | File | Size | Author |
---|---|---|---|
#11 | block-module-help-2.patch | 8.13 KB | keith.smith |
#10 | block-module-help.patch | 8.01 KB | keith.smith |
#4 | block.module.patch | 7.52 KB | gaele |
#2 | block.module.patch | 6.39 KB | gaele |
block.module.patch | 1.96 KB | gaele | |
Comments
Comment #1
Gábor Hojtsy- So is there no value in linking to the Throttle config page?
- While we are here, why not break up this long help text to t()'s per paragraph, as used elsewhere? (And as it is much easier for translators).
Comment #2
gaele CreditAttribution: gaele commentedAt your service ;-)
Comment #3
keith.smith CreditAttribution: keith.smith commentedNote that there is an older issue for block module help text, at http://drupal.org/node/189865. But, it has no patch so I'd say just close that one as a duplicate if you want to continue here.
I haven't looked at this help text in detail, but this may also be an opportunity to add text addressing the new drag-and-drop ordering functionality for the blocks page.
I just happened to notice this existing line at the bottom of the patch:
When we say "make it active and give it a place on the page", this does mean enable and position, right? It may be out of scope here, but this line looks like it could use some tweaking.
Comment #4
gaele CreditAttribution: gaele commentedImplemented Keith's suggestions (except for the drag-and-drop description).
Comment #5
keith.smith CreditAttribution: keith.smith commentedComment #6
Gábor Hojtsy- Thanks for breaking the big stuff up :) Although our code style requires that dots stick to the literal strings, so eg.
'<li>'.
instead of'<li>' .
-
position it on the page using <a href="@overview">blocks</a>
? Using blocks?Comment #7
gaele CreditAttribution: gaele commented- Adjusted to conform to code style.
- "Once you have created this block you must enable it and position it on your web pages using the blocks page." (which is consistent with the description on the blocks page itself.)
Comment #8
gaele CreditAttribution: gaele commentedComment #9
keith.smith CreditAttribution: keith.smith commentedNo patch attached. :(
Comment #10
keith.smith CreditAttribution: keith.smith commentedI started putting the drag-and-drop stuff in per #4, and wound up with a wholescale rewrite.
The attached patch:
* reworks the main help text, plus the text headers over the main block pages.
* It does handle the drag-and-drop stuff, though not every eventuality (like non-js browsers, etc.)
* I put the throttle-specific help on the admin/build/block page inside a conditional so that it only appears if throttle.module exists.
* This patch uses gaele's work from previous patches to split the help text into more manageable strings for translation.
This is a major rewrite, so I'm going to expand the scope of this issue to cover the help text as well.
Comment #11
keith.smith CreditAttribution: keith.smith commentedAdded a note about remembering to click Save blocks after reordering block items.
Comment #12
Gábor HojtsyReading it through, it looks like a huge improvement so I committed this one. I am not sure how non-JS browsers can be accomodated here, but if you have good ideas, feel free to reopen the issue. The "when allowed by an administrator, specific blocks.." part also looked a bit strange, but I tried to come up with better wording myself, and was unable to. So feel free to submit a small improvement for that too in an issue follow up on this same issue, if you have better ideas. Thanks.
Comment #13
(not verified) CreditAttribution: commentedAutomatically closed -- issue fixed for two weeks with no activity.