[ 
http://jira.magnolia.info/browse/MAGNOLIA-2124?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16957#action_16957
 ] 

Philipp Bracher commented on MAGNOLIA-2124:
-------------------------------------------

Some days ago I quickly locked at the  patch and then finally resisted to apply 
it. A share the concerns of Philipp Marschall. I also thought on introducing a 
flag (controls, or tab level). The dialog would then add the additional 
controls for i18n controls/tabs (same result as you have now but less copy 
pasting, easy adding of new language).

For the most users it is important to have the original message in the same 
dialog in which they translate. May I invite you to start a concept page here: 
http://wiki.magnolia.info/display/DEV/Concept+i18n+next+steps

Note about activation: I consider it best to separate the i18n content from the 
real content. So that per language activation should be trivial. The i18n 
support was original designed in that perspective. The nodedata is not 
necesserely stored under the same node. Actually not even in the same workspace.

> I18nContentSupport for dialogs
> ------------------------------
>
>                 Key: MAGNOLIA-2124
>                 URL: http://jira.magnolia.info/browse/MAGNOLIA-2124
>             Project: Magnolia
>          Issue Type: Improvement
>          Components: gui
>    Affects Versions: 3.6
>         Environment: Irrelevant
>            Reporter: Stojan Peshov
>            Assignee: Boris Kraft
>             Fix For: 3.6
>
>         Attachments: i18supportForDialogs.patch
>
>
> In MAGNOLIA-1431 it is stated that we should make copies of dialog controls 
> (or even locale tabs) with locale extensions if we want to have i18n aware 
> dialog.
> This solution leads as to many copies (tabs) if we want to cover many 
> languages.
> Therefore, we've modified the initialization of dialogs in order to have one 
> dialog for all languages.
> The solution works like this: 
> When a page is displayed i18nContentSupportFilter sets the current locale and 
> when a dialog is opened all the control's names are suffixed with the locale 
> (if the locale is different than the fallback locale)
> ex. a dialog with edit control named "text" will become "text_[locale]".
> The rest of the job remains to the Save handler which creates the localized 
> content (nodeDatas).
> If you edit the dialog then the content of the current language is filled in 
> the controls.
> This way we have only one dialog and unlimited number of languages, without 
> the need of creating additional tabs (or controls).
> Note: This way there is no possibility to edit more than one language at 
> once. 
> For this functionality we modified the following:
>  - DialogControlImpl.initializeConfig
>  - I18nContentSupportFilter
>  - DefaultI18nContentSupport
>  - I18nContentSupport
> All of the classes have minor changes, that don't affect other 
> functionalities.
> We attached the patch and hope that you'll like it. :)
> p.s. we also have created a language chooser (changer) tag that successfully 
> implements the suggested solution

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.magnolia.info/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

----------------------------------------------------------------
for list details see
http://documentation.magnolia.info/
----------------------------------------------------------------

Reply via email to