Link to RSS feed is missing near 'View all releases' link at the bottom of project pages.

Comments

dww’s picture

Assigned: Unassigned » dww

I'll take a look.

dww’s picture

Assigned: dww » Unassigned

This was removed/lost in commit 8ce8f410 (which doesn't reference an issue nid). In D6, this was building up items for theme('item_list') so we could just manually append the feed icon:

      'view_all_releases' => l(t('View all releases'), 'node/'. $node->nid .'/release') . theme('project_feed_icon', url('node/'. $node->nid .'/release/feed'), t('RSS feed of all releases')),

In D7, this stuff has been ported to theme('links'), and there's no way easy to append something like this anymore:

    'links' => array(
          '#theme' => 'links',
          '#links' => array(
            'view_all_releases' => array('title' => t('View all releases'), 'href' => 'node/' . $node->nid . '/release'),
          ),
    ),

Some possible options:

A) It'd be trivial to add the feed icon as another link, which would show up in a separate <li> but that's probably not what we want.

B) We could switch this back to an item_list, not #links.

C) We could "won't fix" this issue -- who cares? ;)

Thoughts?

drumm’s picture

Issue tags: -drupal.org D7

I'm tempted to won't fix this. Update status provides what people need for notification. Let's push to after launch and see if anyone asks.

drumm’s picture

Issue tags: +Drupal.org 7.1

Fixing tag

tvn’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

I think we can won't fix at this point. The link is present on the actual "Releases for X" page.