We are moving to Git Issues for bug tracking in future releases. During transition, content will be in both tools. If you'd like to file a new bug, please create an issue.

Bug 6901 - Fontsizer Tooltips hardcoded in English Language
Fontsizer Tooltips hardcoded in English Language
Product: OJS
Classification: Unclassified
Component: User Interface
All All
: P3 minor
Assigned To: PKP Support
Depends on:
  Show dependency treegraph
Reported: 2011-09-30 01:38 PDT by Philip Muench
Modified: 2012-12-19 13:25 PST (History)
1 user (show)

See Also:
Version Reported In:
Also Affects:

Changed /plugins/blocks/fontSize/block.tpl (875 bytes, application/octet-stream)
2011-09-30 01:38 PDT, Philip Muench
Changed /lib/pkp/js/fontController.js (3.02 KB, application/javascript)
2011-09-30 01:39 PDT, Philip Muench

Note You need to log in before you can comment on or make changes to this bug.
Description Philip Muench 2011-09-30 01:38:33 PDT
Created attachment 3647 [details]
Changed /plugins/blocks/fontSize/block.tpl

Since the Fontsizer JS introduces it's needed hrefs into the DOM of all pages itself, a localization of the tooltip texts is not possible through Smarty. Removing the introduction of the 3 button hrefs from /lib/pkp/js/fontController.js and putting them instead into /plugins/blocks/fontSize/block.tpl using the Smarty localization mechanism in that case, makes OJS show localized Tooltips. Attached are the changed files.
Comment 1 Philip Muench 2011-09-30 01:39:30 PDT
Created attachment 3648 [details]
Changed /lib/pkp/js/fontController.js
Comment 2 Alec Smecher 2011-10-05 17:07:32 PDT
Thanks for the fix, Philip. Unfortunately we can't integrate it as is, because we're trying as much as possible not to modify external code -- better to either find a more configurable font controller or contribute an improvement to this one to make it configurable. I'll leave this entry open until someone gets a chance to tackle it; in the meantime, your work-around looks good.
Comment 3 Alec Smecher 2012-12-19 12:59:08 PST
Swapped out the font controller for one without hard-coded English.
Comment 4 Alec Smecher 2012-12-19 13:00:04 PST
New font controller
Comment 5 Alec Smecher 2012-12-19 13:20:03 PST
Remove leftovers from old font controller
Comment 6 Alec Smecher 2012-12-19 13:20:03 PST
New font controller
Comment 7 Alec Smecher 2012-12-19 13:20:03 PST
New font controller cleanup
Comment 8 Alec Smecher 2012-12-19 13:20:03 PST
Removed leftover images
Comment 9 Alec Smecher 2012-12-19 13:25:01 PST
New font controller cleanup