? Success

User tests: Successful: Unsuccessful:

avatar jsubri
jsubri
26 Aug 2016

Pull Request for Issue # .

Summary of Changes

Allow TinyMCE plugin to be reset to default language (english).
Updated the xml file to display the default value.

Testing Instructions

  • Edit the TinyMCE pluging, set "Automatic Language Selection = off", select a "Language Code" such as fr, and save.
  • Create a new article, the TinyMCE editor shall display the menus as per the "Language Code".
  • Go back to the the TinyMCE pluging, you can't select default (english)

After the patch you can.

Documentation Changes Required

none

avatar jsubri jsubri - open - 26 Aug 2016
avatar jsubri jsubri - change - 26 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2016
Category Plugins Front End
avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2016
Labels Added: ?
avatar infograf768
infograf768 - comment - 26 Aug 2016

the issue comes from the fact that the English language in TinyMce is not a xx.js specific file, but is included in the main js files.

avatar killoltailored killoltailored - test_item - 27 Aug 2016 - Tested successfully
avatar killoltailored
killoltailored - comment - 27 Aug 2016

I have tested this item successfully on e7ca7e0


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11804.

avatar RonakParmar RonakParmar - test_item - 27 Aug 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 27 Aug 2016

I have tested this item successfully on e7ca7e0


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11804.

avatar infograf768
infograf768 - comment - 27 Aug 2016

This works fine but I propose to also update the language string
from
PLG_TINY_FIELD_LANGCODE_DESC="Editor UI Language. The value will be used if Automatic language is not set."
to
PLG_TINY_FIELD_LANGCODE_DESC="Editor UI Language. The value will be used if Automatic language is not set. 'Use Default' will default to English "

@brianteeman
Would that proposal be OK?

avatar infograf768
infograf768 - comment - 27 Aug 2016

Another idea is to use this code:

                <field
                    name="lang_code"
                    type="filelist"
                    label="PLG_TINY_FIELD_LANGCODE_LABEL"
                    description="PLG_TINY_FIELD_LANGCODE_DESC"
                    class="inputbox"
                    stripext="1"
                    directory="media/editors/tinymce/langs/"
                    hide_none="1"
                    default="en"
                    hide_default="1"
                    filter="\.js$"
                    size="10"
                    showon="lang_mode:0"
                    >
                    <option value="en">en</option>
                </field>

so that we get:
screen shot 2016-08-27 at 07 47 09

@dgt41

What do you think?

avatar brianteeman
brianteeman - comment - 27 Aug 2016

I am not sure it needs updating at all as the tooltip is clear when you enable the option - but if you insist then

PLG_TINY_FIELD_LANGCODE_DESC="Editor UI Language. The value will be used if Automatic language is not set. 'Use Default' will default to English "

==>

PLG_TINY_FIELD_LANGCODE_DESC="Editor UI Language. The value will be used if Automatic language is not set. The Default is English."

avatar dgt41
dgt41 - comment - 27 Aug 2016

IMHO @infograf768 's proposal is the best solution here

avatar jsubri
jsubri - comment - 27 Aug 2016

Indeed this is a better solution, please re test.
Thank you

avatar RonakParmar RonakParmar - test_item - 27 Aug 2016 - Tested successfully
avatar RonakParmar
RonakParmar - comment - 27 Aug 2016

I have tested this item successfully on 63a9627


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11804.

avatar brianteeman brianteeman - test_item - 27 Aug 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 27 Aug 2016

I have tested this item successfully on 63a9627


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11804.

avatar infograf768 infograf768 - change - 28 Aug 2016
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 28 Aug 2016

RTC. Thanks.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/11804.

avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 29 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-29 17:32:32
Closed_By rdeutz
avatar rdeutz rdeutz - close - 29 Aug 2016
avatar rdeutz rdeutz - merge - 29 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 29 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 29 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment