Closed (fixed)
Project:
Localization server
Version:
5.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
15 Oct 2007 at 21:40 UTC
Updated:
31 Dec 2007 at 21:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
anders.fajerson commentedHere is a screenshot, before and after.
Comment #2
anders.fajerson commentedMore explanation:
* Removed the group link, not 100% sure about that but I think it's enough to link to it on the next language page. This also (unintentionally) fixed a bug as the group linked was not wrapped in
array(). It feels like the group thing is not fully explored yet (at least not by me), so it will evolve...* Percent is rounded to whole integer (73% instead of 73.94%). Should be exact enough. I also don't display the count for number of translated strings; the percent value + number of untranslated strings gives a rough estimate, so it's redundant.
Comment #3
gábor hojtsyI only looked at the screenshot, but the improvements look fine indeed.
I don't understand this.
I agree the overview pages should be de-cluttered, but as far as I see, the groups link on the translation page itself is very much hidden, so I would not say people will find it easily.
Also "Options" should be "Operations" for consistency with Drupal.
In general, looks very good, but definitely needs some thinking still.
Comment #4
gábor hojtsyI also forgot to mention that "Has suggestions" was carefully worded to indicate that the number is the sum of source strings having suggestions not the number of suggestions added to the system. If 3 strigns have 4 suggestions each, the number will be 3, not 12. I think 3 helps better estimate the data in an overview.
Comment #5
anders.fajerson commentedAgree on all points.
I have to look into the group funtionality a bit more but it may very well be the case that it should be linked directly on this page.
I can see that I was a bit unclear on the label thing. The point was that it now has it's own column, but semantically it may be more correct to have it in the same cell as the bar it describes. It's a minor thing though.
Comment #6
psicomante commentedfajerstarter screenshot looks better. I think it could added as patch :)
Comment #7
pvasili@drupal.org commentedI too think it is necessary to add to release.
It is necessary to update for new pages.inc
Comment #8
pvasili commentedSmall improvement (see http://drupal.org/files/issues/l10n_server_overview.gif):
In l10n_community.css
and small correct pages.inc (rename it)
Comment #9
gábor hojtsyOh, well, the textual explanation was an improvement over the number columns. Now you suggested getting number columns back is a good idea?
Comment #10
pvasili commentedLooks compactly and conveniently:)
Comment #11
gábor hojtsyYes, after you used it for a few weeks, and you know *exactly* what the numbers mean.
Comment #12
psicomante commentedlol :)
it could be a good idea creating an option for short and extended view. Extended like "before", and Short like "After".
But i think is more important manage problems of some modules, like blogtheme, which i can't extract any string.
Comment #13
pvasili@drupal.org commented#11 Anonyms and beginners are not engaged in translation :)
Comment #14
gábor hojtsyWell, the goal of this system is to get beginners to translate stuff.
Comment #15
pvasili commentedThen it is necessary to make "Export in zip" :) (http://drupal.org/node/190436)
Comment #16
gábor hojtsyI should have taken your advice to not shout untranslated stuff in red. In a commit post-alpha2, I fixed this one, so the bars are green on gray. Otherwise I implemented overview page simplifications and made the pages more human friendly, so I think we can consider this one fixed.
Comment #17
(not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.