- Issue created by @joshahubbers
- Merge request !12616Issue #3534107 here was a problem creating field :@message → (Open) created by joshahubbers
- 🇳🇱Netherlands joshahubbers
Sorry, have been messing with the merge request. I think it is allright now?
- Merge request !12618Issue #3522970 by longwave, smustgrave: Remove unused BrowserTestBase::$originalContainer → (Open) created by arunsahijpal
The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.
- 🇳🇱Netherlands joshahubbers
joshahubbers → changed the visibility of the branch 3534107-typo-there-was to hidden.
- 🇳🇱Netherlands joshahubbers
Allright, new try in a clean branch. MR against 11.x. Hopefully this is ok. Sorry for the waste for such a small fix ;-)
- 🇺🇸United States smustgrave
Mind providing before/after screenshots for this issue and add back to the UI section
- 🇧🇷Brazil igorgoncalves
Really thanks for asking that @smustgrave, because its a trick one.
Despite typo @joshahubbers have fixed, the wrong space was being replaced at the line 262 when we pass through the explode(":")
And when i was trying to replicate the issue, i found a false positive as we can see below at the Before screenshot:
Before
The empty space in that case was caused by the lack of label on my test. when we add the label value the message contain no errors.
So, i suggested the change i made on last MR, and i will give the 2 new After screenshots, with and without label field filled.
After