Hi,

I made a sub-module that exposes a DS field or/and a block that shows all footnotes at once.
So if you have multiple fields with footnotes, all footnotes will be displayed as one in that block/field

If someone could verify or add some extra functionality, we could apply it as patch.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

robin.ingelbrecht created an issue. See original summary.

robin.ingelbrecht’s picture

FileSize
3.38 KB
caseyb’s picture

Hi,

Thanks for putting this up. I have downloaded the file and had a few issues trying to install - eventually unzipped as rar and manually added to modules folder.

With your module installed, a new Footnotes block appeared and I added it to the footer region. I can see the heading 'Footnotes' at the bottom of the page and the footnote itself no longer shows under the entry field as expected but I cannot see the footnote content appearing in the new block. Do I need to create a new DS display to view the footnotes in the block?

Update - Just tried moving the block to the main content area and it showed the footnote content there but at the top above all the content.

I'm actually wanting to have the footnotes show in a block at the bottom of a panels page so this would be great if I can get it working. At the moment, if I add the block on the panels page, it doesn't show up and the footnotes content itself still shows with the original field.

Hope this feedback helps and happy to keep testing it.

Tested on:
Drupal 7.39
Footnotes 7.x-2.5
Adaptive Sub-theme

Thanks!!

caseyb’s picture

Update to previous comment - this block seems to work as expected when added to different regions :)

For my particular use case, it doesn't seem to work for panels pages via views.

LNakamura’s picture

FileSize
9.13 KB

Hi, Robin - thanks for creating this submodule!

We addressed a few Drupal coding standards issues in your code and I used that code base to create a patch.

Thanks again,
Lee

kepford’s picture

Status: Active » Reviewed & tested by the community

The patch from Lee in #5 applies cleanly.

DamienMcKenna’s picture

Status: Reviewed & tested by the community » Needs work

Some minor quibbles:

  • The new module has the package set to "EntityOne", it should probably be either removed so it fits with main Footnotes module in the "Other" package grouping, or "Fields" so it fits with the other custom field modules.
  • The CSS is indented by four spaces instead of two.
  • All references to "Display Suite" should be uppercased like that, and it probably would be worthwhile adding a link to https://www.drupal.org/project/ds on the message on the settings form.
LNakamura’s picture

FileSize
9.18 KB

Thanks for the reviews, Bob and Damien!

Damien - I've addressed the issues you pointed out and re-rolled the patch. Please take a look at it when you have a chance.

Thanks,
Lee

kepford’s picture

Status: Needs work » Needs review

Changes look good.

LNakamura’s picture

FileSize
1.89 KB

Adding interdiff...

LNakamura’s picture

Re-rolling for a tiny tweak to the css...

kepford’s picture

Status: Needs review » Reviewed & tested by the community

Ah, I missed the bracket spacing in the CSS. Looks good now Lee.

LNakamura’s picture

Thanks, Bob!

Ada Hernandez’s picture

Version: 7.x-2.5 » 8.x-2.x-dev

Need this functionality in Drupal 8, so We're working in this now (this is a proposal)
1. Deactivate the footnotes for each individual field.
2. Hash the foot note using a deterministic hash and affix it as a CSS class (& html id on the <a> tag).
3. Create a pseudo field to show in page footer.
4. Create a list in pseudo field by loading all fields on the page, hunting and collecting a list of all footnotes.
5. Use js to renumber the footnotes by using the deterministic hash.

yukare’s picture

Version: 8.x-2.x-dev » 7.x-2.x-dev

Do not change to drupal 8 while there is a patch for drupal 7 to commit.

  • LNakamura authored 0ab7dfd on 7.x-2.x
    Issue #2603538 by LNakamura, robin.ingelbrecht: All footnotes of node in...
yukare’s picture

Version: 7.x-2.x-dev » 8.x-2.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Commited to git. This is fixed n drupal 7 but we need a patch to drupal 8.

  • LNakamura authored 0ab7dfd on 7.x-3.x
    Issue #2603538 by LNakamura, robin.ingelbrecht: All footnotes of node in...
robin.ingelbrecht’s picture

If I can make time, I want to write a D8 port...

pifagor’s picture

Status: Patch (to be ported) » Fixed

Done

pifagor’s picture

Status: Fixed » Closed (fixed)