Hello @zenimagine,
Regarding the Bootstrap Bario 5 theme, I noticed that Bootstrap 5 Tooltip support is not enabled by default. To enable it, please refer to the official documentation here: [Bootstrap 5 Tooltip Documentation](https://getbootstrap.com/docs/5.2/components/tooltips/). You'll need to add the specified 2-line JavaScript code to your JS file. For Ref attached screenshot(File Name: SS1) and video with working tooltip.
If this solution will solve the issue the please proceed to update the ticket status to "Closed."
Note: This feature is subjective, so it requires manual activation. If needed,
@Ravi Kant,
Upon reviewing the subtheme of the Barrio Bootstrap 5 theme, I observed that the library.yml file includes an inbuilt CDN with bootstrap.bundle.min.js, and the local already contains popper.min.js. It seems there is no requirement to update the library.yml file. For more details, please refer to the attached screenshot(File Name: SS2). Kindly verify
Adding Patch to resolve overlap issue.
Please review.
For ref attached screenshot on prev comment.
pradipmodh13 → created an issue.
Adding patch to resolve button ovelapping.
Please review.
For ref attached screenshot.
Hello @saschaeggi,
But dropbutton.scss file comes from theme folder not from paragraph module.
After some research, there are two solutions found which is listed below.
- Add css in vertical-tabs.css file (Attached Screenshot)
Note: vertical-tabs.css is coming from core claro theme
- Add special css in paragraph module with tabs varient
Please let me know if you have any suggestions for a better approach.
Thank You @gromani14,
Yes, there is an issue when the paragraph type is set in Tabs.
The solution offered by @bbombachini works when I apply it, but we need to develop a patch using the "dropbutton--multiple" class because the "dropbutton--extrasmall" class is varinet of dropbutton size and may be configurable.
This approach only resolves one type if the Paragraph Type field in the content type Managre From Display is set to "Paragraphs Legacy" (as shown in the attached screenshot - 3336167-after.png ).
If we choose "Paragraphs (Stable)," this solution will fail, so will submit the patch. (Attached screenshot in Issue summary)
Hello @gromani14,
I attempted to replicate this problem following your steps, but it is not reproducible.
It is working fine as expected.
Drupal 9.5 and Gin Theme 8.x-3.0-rc2
I had followed the below steps
- Installed Gin theme and enable as admin
- Installed Paragraph module and enable
- Create 10 Paragraph types and configure on content type
- Edit the page and all 10 Paragraph types are visible (Attached screenshot)
After looking further,
I have made the patch for same and it is working fine as expected.
For ref attached recording.
please review
Working on this ticket
Hello @huzooka,
I have generated a new patch in accordance with D9 because your patch is in D8 and the ticket is for D9, thus the patch is not applying.
There is a better solution presented for screens less than 16.5 em since if we attempt with z-index, the other options won't be seen.
If we try with z-index then other option will not visible so provided one better solution for below 16.5 em screen
In response to a request, I've included CSS with a specific focus on breakpoints below 16.5 em to achieve narrow viewport design.
Since everything is working fine on above 16.5 em, it is a good idea to limit CSS to max-width: 16.49 em so that it won't conflict with any existing code.
I tried this patch with these admin theme Bartik, Claro, Olivero, Seven, and stark and it working fine as expected.
Note: Also this patch is working fine with with admin toolbar module.
Please review.
For ref attached screenshot.
#4 patch applied successfully. It is working fine as expected.
For ref attached video with before and after.
We can move to RTBC.
Thanks !
Reviewing this ticket.
After looking further, it was discovered that the anchor tag's href property was missing, thus href was added in accordance with usual practise.
After applying the fix, I verified that the lighthouse reports were error-free.
Please review
Hello, Closing this ticket as it is already created.
Ticket no:
https://www.drupal.org/project/admin_toolbar/issues/3357884
🐛
On Hover : search links items not visible due to background color
Fixed
Hello @amit.mall,
Your patch is getting failed so changing the status from needs review to needs work.
pradipmodh13 → created an issue.
3809.patch has been successfully applied.
Now toolbar is fixed and working as expected in mobile.
For ref attached recording.
I tested with Drupal 9.5 and 10.1, sidebar content scrolled up fully towards up.
For ref attached video.
I followed the procedures, however the outcome is the same as with the seven then claro theme.
I didn't experience any icon scaling problems.
Please see the following screenshot for reference.
Fixed #126 CCF
Note:
Please Ignore my patch #127, #128 and #129
Fixed #126 CCF
Note:
Please Ignore my patch #127 and #128.
Fixed #127 CCF.
Fixed #126 CCF.
Hey @smustgrave,
If my patch is good then please change the status from needs work to needs review.
Removed display:block css from base file and it is better solution as img is inline level element.
For ref attached before and after screenshot.
Please review.
Hello, @kostyashupenko.
You are correct. The gap between two radio buttons works great with a single column, however when we set two, three, or five columns, the spacing is removed by webform CSS.
We can get our layout space and active radio button will not overlap by eliminating margin-bottom css.
So we should edit webform CSS rather than the olivero theme file.
Please see the following screenshot for reference.
@all
Please suggest me how can I submit the patch of webform contrib module. ?
Hey @smustgrave,
Img is inline level element and in Olivero theme img tag css comes with 'display:block' css property from base.css file so I think we can remove display:block property.
For ref adding screenshot with display block and display inline property.
Can I remove property from base.css file and submit the patch ?
Applied Patch. It's working fine.
For ref attached screenshot.
Hello @bnjmnm,
Yes, you are correct. The ui-dialog box will be hidden on the vertical toolbar for default theme stark.
To resolve this issue, I added a z-index to the ui-dialog box using js and verified that it works properly with all of the themes listed below.
- Default Theme / Stark
- Olivero
- Bartik
- Seven
- Claro
I've included a screenshot for reference.
Please let me know if there's anything I can do better.
Hello @smustgrave,
As requested I am attaching here after and before screenshot.
Admin design looks fine after applying patch.
Hello Folks,
In Drupal 10.1.x-dev anchor tag comes with href attribute. So no need to work.
For ref attached screenshot. So we can move this bug to needs review.
Hello Folks,
In Drupal 10.1.x-dev anchor tag comes with href attribute. So no need to work.
For ref attached screenshot.
Applied patch #96 applied successfully in drupal-10.1.x-dev.
We can move this ticket to RTBC.
Thanks for the patch.
Attached screenshot.
Hello @smustgrave,
Can you please share steps to check this error message.
It will easy for me to share screenshot with After and before.
Successfully Patch Applied #19.
It's working fine as expected.
Attached screenshot.
Applied Patch #12.
It is working fine.
#36 Patch applied successfully.
For ref attached screenshot.
#15 patch is not applied succesfully.
Hello @andy-blum,
Added gap property for button layout to handle global implementation.
Please review.
Hello @andy-blum,
Added flex-wrap: wrap in ui-dialog-buttonset for global implementation.
Please review.
Applied patch #15 but after applying layout was borken.
For ref attached two recording.