I have applied MR !14 which introduced an issue where all dropdown items render with white text. I’ve created MR !15, which sets a white background when the select is focused so the options remain readable.
I am attaching screenshots for reference.
The current branch and the Composer release is not sync. In the branch, the template uses {{ tag.getName|raw }}, which avoids the ampersand issue, but the Composer package doesn’t include this change, so the problem still occurs there.
tanushree gupta → made their first commit to this issue’s fork.
I’m preparing an MR for this issue and checked src/Entity/Webform.php. The change from the “use-config-language” patch is already in core: it already uses \Drupal::languageManager()->getConfigOverrideLanguage()->getId() for the current langcode.
tanushree gupta → changed the visibility of the branch 3548066-RemoveUserRestrictions to hidden.
tanushree gupta → changed the visibility of the branch 3548066-remove-getOriginalId() to hidden.
tanushree gupta → changed the visibility of the branch 3548066-remove-userrestrictionsgetoriginalid to hidden.
I attempted to reproduce the issue, but it seems that the Calendar module requires the Views Template module, which is no longer supported in Drupal 9. I also tried replicating the issue in Drupal 10, but I was unable to observe any design break in the status message.
If there are any steps or settings I may have missed, please let me know. Otherwise, if the issue isn't reproducible by others, we can consider closing it.
Thank you
Created a new MR: https://git.drupalcode.org/project/acquia_claro/-/merge_requests/13
I am attaching screenshot for reference.
Fixed tab title encoding. “&” was showing as “&”.
We stopped pre-escaping titles and now filter the raw title with Xss::filter (allowing limited tags).
For link labels, we pass the filtered text as safe markup.
Here are screenshots attaching for reference.
Created MR of patch.
Please review.
the grouping field is now appearing above the table.
Here are the screenshots I am attaching for the reference.
Install the theme > go to block layout > in main navigation block, in menu levels, Set number of display to unlimited and tick the option expand all menu links.
and then apply MR 1.
The menus will now show on hover.
Here is the screenshot I am attaching for the reference.
tanushree gupta → changed the visibility of the branch 3531877-menu-hovering-fixes to hidden.
Corrected the issue of search overlapping with main navigation.
Attaching screenshot for reference.
Added space on top of login form.
I have followed certain steps to work on this issue:
- Install the theme
- Set as default
- Go to Homepage
- Go to block layout and disable the search from header and add search form in search block.
By doing these steps, I can see that MR 12 is already working
Here I am attaching the screenshots for reference.
So changing state to RTBC.
Thanks.
I have installed the theme in Drupal 10, but I am unable to see any space below the footer.
I am attaching a screenshot for reference.
@pravesh_poonia had worked on this issue and changed its state to Needs Review but forgot to unassign it. Since he worked on it around 9 months ago, I am now assigning the issue to myself.
I have reviewed this issue and find that the problem has been resolved.
Hence, moving the issue to RTBC.
Attaching screenshots for reference.
Thanks.
I have reviewed the issue and attached the relevant screenshot for reference. Upon checking, I observed that the tags are not styled yet. Additionally, the 'Read More' and 'Add New Comment' buttons are styled, but the yellow color used does not align with the site theme. As per the design themeing, you should use #FFC451 instead.
Hence, I am moving this ticket to Needs Work.
Added the previous mentioned lines in ParagraphsPreviewController.php
Working on it.
Updated form css as per comments.
I have attached one screenshot for reference.
Updated submenu
Here is the commit id: https://git.drupalcode.org/project/corporate_lite/-/merge_requests/6/dif...
I have updated submenu and created hamburger menu as well.
Here are some screenshots, I am attaching for the reference.
Rerolled the MR.
Please review!!
I have updated the implementation with the following changes:
1. Refactored the block branding class name for consistency.
2. Resolved the submenu overflow issue by ensuring proper alignment and containment of nested menus.
3. Enhanced the menu to support and render two-level submenus with correct positioning.
4. Implemented the hamburger toggle functionality to dynamically switch between the hamburger and close icons.
5. Added expand/collapse behavior for submenu items on mobile devices to improve navigation usability.
Here are some screenshots for reference.
Thank you!!
We've removed the `file_uri_scheme()` mock from the test as requested. After doing that, the test is now throwing a 500 error — likely because the function isn't available in the test context anymore.
We've updated the PHPStan baseline to reflect the current state.
Just wanted to check how you'd prefer we handle this:
* Should we include the required `file.inc` file manually in the test (using something like `require_once`)?
* Or would it make more sense to convert this into a `KernelTestBase` or another test type where the function is already available?
* Or is it okay to leave things as they are now with the updated baseline?
I've removed the provided lines
Kindly review
tanushree gupta → made their first commit to this issue’s fork.
The changes looks good to me, since patches are deprecated converted the patch to MR.
Please review
tanushree gupta → made their first commit to this issue’s fork.
Working on it.
The issue of overlapping the text has been resolved.
Here is the commit id: https://git.drupalcode.org/project/business_bee/-/merge_requests/17/diff...
Menu dropdown is not visible,
you can do some steps for this, goto block layout and inside it there is a block of main navigation, inside it click on expand menus and on levels increase it.
MR: https://git.drupalcode.org/project/magazine_lite/-/merge_requests/8
tanushree gupta → made their first commit to this issue’s fork.
I have added space above and below the buttons, but the issue in this theme is with the CSS. The CSS file does not exist here, which may be causing the problem. However, once gulp runs and the CSS is generated, the added space above and below the buttons will be visible.
Here is the MR: https://git.drupalcode.org/project/governor/-/merge_requests/10
I applied the patch, and the changes are visible. However, there are some warnings. I tried to resolve them, but they are not appearing. I am sharing a screenshot for your reference. Apart from this, I believe the changes are visible, so you can review them.
No work has been done in the last 8 months, so I am assigning it to myself.
@niharika.s, I hadn't noticed that you had also assigned this task to yourself. I had already taken it on and completed the corrections. Please don't take this the wrong way.
This message came a bit late; I had already assigned it to myself.
Corrected CSS please review.
MR: https://git.drupalcode.org/project/magazine_lite/-/merge_requests/7
tanushree gupta → made their first commit to this issue’s fork.
I mistakenly removed certain lines of code, but I have now corrected this error. I have tested the MR patch in both 1.0.x and 9.1.0 versions, and it is working perfectly in both cases. Regarding the previous screenshot you shared, the hamburger menu is not in the correct position because the cache was not cleared after applying the patch. Please clear the cache after applying the patch.
I am attaching a screenshot for the reference.
There were several issues with that MR, but they have now been resolved. Please check.
MR: https://git.drupalcode.org/project/creative_innovative/-/merge_requests/25
tanushree gupta → changed the visibility of the branch Update-Navbar to hidden.
tanushree gupta → changed the visibility of the branch Update-Navbar to hidden.
The comment form was already present but was hidden due to unnecessary CSS applied to it. So, the CSS has been corrected.
Here is a screenshot I am attaching for the reference.
MR: https://git.drupalcode.org/project/cascade/-/merge_requests/4
Added components to structure/menus/footer and then placed these footer menus into the footer region using the block layout. Additionally, I made some required changes to page.html.twig and added styling for the footer.
I am attaching screenshots for the reference.
MR: https://git.drupalcode.org/project/ui_suite_daisyui/-/merge_requests/50
Update the checkbox field in the form.
Here is the MR: https://git.drupalcode.org/project/ui_suite_daisyui/-/merge_requests/47
I am attaching screenshot for the reference.
The size of the checkboxes is correct, but the design is breaking. I am attaching a screenshot for reference.
tanushree gupta → made their first commit to this issue’s fork.
Solved the issue of tags overlapping the icon in article(when adding tags).
MR: https://git.drupalcode.org/project/business_bee/-/merge_requests/17
tanushree gupta → made their first commit to this issue’s fork.