Currently the configuration of this module allows for the use of a custom #markup output for showing the results as Titles with Links. It also allows for "Full Content" or "Teaser" view modes to be used. Drupal 7 allows for a plethora of custom view modes to be used and I think this module would be better if those were supported. This patch gets rid of this custom #markup bit and uses only view modes. Open to discussion about keeping the "Titles" custom markup and also supporting all available view modes along with it.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | bean_relevant-use-view-modes-1513616-c5.patch | 2.27 KB | saltednut |
| bean-relevant-use-view-modes.patch | 2.32 KB | saltednut |
Comments
Comment #1
saltednutComment #2
seanberto commentedAssigning to bleedev so that reviewing this patch hits his queue. However, @brantwynn, would you consider rerolling to incorporate the option of just displaying titles as links? I like the flexibility that this patch adds with respect to supporting additional node view modes. But "title as link" is important to our client who's funding the project and I think it's good option.
Comment #3
saltednutI'm not opposed to it. I think we just need to make sure we clarify in the form what's going on with the dropdown select. Essentially the first option (title only) is a custom view while the other options are the actual core-driven view modes. I'll see about throwing up a variation on this patch that includes the "title only" markup output as an option.
Comment #4
seanberto commentedThanks Brantwynn. The whole interface does need better help text. I don't think that help text needs to explain where the different view modes are coming from or anything (heck, that would suggest that Drupal itself is usable... ;) ).
Looking forward to seeing your patch. Thanks for the help!
Comment #5
saltednutOkay, I didn't update the help text. Here's a patch that uses either the 'Linked Titles' in the #markup or puts together the results using view modes.
Comment #6
saltednutComment #7
bleedev commentedMade a few changes and committed this feature.