Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Text resizing appears to not work on my site when using Firefox. So I spun up a Pantheon site to test the issue on a bare bones install.
This is a default D7 install with the Accessibility toolkit module added: http://dev-a11y-test.pantheonsite.io/
Text resizing appears to not work there either. tested on Firefox 48 on both OSX and Windows (using browserstack.com).
Animated GIF of issue: https://i.gyazo.com/82e6f01d7961e87fdcee3dd18aa75946.gif
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | text_resizing_broken_in-2786465-2.patch | 1.57 KB | Paul_T |
Comments
Comment #2
Paul_T CreditAttribution: Paul_T as a volunteer commentedNew to patching, so go easy if I did this wrong. :)
Fixed text resizing in Firefox on lines 28 and 31 by setting a default origin of top center, then using scale to resize the UI. Lines 12 and 15 fix an issue I noticed on all browsers, where on first click the UI is resized to 125 instead of 1.25. It was concatenating them as strings, so adding parseFloat avoids that issue.
Tested and works on IE11, Firefox 48, Chrome 52, Safari 9.1.2 all on OS X.
Comment #4
btopro CreditAttribution: btopro at Pennsylvania State University for ELMSLN commentedWell Paul_T, your TERRIBLE [sic] 1st patch is perfect and fixes issues unknown to me (didn't know this didn't work in FF) and known to me but elusive (this used to be inconsistent at best in Chrome)! Now it appears to be working in all browsers I tested w/ the same outcome in each. Congrats and thank you for making drupal & elmsln stronger :)
Comment #5
Paul_T CreditAttribution: Paul_T as a volunteer commentedThanks for reviewing and happy to help.