Show obsolete entities as such
It's currently possible to end up looking at an obsolete entity in Pontoon when using its entity identifier directly as a string query parameter. When this happens, we should highlight for the user that the string they're looking at is not in active use, and ensure that it's treated as readonly.
Some cleanup of the previous/next entity hooks is included.
It's currently possible to end up looking at an obsolete entity in Pontoon when using its entity identifier directly as a
stringquery parameter.
Should we keep allowing that, or prevent it completely as we'll do when https://github.com/mozilla/pontoon/issues/3148 is fixed?
Occasionally I find value in being able to see Pontoon's representation of an obsolete entity and its translations, but we might indeed want to seek to prevent that from happening by default. But even if we do, something like the changes here would be a Good Idea in case any later change makes obsolete entities visible again, even if accidentally.
@flodolo and I talked about this and we both think that we should just prevent obsolete entities from showing.