-
Notifications
You must be signed in to change notification settings - Fork 166
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
Venn diagram #2179
Venn diagram #2179
Conversation
@@ -1704,7 +1704,9 @@ | |||
"saveFromComparisonNameTail": " - From Comparison", | |||
"saveMessage": "Save New Concept Set From Selection Below", | |||
"saveWarning": "You must save the current concept set before you can perform this comparison.", | |||
"viewCountMessage": "View database record counts (RC) and descendant record counts (DRC) for:" | |||
"vennDiagram": "Venn Diagram", | |||
"viewCountMessage": "View database record counts (RC) and descendant record counts (DRC) for:", |
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.
Comma at the end makes the JSON invalid. Remove it.
"viewCountMessage": "View database record counts (RC) and descendant record counts (DRC) for:" | ||
|
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.
Delete empty line. Line count in all localization files should be equal.
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.
Btw: while it's a good check to ensure localization lines should be equal across files, I noticed that the order of keys is out of order across files. This makes it difficult to use something like WinMerge to find differences in files, where we could find out that a key exists in one file and is missing in another (but this would only work if the keys were in the same order).
Just something to think about for a future update.
added locales to venn diagram