<!--IssueSummary start--> <details> <summary> Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards. </summary> - [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=25900) </details> <!--IssueSummary end--> At the moment, the error message when the diff cannot be displayed, it's split among different subclassed objects. The final sentence is rendered as: `"This %{viewer} could not be displayed because %{reason}. You can %{options} instead."`. To build it we need info from the diff viewers because each one of them stores the `%{viewer}` part, and again we need them for the `%{reason}` and `%{options}`. As @reprazent pointed out in https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/23812#note_126229525, this structure can lead to weird translated sentences. For example: `Deze gerenderde diff kon niet worden weeregeven omdat het te groot is. Bekijk de blob` vs `Deze diff kon niet worden gerenderd omdat hij te groot is. Bekijk de blob.` The proposal is to rewrite each message as a whole.