-
-
Notifications
You must be signed in to change notification settings - Fork 113
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: introduce matchSource match template variable #1220
base: master
Are you sure you want to change the base?
feat: introduce matchSource match template variable #1220
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
- Change default fuzzy match pane template with matchSource - Add method MatchesVarExpansion#expandMatchSource - Extend NearString.MATCH_SOURCE to have TM_SUBSEG - Update test expectations of MatchesTextAreaTest, and FindMatchesTest - Add human-readable names of MATCH_SOURCE in Bundle.properties Signed-off-by: Hiroshi Miura <[email protected]>
821d6a9
to
53854ed
Compare
❌ Quality checks failed. Please look a Gradle Scan page for details: |
Signed-off-by: Hiroshi Miura <[email protected]>
This comment was marked as resolved.
This comment was marked as resolved.
Signed-off-by: Hiroshi Miura <[email protected]>
Signed-off-by: Hiroshi Miura <[email protected]>
❌ Acceptance Tests failed. Please look a Gradle Scan page for details: |
❌ Quality checks failed. Please look a Gradle Scan page for details: |
- Ignore NearString warnings Signed-off-by: Hiroshi Miura <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
The acceptance test error is
|
Signed-off-by: Hiroshi Miura <[email protected]>
…-template-variable
Signed-off-by: Hiroshi Miura <[email protected]>
I have updated bundle keys and test expectation. |
With new changes, the part of "<50/50/60% >" will become like
@t-cordonnier @brandelune is the resulted change suitable for your assumption? When #1226 is merged, No4 will become like "<50/50/60% Sub-segmented match tm/file1.tmx,tm/file2.tmx>" |
Ok. So we should first modify the second item in that representation of the match pane this way:
→
Is that correct ? In the case of orphan segments (we should not has a plural here, right @Kazephil), what will be the display ? Since orphan matches all come from the project file, there will be a double indication. Regarding the various labels:
Ok for now.
should be
should be
not easy to understand. What about
@t-cordonnier, what do you think about all that? |
Answer the question.
Current master branch has been programmed to store localized of "Orphan segments" as a TMX name field. see.
It may become fuzzy match pane to show
|
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
return localTemplate.replace(VAR_MATCH_SOURCE, OStrings.getString("MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_MEMORY")); | ||
case SUBSEGMENTS: | ||
return localTemplate.replace(VAR_MATCH_SOURCE, OStrings.getString( | ||
"MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_TM_SUBSEG") + " "); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_TM_SUBSEG") + " "); | |
"MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_SUBSEGMENTS") + " "); |
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_TM=External TM | ||
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_FILES=Source Files | ||
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_MEMORY=Project | ||
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_TM_SUBSEG=Sub-segmented match |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_TM_SUBSEG=Sub-segmented match | |
MATCHES_VAR_EXPANSION_MATCH_COMES_FROM_SUBSEGMENTS=Sub-segmented match |
split of #963
Pull request type
Which ticket is resolved?
RFE#1251
What does this PR change?
Other information