When I click on a project's language strings to translate or review, don't list strings in locked components

[emmapeel2]

We have locked a component in our project because we don't know if we will translate it again yet.

While we have it on hold, I have locked it because I don't want translators wasting efforts if we don't keep releasing new translations. And I haven't removed it because is hard to backup all the configurations with the reviewed states and Add-on, etc.

While this project is locked and editions cannot be done, the strings keep appearing everywhere. For example if I go to the project/language view, and click on 'Strings waiting for review' on the Zen view, to review some strings, there is a message on top saying "The translation is temporarily closed for contributions due to maintenance, please come back later. Get notified when this project is unlocked again".

But is not the project that is locked, is just some strings. This strings should simply not appear on the list of 'strings waiting for review' or 'untranslated', as they are not waiting for any action: they are locked.

[nijel]

I don't think it's a good idea to filter them out completely because the translate view would be inconsistent with observed project status (project status would show strings to translate, but if all of these were in the locked component, the user would end up with the message that there are no strings to translate).

Instead, I've added lock status to ordering, making the locked translations appear last in the search: #18013