- 🇺🇸United States smustgrave
Closing 📌 Remove @todo-comment linking to closed issue in layout-builder.css Closed: duplicate and reopening this. Can we confirm the code in question with the todo isn't providing a workaround for the problem?
- 🇮🇳India nayana_mvr
Verified the issue mentioned in the ticket description in Drupal version 11.x and can confirm that it is not reproducible. Please see the attached screenshot for reference.
So I think the the code with @todo comment can be removed.
- 🇮🇳India nayana_mvr
I have removed the unused code and the comment and verified that it is not affecting the existing layout builder style. Please review.
- 🇺🇸United States smustgrave
The issue summary does not match the solution seen in the MR.
Any UI change should have before/after screenshots in the summary showing the issue
Tagging for issue summary update.
- First commit to issue fork.
- 🇮🇳India supriyak
Hi,
While going through issue, got reference of linked issue ( https://www.drupal.org/project/drupal/issues/3347648 📌 Remove @todo-comment linking to closed issue in layout-builder.css Closed: duplicate ).
There we have suggession to remove only @todo-comment linking. So removed only comment line and raised merge request(https://git.drupalcode.org/project/drupal/-/merge_requests/9763) for that.
@smustgrave As there is no UI change or any such change do we need to update summary? Please let me know.
Thanks!