I'll review the MR
hi @rikki_iki,
I've reviewed all the commands you added in the Merge Request and resolved them accordingly. The MR has been updated and is now ready for review. Please let me know if any further changes are required.
Thanks!
Moving the ticket for review now that the pipeline issue has been resolved.
There are some deviations in the MR. I will review them and update the MR accordingly.
Thanks for the update! I've already started work on the footer component, but no issues on my end—I’ll proceed with the review/testing. Let me know if anything else comes up!
I have fixed the issue and raised a merge request for review. I have also attached a screenshot for reference. Please let me know if any further changes are required. Thanks!
I've addressed all the comments and updated the MR accordingly. Moving the issue for review. Let me know if any further changes are needed!
Thank you for the review! I'am currently looking into them and working on fixes.
I’m picking up this issue and will update the status here.
Thank you for the new design update! I have updated the MR based on the new design. Additionally, @mherchel, I have used the element for the accordion.
Thanks for the feedback Jim Birch!
I’ll review the approach using the element and update the component accordingly.
I have created the Accordion component, but a few improvements are still needed. I am actively working on them.
Please review it and share your suggestions, as they would be very helpful.
Thanks!
vasantha deepika → made their first commit to this issue’s fork.
The page seem
I attempted to reproduce the reported issue but found that the functionality is working as expected. The mobile menu button only appears when there are items added to the primary or secondary menu regions. If no menu items are present, the button does not appear, ensuring the expected behavior.
Please let me know if there are specific steps or conditions under which the issue occurs so I can investigate further.
Let me check this issue
vasantha deepika → changed the visibility of the branch 3507482-page-not-visible-fullwidth to hidden.
Create the MR #11251
Apologies for the earlier approach. Since Olivero is a core theme in Drupal 11, I have updated the relevant tags accordingly and will be raising a Merge Request (MR) against Drupal core.
The issue has been resolved, and I have raised a merge request. Please review and provide feedback as needed.
Thanks!
working on this issue
vasantha deepika → created an issue.
hi @saschaeggi,
Based on your feedback, I have updated the merge request (MR ID: #558).
Thanks!
Not able to re produce even i could see the aria-describedby in detail tag.
I attempted to reproduce by installing the Navigation module, but the feature request has already been implemented. Therefore, I am changing the status to 'Closed - (Cannot reproduce)' Please feel free to reopen if further action is required.
As the mentioned issue has already been resolved in the related ticket, I will proceed with closing this issue. Please feel free to reopen if further action is needed.
I have updated MR #372 based on the feedback provided in the comments and changed its status to 'Needs Review' for further evaluation.
@balintbrews Thank you for your feedback; I am updating the MR accordingly.
@boulaffasae Thank you for your clarification!
I need clarification regarding the desktop design—there seems to be a variation in the padding of the card. Could someone confirm which desktop design we should proceed with?
Reverting the image slot back to props due to limitations with the current image implementation.
Thank you, @balintbrews, for your suggestion—it makes the process more streamlined.
Hi,
Thank you for the details. Could you please share the exact steps to reproduce the issue, including your View configuration and behavior in the Views preview? This will help to investigate further.
Hi @saschaeggi,
I have updated MR #527 with the required dist file. Please review the changes.
vasantha deepika → changed the visibility of the branch 3487605-view-for-submit to active.
vasantha deepika → changed the visibility of the branch 3487605-view-for-submit to hidden.
Replace "Local Tasks" with "Quick Links" to make navigation more intuitive and user-friendly for non-technical users as well. Raised an MR #108
vasantha deepika → made their first commit to this issue’s fork.
Hi @jayemel,
I am unable to reproduce the issue you mentioned. Could you please provide more detailed steps to help me replicate it?
Thanks!
Thank you for confirming that the fix works as expected and for the suggestion regarding the initial changes.
Based on your feedback, I have reverted the first two changes and updated the merge request (MR ID: #558).
Thanks!
@jurgenhaas I have fixed the issue with the missing span wrapper and the visual inconsistencies on the initial page load. The span wrapper now applies correctly, ensuring the button appears as expected with the bold text and icon after the page loads.
For reference, I have attached a video demonstrating the resolved behaviour.
Please review the updated merge request (MR ID: #558) and let me know if there are any further adjustments needed.
Thanks!
Thank you for making the merge request visible again and for the detailed feedback!
I will investigate further to determine why the wrapper is not applied after the page loads and work on a fix to address this behavior completely.
1. I have fixed the toggle issue for the 'Show row weights' functionality.
2. Regarding the icon and font weight changes, I investigated further, and these are not theme-based issues. I verified this behavior in the Claro theme, and the same issue persists there.
Please let me know if any additional adjustments are needed or if you'd like me to investigate further.
vasantha deepika → changed the visibility of the branch 3495320-show-row-weights to hidden.
I had the opportunity to volunteer at DrupalCamp Pune 2024, where I served as part of the Speaker Assistance team. My role involved assisting speakers with any requirements, coordinating effectively to ensure seamless transitions, and supporting the preparation and distribution of welcome kits. It was a fantastic experience contributing to such a vibrant community!
#dcp2024 #DrupalCampPune2024
Vasantha Deepika → created an issue.
Fixed the alignment issue and created the patch for it, Please check.
Attached screenshots after fix.
Thanks!
Hi,
Based on the comment #59 I've created a commit for the changes.
Kindly review and let me knows if anything need to be done further.
Thanks!
I fixed the issue mention in comment #54 and re-upload the Patches.
Hi,
The issue is Not because of the patch, it is happening when we are placing the filed inside the field group with Accordion, tab and detailed configuration. Working on this issue.
Thanks!
Vasantha Deepika → created an issue.
The dropdown overlap issue for all the admin pages is fixed and the patch has been created for it.