Account created on 7 July 2021, about 3 years ago
#

Merge Requests

Recent comments

MR created to resolve issue for edit Node form, I was not able to reproduce issue on block edit form, user edit form.

Attaching video for reference.

Step to reproduce

1.Install and enable Gin theme, set it as default.
2. Open any content and edit . Here I have edited Article.
3. Start using Tab on keyboard, now you can access Sidebar easily.

Hi

Reviewed this issue, works fine after applying MR.
Gin theme version 8.x-3.0-rc13
Drupal version 11.0.5-dev

Steps taken

1. Enable Gin admin theme
2. Create new view with some fields, displayed as a table
3. In Format: Settings select any field as Grouping field Nr.1
4. In the same form enable "Drupal style sticky table headers"
5. Observe dupliacted table headers, hiding name of the first group.

This issue can be moved ahead.
RTBC +

Hi,

Reviewed this issue ,MR 1 applies cleanly.
This resolves issue mentioned in #3 that "Drupal's philosophy that the administrator role should have access to everything".

Drupal version 9
Disabled field version 8.x-2.x-dev
Steps to reproduce were not clear so adding it here

1. Install Drupal and mentioned module version, and enable it.
2. Create a role with limited access , here I have created Editor role.
3. Create a content type with some with field, you will find option to Disable field settings.
4. Select Disable for certain roles and select all roles for reproducing issue at first.
5. Add content and check filed is disabled for admin and editor role(with limited access).
6. Apply Patch 1 and check same field is now enabled for Admin role.
Adding before and after screenshot for Editor and admin roles.

Hence patch works correctly.

RTBC +

I tried to reproduce issue , not able to reproduce.
Drupal Version - 10.3.2 , 10.3.5
Themes - Olivero
As issue is not reproducible adding screenshot for reference.

Please add clear steps to reproduce.|

Hi
Drupal version 11.x-dev
Theme Olivero

I have created MR to address issue for mis alignment. As per suggestions in #17 and #31 it needed to styles single-tab in vertical tabs.
In MR I have added .first and .last classes , both of which appears when there is only one tab(refer after screenshot).
If there are multiple vertical tabs, first tab will have .first class and last tab will have .last class. (Refer screenshot)

Steps to review

1. Enable layout builder module and select Olivero theme
2. Use Layout Builder from Administration > Structure > Content types > Article > Manage display > default > layout options
3. Create Article from Content > Add content > Article and Save
4. Click on Layout tab

Reviewed latest MR ,it applies cleanly and MR resolves mentioned issue.

Drupal version 11.x-dev

1. Create a layout with the Layout Builder.
2. Click on the "Discard changes" button in the Layout Builder.
3. Before MR, it would redirect to /node/[nid]/layout/discard-changes to confirm the dialog
4. After applying MR is opens in dialogue box.
5. Attaching screenshot for before and after.
Issue can be moved forward.
RTBC +

Followed given steps not able to reproduce alignment issue related to row eight field.

Drupal Version 11.x-dev
Paragraphs module version- 8.x-1.18
Themes- Claro , Olivero
Steps to reproduce already mentioned in #15.

Steps followed
1. Created Paragrpahy Type, added fields to it.
2. Add a paragraph type to article content type.
3. Add/edit content in article content type.
4. Check for Olivero and Clro theme.

I have reviewed this patch.
I have applied MR !8893 on drupal 11.x, applies without any issue.

Steps to Reproduce:
1. Enable the Navigation module.
2. Goto the module’s settings page.
3. Choose a custom logo from the logo options.
4. Upload a small logo.
5. Check the horizontal alignment of the logo in the collapsed toolbar state.

The logo position has been centred after the MR changes.

Attaching screenshots for reference.

Status of this issue can me moved ahead

RTBC +

I am not able to reproduce issue , not getting Title dropdown

Steps to followed
1. Added Plain text filed in Article.
2. Configured in manage display , not getting title Drop down in Format column fort "Title".
3. Adding screenshot for reference.

Off canvas styling broken in firefox after latest update.
Firefox version- 126 and above

When trying to use layout canvas on Firefox Title of layout canvas is hiding behind toolbar.
Also there is white spacing at bottom.

Steps to reproduce.

1. On firefox open layout canvas
2. Click on add section, when Layput canvas try opens n right side observe misalignment

I have fixed Spacing issue between Checkbox label and button for Claro theme.

Adding screenshot for reference.

Steps to reproduce.
1 : Fresh install Drupal 11 and use default Claro theme.

2 : Enable layout builder module

3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.

Drupal version 11.x-dev,
Patch applied 36- https://www.drupal.org/project/drupal/issues/2975863#comment-15614291 ✨ Taxonomy overview list with exposed filter Active
Steps followed

1. Create long list of Vocabulary
2. You can use Devel to generate if you don’t have list.
3. This Feature is very useful in finding / filtering terms.
4. It also finds /Filters  hierarchical terms.
5. Placeholder text is helpful it make it clear how to use.

Given MR resolves mentioned issue.
Reviewed and can be moved forward.

Steps to reproduce

1. Observed this issue with light theme(local set up windows/Mac ).
2. Tested on Drupal version 11X and 10.3
3. Install and open site in chrome
4. Inspect and follows steps given in https://devtoolstips.org/tips/en/emulate-forced-colors/
5. This will display page in high-contrast mode.
6. Check icons.
7. Given MR is fixing issue
8. Attaching screenshot for before and after.

Tags overlapping with process icon is seems to be fixed.
Drupal version 11.x-dev
Theme. Claro
Steps

1. Install Drupal 11
2. Set Claro as default theme.
3. Add Article content type.
4. Add tags in it.

This ticket can be moved ahead.

I am not able to reproduce this issue.
Drupal version 10.3.x-dev

Steps taken
1. Checked for themes that use sanitize.css, modern-normalize.css
2. Created Custom theme and added above code, and mentioned files.
3. Not able to reproduce issue on front end. Adding screenshot for reference.

Need more clear steps to reproduce issue

1. Which theme ?
2. Screenshot where it is checked .

As per previous comments changes are now moved to Olivero theme and targeted the ID for that div.
I can confirm on 11.x updated MR addresses the problem. https://www.drupal.org/project/drupal/issues/3463111#comment-15746322 πŸ› Spacing issue between Checkbox label and button Needs review

Adding screenshot for reference.

Steps to reproduce.
1 : Fresh install Drupal 11 and use default Olivero theme.

2 : Enable layout builder module

3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.
5. Issue is fixed with give MR.

Reviewed Issue is fixed.

Steps to reproduce.
1 : Fresh install Drupal 11 and use default Olivero theme.

2 : Enable layout builder module

3 : Use Layout Builder from Administration > Structure > Content types > Basic page > Manage display > default > layout options

4 : Create Basic page from Content > Add content > Basic page and Save
Step5 : Click on Layout tab and you can find the spacing issue between Checkbox label and button.
5. Issue is fixed with give MR.

Hi
Please check added patch for sub menu expand arrow hiding behind the Hamburger.

Attaching screenshots for before and after. Patch submitted in #3.

Steps to reproduce

  1. Install Mediteran administrator version 8.x-1.x-dev.
  2. Create menu with child links, also create child of child link.
  3. Select show as expanded in Parent link.
  4. Inspect and zoom, you can see ">" sign in front of Parent menu link.
  5. As mentioned ">" is coming from svg and has property of style="fill:#fff"
  6. Attaching screenshot for reference .

I have added list style similar to parent of other links.

Hi @shiv_yadav
I have review this issue and just added steps to review, also added before and after screenshots.

Hi
Applied patch from #4 it is working correctly attaching screenshot for reference .
Steps to reproduce
1. Created menu.
2. Add menu in block layout -> Primary menu
3. check on mobile

Hi
I have reviewed this patch.
Patch applied successful and working fine.
Steps to reproduce
1. Install and enable theme.
2. From theme setting enable "Scroll to top".

Hello,
Tried to reproduce issue in mentioned version , I am not able to reproduce. Please mention steps to reproduce. Please refer below screenshot

Added patch for redoc & swagger UI pages (/api-docs/3/redoc & /api-docs/3/swagger). Please review

Hello
I checked this issue, followed all the steps mentioned. Here are my findings "Add Paragraph " button looks like other buttons in Adminal theme. For e.g Place block button in Block layout, Please refer screenshot.

This issue is reproducible in

Drupal 10.1.2

Steps to reproduce

  1. Set Claro as admin theme
  2. Add content which uses Nested Paragraphs
  3. Click show row weights, you can see container overflowing
  4. This issue might be related to https://www.drupal.org/project/paragraphs/issues/3112245 β†’

Applied the patch and working fine, adding screenshot for reference.

Production build 0.71.5 2024