Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
comma separated lists (css inline un-ordered lists) format without proper punctuation.
example:
Comment | File | Size | Author |
---|---|---|---|
#23 | 2722457-nr-bot.txt | 1.72 KB | needs-review-queue-bot |
#18 | Screenshot from 2021-01-30 12-35-00.png | 66.77 KB | anacolautti |
#5 | 2722457-follow_up--other-use-example.png | 35.87 KB | SKAUGHT |
#2 | 2722457-comment1__patch-applied.png | 14.78 KB | SKAUGHT |
#2 | 2722457-1--comma-list-item-list-element-humanizes-poorl.patch | 4.78 KB | SKAUGHT |
Comments
Comment #2
SKAUGHTpatch includes '&' symbol for the following languages:
Source is Google translate (best source i could find). Accordingly, These are the only languages that do not use '&'
Comment #3
SKAUGHTComment #4
SKAUGHTComment #5
SKAUGHTadding other example. as this element is used in a variety of spots. (ie: /admin/modules )
can be found by searching for
['list_style' => 'comma-list']
Comment #7
SKAUGHTi have no idea why this test would fail against a CSS change. especially, considering it's looking for a drupal_set_message that isn't using a comma-list at all. i have to assume there something wrong with this test on it's own.
Comment #8
SKAUGHTsimplytest.me test result on individual MigrateUpgrade7Test run on 8.1.1 (current release)
also:
Fatal error: Trait 'Drupal\Tests\SessionTestTrait' not found in /home/d0s72/www/core/modules/simpletest/src/TestBase.php on line 28
Comment #9
SKAUGHTpasses manually run test. ???
Comment #10
SKAUGHTjust linking
seems to be original issue for comma_list creation
Comment #18
anacolautti CreditAttribution: anacolautti at easytechgreen commentedI tried reproducing this issue in order to test the fix, but I can't. I don't understand how to trigger the error presented in the first screenshot.
In a Drupal 8.9-dev site (from 2021-Jan-20), I tried to enter incorrect values to the user edit form, but got three separated error messages that don't resemble the one in the screenshot:
Could someone explain how to reproduce this issue a bit more? Thanks!
Comment #23
needs-review-queue-bot CreditAttribution: needs-review-queue-bot as a volunteer commentedThe Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.