Should "Read More" tweak be in core for Drupal 7

SeanBannister - August 4, 2008 - 12:21
Project:Read More link (Drupal 6 and earlier)
Version:6.x-3.x-dev
Component:Miscellaneous
Category:feature request
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

I really think this should be in core for Drupal 7 and wanted to chat to anyone else who would be interested in getting this into core before we post it to the list of Drupal 7 patches.

#1

grateful_drupal_user - November 28, 2008 - 16:53

+1 from the author.

I use this module on every site I touch.

The current implementation of ed_readmore has some problems dealing with finding a proper insertion point for the inline read more <span> - so it this ends up in core, it'll need some work.

#2

SeanBannister - August 5, 2008 - 02:52

Well if it ends up in core we can play directly with the core code so we won't be stuck with module limitations.
I can't say I've looked at the code yet but I'm sure it'll be a completely different approach when modifying core.

#3

fletchgqc - August 16, 2008 - 15:43

The option to have Read more inline should be in core in some way shape or form.

#4

SeanBannister - August 18, 2008 - 00:03

Ok, there's an issue at http://drupal.org/node/16161 talking about getting it into core.

#5

thomas23@drupal.org - September 11, 2008 - 16:12

subscribe

#6

grateful_drupal_user - November 28, 2008 - 17:06

I've added a blurb mentioning this and #16161: Move Read More link to end of node content in the project description; maybe this will help get some traction in the effort to move this functionality into core. Thanks to Sean Bannister for the suggestion.

#7

kees@qrios - January 2, 2009 - 10:35

+1 for this to be a simple theme-dependant switch "Inline readmore links TRUE/FALSE" in the theme administration area.

#8

tobiass - January 23, 2009 - 12:34

supporting #7

#9

duvien - January 28, 2009 - 17:34

I think this is great and perhaps should be part of the core for D7 and filed under post-settings page. However, i also think only when you can define a different 'read more' text for other content-type teasers, it will be more useful.

Perhaps it should be in Views 2 as a field option?

#10

kees@qrios - January 30, 2009 - 09:31

I think a basic implementation in core with an overrride option for e.g. Views2 to have more advanced options would be perfect :).

#11

momper - February 4, 2009 - 08:43

+1

#12

Todd Nienkerk - February 13, 2009 - 12:53
Version:6.x-2.0-beta1» 6.x-3.x-dev

+1

Please review major changes to 6.x-3.x-dev. They could be useful for 7.x.

#13

funana - February 19, 2009 - 15:59

definitely! +++1 for that!

#14

drifter - March 20, 2009 - 09:32

+1 yes!

#15

Todd Nienkerk - March 22, 2009 - 01:12
Status:active» won't fix

I'm glad that everyone is excited to include this in core, but it's totally out of scope for this module. We can't address core issues here. But... You can talk about it here: #16161: Move Read More link to end of node content.

#16

danfinney - April 20, 2009 - 21:10

+2 (my wife thinks so too)

#17

Todd Nienkerk - April 20, 2009 - 22:35

danfinney: Thanks for +2, but please note that this issue won't get anything contributed to Drupal core. Please voice your support here: #16161: Move Read More link to end of node content.

#18

Cactii1 - April 25, 2009 - 07:42

+1

#19

dbabbage - August 3, 2009 - 10:03

Before you post a +1 to the issue that Todd Nienkerk linked to, please see: http://drupal.org/node/16161#comment-1838922

At this point, it would be highly counterproductive to "voice your support" there—unless you are doing so by attaching a patch that fixes the issue. Just a friendly suggestion. :)

#20

Gabriel Radic - September 27, 2009 - 19:33

A big +1 from me too. I use the module on every site, on various languages.

#21

Todd Nienkerk - September 29, 2009 - 15:18

Please don't +1 this issue. There's nothing that can be done about Drupal core here. Furthermore, Drupal 7 is in code freeze. If you want this feature added to Drupal core, please submit a patch to Drupal 8 after Drupal 7 is release.

#22

SeanBannister - September 30, 2009 - 05:21

Might still be able to get it in as a usability improvment even though we're in code freeze, especially if it's before October 15th because we're in a code slush

 
 

Drupal is a registered trademark of Dries Buytaert.