- Issue created by @Chris Matthews
- Status changed to Needs review
over 1 year ago 10:02pm 3 June 2023 - last update
over 1 year ago 29,287 pass, 4 fail The last submitted patch, 2: alignment-checkbox-options-3364656.patch, failed testing. View results →
- last update
over 1 year ago Patch Failed to Apply - Status changed to Needs work
over 1 year ago 12:47pm 5 June 2023 - 🇺🇸United States volkswagenchick San Francisco Bay Area
Patch 2 and 4 are the same.
Needs a new patch as last one failed testing. See comment 3
Tagging this as novice and for event.Please save for a mentee at the mentored workshop at DrupalCon Pittsburgh 2023.
- 🇬🇧United Kingdom ChrisDarke London
Migrating Pittsburgh 2023 to Pittsburgh2023 tag for cleanup
- Merge request !4118Issue #3364656: Change the enabled alignments checkbox options to titlecase → (Open) created by toddses
- Open on Drupal.org →Environment: PHP 8.2 & MySQL 8last update
over 1 year ago Not currently mergeable. - last update
over 1 year ago 29,430 pass, 1 fail - last update
over 1 year ago 29,430 pass, 1 fail - last update
over 1 year ago 29,436 pass I think this is resolved! Added a title property for display purposes, as modifying the casing on the name itself causes ID mismatch issues.
- Status changed to Needs review
over 1 year ago 7:57pm 7 June 2023 - Status changed to RTBC
over 1 year ago 5:01pm 8 June 2023 - 🇺🇸United States smustgrave
Nice simple change. Think this is good for committer review.
- last update
over 1 year ago 29,437 pass - last update
over 1 year ago 29,437 pass - last update
over 1 year ago 29,445 pass - last update
over 1 year ago 29,450 pass - Status changed to Needs work
over 1 year ago 1:37pm 16 June 2023 - 🇫🇮Finland lauriii Finland
Sorry about the scope creep in the MR.. I think we should address it here though because if we're adding new key for human readable labels, we should make sure they are translatable.
I have applied the patch provided by @Chris Matthews, and it worked correctly
- 🇮🇳India Nitin shrivastava
@DavidMV97 Please add screenshot before or after the patch applied for betterment..
- 🇮🇳India rohit sankhla
I had tried #2 patch provided by @Chris Matthews. Not working for me. Attaching new patch. Please review.
- Status changed to RTBC
7 months ago 11:46am 2 May 2024 - 🇮🇳India rushiraval
I have applied patch #17 is working fine for me. I have attached screenshot for before and after patch.
- Status changed to Needs work
6 months ago 11:51pm 4 May 2024 - 🇺🇸United States xjm
Given that we have three experienced contributors to CKE5 disagreeing about the approach I think this is not really in a novice state. 😀 Thanks!