sanket.tale โ made their first commit to this issueโs fork.
- Merge request !86723226309: Moved the svg image to the images folder โ (Open) created by sanket.tale
- Status changed to Needs review
2 months ago 11:58am 5 July 2024 Created MR for the issue please review it.
Steps to reproduce
- example : Go to admin/structure/menu/manage/admin
- Check below menu link Here you can see parent and child link.
- You can see the space before the child menu. If you inspect before the MR you can see the svg tag and after the MR you can see the ::before pseudo element.
Before :-
After :-
- Status changed to Needs work
about 2 months ago 1:26pm 11 July 2024 - ๐บ๐ธUnited States smustgrave
Issue summary should follow standard template.
Also may need submaintainer sign off
- Status changed to Needs review
about 2 months ago 6:55am 12 July 2024 - ๐ฎ๐ณIndia BhumikaVarshney Delhi
The issue mentioned that svg image code drupal/core/themes/claro/templates/admin/indentation.html.twig added in this template need to be moved to image folder.
But this merge request https://git.drupalcode.org/project/drupal/-/merge_requests/8672/diffs?co... added the same svg by pseudo element.
So need more information do we need to include svg from img tag by using the image source in the url.
As you can refer the
Proposed resolution
Remove the svg tag code from the drupal/core/themes/claro/templates/admin/indentation.html.twig file and create a new svg file in the images folder. - ๐ฎ๐ณIndia ehsann_95
The SVG code has been removed from twig file and SVG has been moved to images/src folder.
- ๐ฎ๐ณIndia Kanchan Bhogade
Hi
I've tested MR 8672 on Drupal 11.x
MR is applied cleanly...Test Result:
The SVG code has been removed from the twig file.Attaching SS for reference
- Status changed to Needs work
about 2 months ago 8:54am 24 July 2024 The Needs Review Queue Bot โ tested this issue. It fails the Drupal core commit checks. 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.
- Status changed to Needs review
about 2 months ago 10:37am 24 July 2024 - Assigned to sagarmohite0031
- ๐ฎ๐ณIndia sagarmohite0031
I have tried to apply MR !8672 but MR is failing.
Please check Error screenshot.
I have applied MR 8672, but when I try to apply it locally, I receive an error. Attached a screenshot.
- Status changed to Needs work
19 days ago 11:39am 20 August 2024 - Status changed to Needs review
11 days ago 7:15am 28 August 2024 - Status changed to Needs work
11 days ago 7:34am 28 August 2024 - Status changed to Needs review
1 day ago 7:00am 6 September 2024