Provide a top level 'internationalisation' menu item.
catch - February 2, 2009 - 17:59
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | user interface text |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
| Issue tags: | Needs Documentation, Screenshot, Usability |
Description
In core we have locale.module and translation.module. Contrib has i18n, i10_server and others. I think it makes sense to collate the administrative items for these modules under a single "internationalisation" menu item - this would include the languages settings in site configuration as well (adding a language to a site isn't really a 'setting' IMO).
This would have the advantage of moving interface translation out of site building (it's not really site building), and making these features easily accessible in one place.
I'm also going to open an issue for a development menu item - for simpletest (core), but also devel and schema modules to add themselves to. Again, for much the same reasons.
#1
Agreed. These admin items can very well be grouped into their own top level admin section:
- The items in core are already a bit misplaced because it is indeed not really site building
- There's quite a family of internationalisation contrib modules available that will be much easier to find with this dedicated admin section.
#2
Here's a patch, turns out that translation module doesn't actually define an admin menu item anywhere - just form_alters the content type admin forms. However locale module has two top level items, and the translation settings could arguably not be hidden in workflow settings so maybe move here too.
Need to verify that if two modules define the same router item it'll just overwrite/ignore the second entry rather than break - going to do that with the development item now.
#3
screenshot.
Also, tests not updated yet.
#4
+1
Ultimately I think more work needs to be done. Seems like users should be able to enable multi-language support, and tweak the settings of it's output via the modules settings. I think this is getting closer: http://skitch.com/yoroy/brmpw/localization-client
None-the-less, this is better than what's there today.
#5
I agree this is an improvement
#6
All tests should pass with this one.
#7
The last submitted patch failed testing.
#8
The test I wrote came back to bite me. Updated that one too.
#9
Let's try some simpler wording. What is this setting-group's name in your favourite OS? e.g. in OS X it's called 'International'. How about:
International
Languages and translation
#10
In Ubuntu I have 'Language settings' and a separate section for date/time. Internationalization makes for a long path, and international sounds better to me.
New, non-empty patch and screenshot.
#11
The last submitted patch failed testing.
#12
Re-rolled for the new locale test, minor whitespace fixes crept into the patch, sorry.
#13
The last submitted patch failed testing.
#14
I would not create a top level menu item for 2 links..
The i18n and l10n_client modules do not add any new links to that group. The l10n_server module is used on 50 sites and all will be replaced by one translation server on d.o, so that is a bad example..
I would consider the possibility moving the "interface translations" inside "languages" because I many times searched for that inside the Language settings.
Or redesign the whole menu structure for language related menu links, because I am not convinced with the reasoning, this change isn't an improvement yet..
#15
Yes, all language locale and translation related menu items should move into this section. Contrib modules should update to move their links into this section as well.
#16
Sounds like a solid improvement, failed to apply patch - can someone review why?
#17
Re-rolled.
No idea why the patch failed, it were just text (url) replacements, some worked, some not.
#18
Marking this rtbc, I don't see any objections to this patch other then "other things should be under this as well".
#19
Awesome! Committed to HEAD.
This should be reflected in the ugprade docs for other modules that might want to add settings under here.
#20
documented :)
#21
Automatically closed -- issue fixed for 2 weeks with no activity.
#22
This broke the edit and delete links in the table on admin/international/translate/translate.
#23
We probably need some tests then...
#24
I tested this patch, and it fixes the bug (just a typo with leading slashes). Really this is not something likely to regress and we generally don't test our clickcable links everywhere in the UI with simpletest.
#25
Committed to HEAD. Thanks!
#26
Automatically closed -- issue fixed for 2 weeks with no activity.