Hello, applied patch #2. It is working as per requirement. Here are before and after screenshots for the same. RTBC++.
hello, This is not replicated on Drupal 11 Claro theme.
Can you please provide any more details how this gets replicated?
Hello, applied patch #3 and it is working a expected. Here are the before and after screenshots for desktop and mobile. RTBC++.
hello, we have a duplicate issue created
https://www.drupal.org/project/scheduled_transitions/issues/3073556
🐛
Add a form label to 'Add scheduled transition' target state dropdown
Needs work
created under core issues.
I was working on this issue and applied patch #10 and it is still looking off for admin themes like gin, Thunder ( as it was mentioned to check in #8)
It was not fully perfect so after applying patch #10 this is how it looked (see the before screenshots)
Here is my patch to fix the issue in these admin themes and the after screenshots for the same.
Working on this issue.
This is resolved. Here is the patch.
snehal-chibde → created an issue.
Reviewed and applied the patch, it is working as expected.
Added before/after screenshots for reference.
Hello, this issue is resolved. On enabling the verbose error logging the error was noticed as below
"Error: Call to undefined method Drupal\node\Entity\Node::URL() in professional_responsive_theme_preprocess_breadcrumb() (line 198 of themes/contrib/professional_responsive_theme/professional_responsive_theme.theme)."
Here is a patch to resolve the same.
Added before and after screenshots for reference.
Hello @jannakha, here is the updated patch.
Thanks!
hello @Nitin, The issue does not replicate.
I have used,
Drupal 9.5.11
professional responsive theme 2.0
Attached screenshots for reference.
Hello @jannakha, here is a patch to resolve the above issue without the function bootstrap5_form_system_file_system_settings_alter.
Hello, this issue replicated in slightly different way for me (check screenshot). Worked to resolve the same.
Here is the patch and screenshots.
Reviewed the patch and it is working as expected. Here are the before and after screenshots for the same.
This is resolved. Here is the patch.
Hello, This issue has been resolved.
Here is the patch and before after screenshots.
snehal-chibde → created an issue.
Hello @nubeli, Implemented the proposed solution in a patch and verified, it is working as expected.
Please review patch and refer the before after screenshots.
hello @ hkirsman, I have added the above code in respective files and this is working as expected.
Added before and after screenshots for reference.
hello @Philalawst, currently we do not have an option to add alt tag for image block. I will work on the same, and create a patch.
hello @bnjmnm, added dotted underline as per the comment. Here is the updated patch and screenshot.
The above patch fails for me, also I have added a patch with a selector which will target all the a tags inside p which does not have any class, as needed as per the comment by @hudri.
Hello, checked the above patch coffee-aria-fix.patch and it works as expected.
Added before and after screenshots for reference.
@hudri sure, will check on this.
Here is the patch for fixing this issue.
Attached screenshots for reference.
I am working on this issue.
This is resolved.
Refer before and after screenshots for reference.
This issue is getting replicated. I am working on the solution.
Hello , Patch #3 applied successfully on on Drupal 9.4. Issue solved
Refer screenshots.
Hello @gaurav-mathur, the patch is failing.
This is working as expected in the 2.2.0 release.
Added screenshots for reference.
The color variables are working as expected.
hello, taking this up for review.
hello, taking this up for review.
I am working on this issue
hello @gaurav-mathur, Added a menu item with link text only and tested the patch. It is working as expected.
Refer before and after screenshots.
Hello, the remaining 2 issues have been resolved. Please review the patch.
Refer screenshots.
I am working on this issue.
Hello, the MR gets applied successfully, patch #2 fails.
Added screenshots for reference.
Hello, created a patch to fix this issue.
Here is the patch to resolve the issue. refer screenshots for changes after applying the patch.
I have picked up this issue, submitting a patch soon.
snehal-chibde → created an issue.
Here is a patch to resolve the spacing issues.
I am working on this issue, will post a patch soon.
snehal-chibde → created an issue.
Hello, here is the patch to resolve the issue
I have picked this issue , working on it
snehal-chibde → created an issue.
hello @david_nawrot, applied the above patch and it is working as expected.
hello @priya, yes created a separate issue ticket for the same https://www.drupal.org/project/tara/issues/3372164 🐛 Submenus in Mobile not toggling. Closed: works as designed . working on it.
snehal-chibde → created an issue.
hello @priya, under the menu "Show as expanded" checkbox should be checked.
Please check screenshot attached.
Hello @Harish,i nstalled the Drupal 10.0.x version and set Tara theme v10.04 version and applied the patch. The transition of dropdown menus are better. Attached recording.
@Priya_Degwekar, I Installed Drupal 10.0.10 and Downloaded Tara Theme 10.0.4 and checked the main navigation menu in mobile. It seems that submenus are appearing. The issue is observed in the alignment of the same. Screenshot attached for reference. Please let know if I am missing to check something.
@Alok singh, I installed Drupal core 9.2.0 and Tara Theme 9.2.0 and followed the steps as mention in the "Steps to reproduce" but not able to see the issue mentioned. Attached screenshot for reference.
Applied patch #3 on Drupal 9.5.10 with Business Responsive Theme 2.0.3
The patch worked perfectly and we can see the contextual filter as expected.
Attached screenshots for reference.
@sassafrass, I installed Drupal core 10.1.0 and Webform 6.2.0-beta6 followed the steps as mention in the "Steps to reproduce" but not able to replicate the issue.