- ๐ฎ๐ณIndia Preeti.chawla
Hi,
I checked both issues on the 11.x-dev version. The breadcrumb display on mobile was not working correctly, but itโs functioning well after applying the patch. However, I couldnโt reproduce the primary issue; it works fine without any patch. So Iโve created a merge request for patch #33.
- ๐ฎ๐ณIndia Preeti.chawla
preeti.chawla โ made their first commit to this issueโs fork.
- ๐ฌ๐งUnited Kingdom rick bergmann
The patch in #90 ๐ Field Groups marked as required are missing red asterisk Needs work works for me on Drupal 10.3. I had to apply the config workaround to remove the duplicate asterix, however there is a scenario that doesn't work.
In 1 of my fieldgroups I have a fieldset which contains a required field. In that case there was no duplicate asterix and I had to keep the `Mark group as required...` setting enabled.
So the result is I have 2 field groups with `` and 1 field group with `` which is visually noticeable.
- @arunkumark opened merge request.
- @arunkumark opened merge request.
- First commit to issue fork.
- ๐ฎ๐ณIndia arunkumark Coimbatore
Created drafted Change Record โ . Feel free to update the change record if needed.
https://www.drupal.org/node/3484054 โ Hello,
I found the issue still persists after installing the seven theme version 1.0.0 in Drupal version 10.3.6. As mentioned in #23 ๐ In ui-dialog box at Views button hover effect is odd Active the theme is removed from Drupal core. The MR provided in this issue was for the core hence I moved the code to the seven theme and raised the MR!13.
I also encountered another problem related to the dialog-box button in/en/admin/content
page. After visiting the content page , I went to delete option for a content. The dialog-box opened with the delete button being displayed in focused form. On clicking anywhere except the delete button I noticed the visibility issue in the delete button. I have fixed this issue in MR!13. Please find the below image for reference.
I've attached the video for the before patch and the mentioned issue.
Please review.- @esha_kundu opened merge request.
- First commit to issue fork.
- ๐ฎ๐ณIndia nayana_mvr
Can someone please confirm if this ticket is still relevant? I found another ticket https://www.drupal.org/project/drupal/issues/3396523 ๐ node.module.css is obsolete and can be removed Active in which the file
node.module.css
will be removed since it is no longer used. This is the same file which is being updated in the last patch submitted here #3 ๐ Make two column node CSS reusable Active . Also, noticed that classes in Block layout UI are different from the one attached in last comment. - ๐ท๐ธSerbia finnsky
Thank you!
CSS changes looks good to me.
Also tested basically and didn't found visual regressions.+1 RTBC. Would be cool if anyone can make before/after screenshots.
- ๐ท๐ธSerbia finnsky
finnsky โ changed the visibility of the branch 3303546-claro-dialog-refactor-d11 to hidden.
- ๐บ๐ธUnited States Stanzin
I am requesting a REVIEW for a new MR. Thanks )
Automatically closed - issue fixed for 2 weeks with no activity.
Automatically closed - issue fixed for 2 weeks with no activity.