🇵🇭Philippines @Zed9

Hi
I've tested the button hover and it looks good.
Please see attached image for reference.

Hi @arti_parmar
I'm still getting issues below

FILE: /themes/contrib/writer/PATCHES.txt
--------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------
5 | ERROR | [x] Expected 1 newline at end of file; 3 found
--------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------

FILE: /themes/contrib/writer/writer.theme
--------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------
11 | ERROR | [x] Use statements should be sorted alphabetically. The first wrong one is Drupal\comment\CommentInterface.
--------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
--------------------------------------------------------------------------------------------------------------------------

gab. → created an issue.

@sanket.tale
I've tested the breadcrumbs on mobile and it is fixed.
Thank you

The fix seems to working, not encountering any issue during import. However this might be another issue if I interchange parent and description columns on csv, description and parent are imported.

Gab. → created an issue.

I'm still finding cs issues after applying MR https://git.drupalcode.org/project/openai/-/merge_requests/94

FILE: /web/modules/contrib/openai/PATCHES.txt
-----------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-----------------------------------------------------------------------------------
 5 | ERROR | [x] Expected 1 newline at end of file; 3 found
-----------------------------------------------------------------------------------
PHPCBF CAN FIX THE 1 MARKED SNIFF VIOLATIONS AUTOMATICALLY
-----------------------------------------------------------------------------------


FILE: /web/modules/contrib/openai/modules/openai_devel/src/Plugin/DevelGenerate/ContentGPTDevelGenerate.php
-------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
-------------------------------------------------------------------------------------------------------------------------------------------------
 254 | ERROR | The array declaration extends to column 82 (the limit is 80). The array content should be split up over multiple lines
-------------------------------------------------------------------------------------------------------------------------------------------------


FILE: /web/modules/contrib/openai/modules/openai_content/openai_content.module
--------------------------------------------------------------------------------------------------------------------------------------
FOUND 1 ERROR AFFECTING 1 LINE
--------------------------------------------------------------------------------------------------------------------------------------
 217 | ERROR | The array declaration extends to column 95 (the limit is 80). The array content should be split up over multiple lines
--------------------------------------------------------------------------------------------------------------------------------------

This is already fixed on 9.1

I added a simple patch to fix the problem on my end

Gab. → created an issue.

Closing issue as this has already been fixed.

I used patch #25 and works perfectly. Will have to find a workaround for using the @start token though.

Hi @adarshv
I've reviewed and tested patch #6, and can confirm that hamburger menu on mobile is working. Please refer to the image.
Thank you

Gab. → created an issue.

Hi @shweta__sharma
I was able to reproduce the same issue upon installable. Applied the MR(event_s-3412048) as well and the theme installed successfully.
Thank you

Gab. → created an issue.

Hi @mkinnune,
I applied patch (smart_date-repeat-end-date-3413612-0.patch) and performed the steps to reproduce the issue. I can verify that the validation works. Please see attached photo.
Thank you

Create a patch to center the button text.

Created patch to fix this issue

Gab. → created an issue.

This patch worked for me.

Hi alok_singh
I've applied your changes from branch (yg_corporate-3411284) and can confirm that the Twitter logo is updated. Please see attached file.
Thank you

Hi @alok_singh

I've applied the changes from branch (sidus_travel-3408366) and can confirm that the read more is now properly themed. However, there is a blank button which need to be themed. Please see attached image.

Thank you

Hi @alok_singh

I've applied the changes from branch (sidus_travel-3408667) and can confirm that the pagination is now properly themed. Please see attached image.

Thank you

Hi @alok_singh

I've applied the changes from branch (sidus_travel-3408668) and can confirm that the footer is now properly themed. Please see attached image.
Thank you

Hi @sakthi_dev

Looks like patch subscript-links-3391922-12.patch #12 is working great. Please see attached images.

Thank you

Hi chetan 11

I've pulled and tested you're merge request. Twitter logo is updated.

Thank you

Hi @shweta_sharma,

I've reviewed and tested your branch changes on d8_blog_theme-3391243, and can confirm that the spacing is fixed. Please see attached image.

Thank you

Hi @shweta_sharma,

I've reviewed and tested your branch changes on d8_blog_theme-3391285, and can confirm that the alignment is fixed. Please see attached image.

Thank you

Hi kksandr,
I've applied the changes from branch (3390996-fix-css-errors) and can confirm that the css errors were addressed, also the notification widget was not affected. Please see attached image.
Thank you

Hi bobi-mel,

I've tested the patch and can confirm that the padding was added. Please see attached image.

Gab. → created an issue.

Gab. → created an issue.

Gab. → created an issue.

Hi Gaurav Gupta,

I was able to reproduce the error upon deletion of login destination. Also checked on your commit and applied it as patch for review, the issue was resolved. Please see my attachment.

Thank you

Hi alok_singh,

I've reviewed your changes made on branch (fepper-3389720) and applied it as patch. I can confirm that the space is on the search button on mobile view. Please see attached file.

Thank you

Hi alok_singh,
I've applied your changes from branch (fepper-3389718) and can confirm that the space is added username. Please see attached file.
Thank you

Hi @carolpettirossi I've reviewed your patch (3375915-creating_block_form_errors_scroll-3.patch) and can confirm the modal scrolls up when an error on the form occurs. Please refer to the attached image for reference. Appreciate your efforts!

Zed9 → made their first commit to this issue’s fork.