Set alt attribute. [#3099262]

Problem/Motivation

Alt attribute is not set, your module always owerride this parameter.

Proposed resolution

Set alt attribute.

Next, maintainer, please do not close issue after resolving it, change the status to fixed only, commit this path using the command from summary under this issue, give credits.

Comment	File	Size	Author
#2	media_pdf_thumbnail-set-alt-attribute-3099262-2.patch	903 bytes	lamp5

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

6 December 2019 at 11:24

lamp5 created an issue. See original summary.

Comment #2

lamp5

Polish

Rzeszow

CreditAttribution: lamp5 at abventor commented 6 December 2019 at 11:25

Status:

Active

» Needs review

File	Size
media_pdf_thumbnail-set-alt-attribute-3099262-2.patch	903 bytes

Comment #3

sgostanyan CreditAttribution: sgostanyan as a volunteer and commented 8 December 2019 at 11:19

Thanks for finding this issue. I added the alt attribute with a slightly different approach.
Regarding the HTML5 rules, the alt attribute is mandatory and its value can be blank but it should not be the filename. However, I still set the filename without extensions, hoping the filename describes the file content.
The fix is included to the new release, so this issue is fixed and can be closed (please update the module). It doesn't prevent people using old versions to find that issue.
Also for the next issues, please ensure to select the right version of the module (version 8.x.1.x is no longer supported)

Thanks for contributing