Closed (fixed)
Project:
Drupal core
Version:
7.x-dev
Component:
node system
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
27 Jun 2009 at 18:50 UTC
Updated:
3 Jan 2014 at 00:29 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
eigentor commentedComment #2
eigentor commentedThis is the actual text:
"The maximum number of characters used in the trimmed version of a post. Drupal will use this setting to determine at which offset long posts should be trimmed. The trimmed version of a post is typically used as a teaser when displaying the post on the main page, in XML feeds, etc. To disable teasers, set to 'Unlimited' . Note that this setting will only affect new or updated content and will not affect existing teasers."
Comment #3
eigentor commentedHere is a patch that changes the text. Looks like showed in the screenshot now.
Comment #4
eigentor commentedAlso found the same text in two other places: modules/node.admin.inc and modules/aggregator/aggregator.preprocessor.inc
Since this is basically the same issue, I changed the description there as well. I hope it is O.K. not opening issues for those separately.
So first node.admin.inc here
Comment #5
eigentor commentedAnd also for aggregator.preprocessor.inc
Comment #6
keith.smith commentedI like these text changes.
For ease of application, though, can we combine the three separate patches into one unified diff?
Comment #7
eigentor commentedI guess so. Was a bit uncertain if this was best practice. Not so experienced in patching.
But thinking about it, much larger and touching more files patches have been committed... Will create one patch for all of this.
Comment #8
eigentor commentedRolled the patches into one.
As the description appears to be gone from node.admin.inc it is only for
aggregator.processor.inc and content_types.inc
Comment #9
keith.smith commentedOne small thing that I just now noticed: "This setting will only affect new or updated content: existing posts keep their teaser length."
IMO, that colon should be a semi-colon. Other than that, I'd happily RTBC this as I think these are great changes.
Comment #10
tic2000 commentedThe teaser length affects new and old content now (that's why
This setting will only affect new or updated content: existing posts keep their teaser length.was removed from content_types.inc) so you shouldn't add it back.Comment #11
Bojhan commentedTried something, we shouldn't use post.
Comment #12
stborchert@Bojhan: sounds much better.
Slight modification:
"The maximum number of characters used in the trimmed version of newly added content.".Comment #13
tic2000 commentedI don't know for aggregator, but for node content the filed is actually a summary OR trimmed version. So I think this should be clear for the user and I don't know if it is right now.
Comment #14
eigentor commented@stBorchert If the teaser now affects new and old content now, we don't need to mention "newly added".
Comment #15
Bojhan commented@tic2000 Can you suggest a solution? Remember that we don't have a teaser splitter anymore so this only applies to automaticly generated teasers (as far as I know)
Comment #16
tic2000 commented@Bojhan I don't know. "Maybe The maximum number of characters used in the trimmed version of content. Used if a summary is not provided."?
Comment #17
eigentor commentedI wonder if we really got to mention that or if we confuse the user. If a user is creating a teaser manually, he / she should expect it to be exactly the length that is entered there? Thus this would not be needed.
Who will read this description? Someone who first has to find out that you can influence teaser length at all, which might be surprising. Then I would him confuse the least possible.
If it has to be.... How about The maximum number of characters used in the trimmed version of content that is automatically created.
Comment #18
tic2000 commentedIf the usability guys (I don't know who they are) think the text is enough then it's enough for me too. I just want to be sure it's enough.
Comment #19
Bojhan commentedI don't think it makes much sense to provide an "in case" text. Especially because summary might not always be there.
Comment #20
eigentor commentedBojhan so with or without the mentioning of hand-created teasers?
In a radical lust of the shorter the better #11 has my vote.
Comment #21
yoroy commentedI vote for version in #11 as well.
Comment #22
tic2000 commentedWho wants to RTBC?
I'm not an Usability expert and I'm not a native English speaker so I can't do it.
Comment #23
seutje commentedimo, not adding a mention that this only applies to automatically generated teasers will confuse users in the long run
how about "The maximum number of characters used in the trimmed version of content for which a trimmed version was not provided."
or "The maximum number of characters used in the trimmed version of content unless a trimmed version was provided."
or even "The maximum number of characters used in the automatically generated trimmed version of content."
¢2
Comment #24
catch#11 looks fine to me too.
Comment #25
tic2000 commented@seutje
Keep in mind that the text is on an administration page (content type creation/editing) and not on node creation page.
Comment #26
dries commentedCommitted to CVS HEAD. Thanks.
Comment #27
moshe weitzman commented"trimmed version" should be "summary version" IMO. Summary is the word used by field API and thus used for the body field. Not a big deal.
Comment #28
tic2000 commentedIn this case trimmed is the perfect word. The field is a summary or trimmed text. This text is for the configuration of the trim, we don't touch the summary if any is provided.
Comment #29
eigentor commentedmoshe "teaser" has vanished from UI text completely and been replaced by "trimmed version". So it is a matter of consequent use.
I've also seen teaser in some places, and as it is the word we commonly use when talking about it, there is probably no way of getting rid of it... But in Core interfaces, we must decide on one direction.