I've applied your changes from
#2 →
and can confirm that the issue is resolved & looks good. Please see attached file.
Thank you
Tested with the patch - 3379974-5patch. The issue is resolved on desktop and mobile. This is good to go. Please check the screenshots.
Testing steps:
- Install the Gin theme.
- Install the Admin Toolbar module.
- Go to the theme settings and navigate to Navigation (Drupal Toolbar), then choose Legacy or Classic Drupal Toolbar.
- Proceed to admin/config/user-interface/admin-toolbar-search-settings.
- Activate "Display the search input as a menu item."
- Click on the search bar.
Hi Alok,
I have reviewed your Merge Request. The MR works properly, but I have one suggestion regarding your code. Due to the 'overflow hidden' property in the 'li' element with links, the 'padding top' and 'bottom' are not being applied to '.node-readmore a'. So, please make these changes or we may need to focus on this area more.
Thank you.
after
before
remove overflow hidden
Hi Alok,
From my side, the menu looks proper. Can you please confirm if you still encounter any issues?
Thank you.
Hi Alok,
Now the regions look proper and systematic. I have reviewed it from my side as well. I have also attached before and after screenshots.
Thank you.
After
Before
Hi alok,
I've applied your changes from branch (corporate_responsive_theme-3390032) and can confirm that the button UI looks good. Please see attached file.
Thank you
Hi Shiv & Shweta,
Could you please provide steps to reproduce the issue? I'm having trouble with the theme installation steps. Thanks in advance.
Hi alok,
I've applied your changes from branch (fepper-3389708) and can confirm that the footer section UI looks good. Please see attached file.
Thank you
Hi Shiv
We cannot check the image every time to see if the text is appearing properly on it or not, so we will have to work on the text itself, either through text shadow or explore another solution.
Hi,
patch #2 has been successfully applied on my local environment. I have also added screenshots, both before and after the translation.
Before patch
After patch
Hi
On my local system, patch number #2 is not applying. However, after adding the code and manually incorporating the code changes from comment #4, the asterisk mark is displaying correctly.
Hi
patch #24 has been successfully applied on my local environment. I have also added screenshots, both
before
and
after
Added wrong SS replacing new SS
Hi,
Verified patch #64 on Drupal version 10.1.x with the Olivero theme. The patch applied cleanly, and I can confirm that the issue is fixed for the Olivero theme. I have attached a screenshot for reference.
Hi @Gauravvvv, I followed the steps you provided to check the issue, but I'm unable to reproduce it. I followed these steps:
Created a new view.
Added it to the filter criteria.
Opened that filter criteria.
Then, I removed it and added another filter criteria.
Please let me know if there's anything else I should try or if you have any additional information. Also I have attached a video for more information. Please take a look.
Hi, the issue has been resolved in version 11.x. Please refer to the attached screenshots for verification
Hi
Patch
#2
🐛
Switch to Horizontal orientation button disappears when sub menus are opened.
Needs work
has been successfully applied, and the code is now working on my local machine with the icon visible. I have also attached screenshots from before and after.
Hi
This module doesn't support D11. I even tried it with the beta version, but it still gives an error. So, as of now, this module is only for D10.
Sorry miss the patch in comment #7, adding the in this comment
Hi
as per the last comment i have reproduce issue on my local & created patch for the issue Background colour of UI widgets get overridden on Ajax load, attached the images for reference
Needs review
Before patch
After patch
I'm unable to reproduce the issue. I have followed the provided steps, and there is already a "position: absolute" property in place. I've also tested it with different roles and added a screenshot.
Reviewing this ticket.
The given patch does not work for me showing an error. Please refer to the screenshot
Done patch work successfully working as expected
I have attached a screenshot.
I will double check patch #19