Opened as followup to #2420737: Differences in dynamic language names are confusing in views, content, etc. based on @xjm:
+++ b/core/modules/views_ui/src/Tests/ViewEditTest.php
@@ -138,6 +138,73 @@ public function testEditFormLanguageOptions() {
+ // Test that the order of the language list is similar to other language
+ // lists, such as in Views UI.
Minor: This is the test for the Views UI. I think this comment should be "in the content translation settings" or such? I was going to just fix this on commit but my brain has melted and I also am not sure of the proper label for the... thingy. So this is a tiny (and novice-friendly) followup to make.
| Issue category | Task because comment is incorrect in the test (not a functional bug). |
|---|---|
| Issue priority | Minor because its a comment in a test. |
| Unfrozen changes | Unfrozen because it only changes documentation. |
| Disruption | Not disruptive. A comment in a test. |
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | 2494915.patch | 744 bytes | gábor hojtsy |
Comments
Comment #1
gábor hojtsyThat is the right text for this one yup.
Comment #2
cilefen commentedComment #3
gábor hojtsy@cliefen: really, for one line of comment change in a test? Not sure it makes it more clear :D
Comment #4
cilefen commented@Gábor Hojtsy It's your call. We are trying to get more novices involved, and different contributors have different trust levels with the committers.
Comment #5
cilefen commentedComment #6
jhodgdonThis is correct.
Comment #7
webchickCommitted and pushed to 8.0.x. Thanks!
Comment #9
gábor hojtsySuperb, thanks.