Support from Acquia helps fund testing for Drupal Acquia logo

Comments

joseph.olstad’s picture

Version: 6.x-1.11 » 7.x-2.0
Assigned: Unassigned » joseph.olstad
Category: Support request » Feature request
Issue summary: View changes
Status: Active » Reviewed & tested by the community

all that is needed is to add download='' to the anchor, javascript built into drupal handles the rest.

While in there, not needed but might as well, change to use the drupal link function (the l function aka l() )

see commit message

joseph.olstad’s picture

For 7.x-1.x users, you can use the attached patch

For 7.x-2.x users, upgrade to the latest version of 7.x-2.x , I will tag a new release for this. or if you are on 7.x-2.x and dont want to upgrade you can patch with this patch

  • joseph.olstad committed a4704f6 on 7.x-1.x
    #1394316 force download link, and also add some translation for issue #...
joseph.olstad’s picture

see explanation by w3schools , the download attribute is part of html5

http://www.w3schools.com/tags/att_a_download.asp

joseph.olstad’s picture

Status: Reviewed & tested by the community » Fixed

  • joseph.olstad committed 1bbd383 on 7.x-2.x
    #1394316 add missing array declaration to fix warning messages
    

  • joseph.olstad committed 89a72fe on 7.x-1.x
    #1394316 add missing array declaration to fix warning messages 1.x...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

joseph.olstad’s picture

Status: Closed (fixed) » Needs review

This change might have caused a regression, looking into it now.