Version 7.x-2.21 and before applied the "Media browser" widget to "Image" field types in such a way where there was a pair of buttons next to the image thumbnail after uploading that said "REMOVE / EDIT" after upgrading to version 7.x-2.22 and above that has been removed.

  • Title/Alternative Text” fields used to not show directly on the blog editing page.
  • The “EDIT” button that used to display next to the “REMOVE” button is no longer there.
  • Modal no longer accessible via EDIT button
  • 80 character limit in alternative text field is being enforced.

We are running withing Drupal 7.67 and are up-to-date on all modules.

Change in UI Functionality

CommentFileSizeAuthor
#7 D7_media-3068756-7.patch780 bytesjoseph.olstad
media-bug.gif47.35 KBtbadaczewski

Comments

tbadaczewski created an issue. See original summary.

deadpoet’s picture

We're seeing the same issue after upgrading from 7.x-2.21 to 2.23, it's proving very troublesome. Is there any way to make the 80-character alt text limit optional, or adjust the limit?

[update] Ah it turns out you can control the max character limit, see comment #4 for a solution to this.

joseph.olstad’s picture

Hello @44Sunsets, please have a look at the release notes and revert the change that you think might be causing this, please let me know what the solution is and I will then investigate.7.x-2.22 release notes

Thank you for your assistance.

deadpoet’s picture

@joseph.oldstad Thank you for your reply. I just discovered that we can in fact control the image alt text and title max length via the image_alt_length and image_title_length variables. So that is a satisfactory resolution for our particular use case.

joseph.olstad’s picture

Component: Media Browser » Documentation
Category: Bug report » Support request

ok, could probably write a note in the README.txt about that if it's not already there, feel free to submit a patch for the module README.txt

tzt20’s picture

While we can alter it via the variables described in #4, I'd like to make a recommendation if it's included in the module as it is now that we move the max character length to 100 instead. That change aligns with most suggestions for ADA compliance (even though the max is 125).

joseph.olstad’s picture

StatusFileSize
new780 bytes

please review this patch with 100 as a default instead of 80

joseph.olstad’s picture

Component: Documentation » Code
Category: Support request » Bug report
Status: Active » Needs review

see patch

tzt20’s picture

Merci, Joseph. C'est bon. Patch works great. Hopefully, we can get this minor change added to next release.

  • joseph.olstad committed 0de2cb4 on 7.x-2.x
    Issue #3068756 by joseph.olstad, tbadaczewski, 44sunsets, tzt20:...

  • joseph.olstad committed e9d51e4 on 7.x-3.x
    Issue #3068756 by joseph.olstad, tbadaczewski, 44sunsets, tzt20:...

  • joseph.olstad committed eac0c2c on 7.x-4.x
    Issue #3068756 by joseph.olstad, tbadaczewski, 44sunsets, tzt20:...
joseph.olstad’s picture

Status: Needs review » Fixed

this is in the dev branch now, not sure when the next release will be. if you need this now, use a dev release.

joseph.olstad’s picture

Ok, I've tagged a release with this change , 7.x-2.24

Enjoy!

Thanks

tbadaczewski’s picture

Thank you!

Status: Fixed » Closed (fixed)

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