Account created on 30 September 2011, about 13 years ago
#

Recent comments

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I am getting the same error and i am using the ddev.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I have tested the #4 now the focus state is visible. Attaching a screenshot for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I have tested the above #7 patch the breadcrumb icon and back icon are visible properly. Attaching a screenshot for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I have tested the #2 and this is working as expected.

I have noticed !important is applied in certain areas, yet the icons are also visible without the need for !important. So I am removing the !important from the patch.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

@bnjmnm
It looks fine then attaching a screenshot for reference. I have noticed the message icon is not visible so I have made fixes in the MR.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I am not able to reproduce this. Attaching a screenshot for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

@bnjmnm
I have tested the code the cross icon is there but not visible properly. I am attaching a screenshot for reference.

I also noticed the status icon is not visible.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

The reason for this problem is that the primary-navigation library is only attached to the main menu template, causing the issue. And if there is no link in the main menu then the main menu block is not rendered because of this library is not attached. I think we need to attach library globally to fix this issue.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

The above patch working fine but only for the Legacy, Classic Drupal Toolbar not for the Horizontal, Modern Toolbar. Also, there is also some issue with the tablet screen. Attaching a screenshot for reference.

Adding patch with changes.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

Mukeysh โ†’ made their first commit to this issueโ€™s fork.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

Applying a patch to fix the above issue. Also attaching a screenshot for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I have tested locally and the image is optimized as mentioned above

Attaching screenshots for reference.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

in #3 @ravi kant mentioned web form provides the default state for read-only and disabled but this CSS comes from the webform module which is the contrib module.

We need ready-only styling in the claro theme as well.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

I am adding a patch for the above fix. I have noticed that the system module does not contain the files system.theme.css and system.menus.css. However, the file system.admin.css is present. Therefore, I have removed the system.theme.css and system.menus.css files from the patch.

๐Ÿ‡ฎ๐Ÿ‡ณIndia mukeysh

Adding a patch to fix the issue.

Production build 0.71.5 2024