Gutenberg teasers are not linked (or is it just me?)

dman - August 9, 2007 - 13:06
Project:Gutenberg
Version:5.x-1.1
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

I guess this was just an oversight, but I found that the teasers produced by Gutenberg didn't have the hrefs around the titles at all. It sorta cripples everything :)

I may be missing something if nobody has noticed this before me ... This patch is against what I pulled down as the DRUPAL-5 release.

However, this was an incredibly clever idea as a theme port, I'm really impressed you could pull it off. It really saved the day when a photographer wanted a (freebie) portfolio site, and I knew the available Drupal themes were all too bloggy.

Really really great achievement, I've pointed a few users at this project already.

I'm not convinced you needed to strip out ALL the Drupal class baggage, I'd have (inefficiently) doubled them up, as I found a few layout tweaks that I'd modularised could no longer even count on blocks being called blocks or menus menus. That was a bit mean.
I say yay, absorb the best from other templates ... but don't branch away so far that the Drupal conventions are broken in turn.

AttachmentSize
gutenberg_teaser_links.patch575 bytes

#1

faunapolis - August 27, 2007 - 00:53

thanks for the patch, great project... this opens up a great deal of options for drupal!!!

#2

jt_jones - January 29, 2008 - 20:21

another thanks for the patch. I'm working on a new site I was getting worried since a Gutenberg theme had already been approved by the powers that be.

 
 

Drupal is a registered trademark of Dries Buytaert.