Highlight leading and trailing spaces, and double/multiple spaces in the middle #1500
Conversation
|
@amieiro currently the Translation markup is added to editor |
|
Using the
|
|
Added support for strings with multiple lines: |
|
Adding another highlight: double/multiple spaces between words.
Changing the CSS to a darker color with opacity 50% so it works better on different backgrounds of the string status:
|
pedro-mendonca
changed the title
|
Added unit tests for both the spaces highlight introduced in this PR, and the already existent markup for invisible characters as tabs and line breaks |
|
We agreed on our triage session that this can be merged after a code review. The functionality is fine. |
|
just ... thank you so much !!! |
4 participants
What?
Highlight the leading and trailing spaces both in the preview and editor rows
Also highlight double and multiple spaces in the middle.
This PR also adds a missing unit test for the invisible characters markup added on prepare_original() for line breaks and tabs.
Why?
Allows the translator to see the spaces at the beginning or the end of an original/translation.
Allows authors and translators to see double and multiple spaces in the middle of strings.
How?
Wrap the spaces in a
<span>withinvisible-spacesclass, which adds a grey color.Testing Instructions
Fixes #863.