sort by Alphabet benefit:

- a simple way to find out similar structure, can help to be consistent in using words
- we can easily to copy same structure translation, save time to do same translation again and again, improve workflow and efficiency

also it do not cause performance issue (on my local test).

Comments

toemaz’s picture

@KayL Could you provide a patch? Otherwise, copy your changes into a comment and I'll make the patch. Thanks in advance.

gábor hojtsy’s picture

Title: strings sort by Alphabet » Sort strings alphabetically

Agreed, that the current "in the order of being added" ordering is counterproductive.

toemaz’s picture

Status: Active » Needs review
StatusFileSize
new722 bytes

Find attached a patch. Tested but it needs review.

droplet’s picture

StatusFileSize
new2.52 KB

Add sort filiter

Status: Needs review » Needs work

The last submitted patch, sorting.patch, failed testing.

droplet’s picture

include_once(Archive/Tar.php): failed to open stream: No such file or directory

l10n_community.test bug ??

droplet’s picture

Status: Needs work » Needs review

#4: sorting.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, sorting.patch, failed testing.

gábor hojtsy’s picture

Well, the testing server should have a copy of Tar.php, and it does seem to have a PEAR reference in its include path: include_once(): Failed opening 'Archive/Tar.php' for inclusion (include_path='.:/usr/share/php:/usr/share/pear')

gábor hojtsy’s picture

I've opened #712516: No PEAR installation on testing for the testing issue.

tobiasb’s picture

StatusFileSize
new2.56 KB

updated

droplet’s picture

last patch only failed at Tar.php. let me review this again.

droplet’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, l10n_server_598898.patch, failed testing.

gábor hojtsy’s picture

Status: Needs work » Fixed

Committed #3. I don't think having a sort order of ascending and descending or no order as options is of usefulness. Thanks.

Nenad’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Component: Miscellaneous » User interface
Status: Fixed » Active

Can we have sort 'by module' like an option?

So far we have organized translation community to translate strings by page and we did a lot of work in this way,
since the sorting order has changed, we lost string's organization and would have to go from the beginning...

The whole Serbian translation team agrees that sorting option 'by module' is better solution for us.

gábor hojtsy’s picture

@Nenad: by module *filtering* was always an option, and it is safer to use filters compared to paging due to bugs with how the paging could behave (see #787068: Translate form re-building should not rely on pagination).

droplet’s picture

can we add sort by suggestions ??

Admin & User able to understand which suggestion last submitted. we can improve it or approve it. If it mixed, we usually miss new suggestion has been added. (suggestions lists growing everyday, we can not approve & decline each strings because sometimes we are waiting better suggestions)

also add last approved stings, admin can check each others work, double check, etc...

Nenad’s picture

We didn't understand each other...
We do NOT want "filtering" per module, we want to "sort" by module, to be able to see the strings ordered "by module" as before.
Currently there is no option for sorting "by module".

Is there a way to return to the old sort(by module) to be optional, that people can choose how they want to sort?

I already asked you about this subject:

http://drupal.org/node/598898#comment-3031924

http://drupal.org/node/810020

Tnx.

pivica’s picture

Loosing of sorting order by module and organization problem that we have now with new sorting is one problem that we have.

Second problem is that for this new sorting by first letter in translation string, translation is more difficult for us - old sorting by module is better because we know in which context we are and that helps us a lot with translation - because some word we can translate differently in different context.

droplet’s picture

Status: Needs work » Active
StatusFileSize
new3.56 KB

I patched for my own server and now share to community.

added Sort by:
No Sort - some strings like Mo,Tu,Web...., if sort by Alphabetically, it goes to everywhere ..
Alphabetically - read #0
Suggestion - read #18
Approved time - read #18, a way give admin to understand other admin's recent work
String Length - I always translation short STRING first & gain user point :)

If you think it worth to add into l10n_server, please turn the status to "needs review" and improve my patch.

:)

droplet’s picture

Status: Active » Needs review

my group members told me they hope it on L.D.O.
I switch it to reviews patch on QA first, no matter it add or not.

Thanks.

gábor hojtsy’s picture

Status: Needs review » Needs work

Hum, to not interfere with the live service that much, I rolled back the commit so we can take our time to think this through properly. I'm not sure for example that a sort widget belongs in the filtering fieldset.

droplet’s picture

Status: Active » Needs work

put it on right top of list

---
share my 2 cents,
I think the most important stuff on LDO is not how to make it easier for translator. while they can download .po, there are thousand way to do translation. But admin MUST working on LDO UI. so how to reduce admin's work is priority.

1. more sorting way.
2. mark it as Need Work & can be hide it from admin side

Thomas_Zahreddin’s picture

another nice way of sorting would be the sting length: the shorter the string, the easier to translate

and

as short strings often have only few words, this way the words are translated first and can therefore consistent translated also in longer strings.

droplet’s picture

@Thomas_Zahreddin ,
look at #21, I patched multiple way to sort.
Now wating for UI change ?? or ..... ? I'm not sure. Look forward to see Gábor's advice

droplet’s picture

Title: Sort strings alphabetically » Sorting in more way
StatusFileSize
new3.79 KB

reroll patch

droplet’s picture

Priority: Normal » Major
Status: Needs work » Needs review
gábor hojtsy’s picture

Added #1118430: Set up the filtering/display setting UI for future growth to set us up for filter extensions like this.

SebCorbin’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev
Issue summary: View changes
StatusFileSize
new2.77 KB

Re-rolled against D7