Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
We should apply position to a single parent node, not each child elements.
In current CORE design, we have no problem. But if custom code modifies the trigger area a bit, it may not work. And with applying to parent, we stripped few more lines and KISS.
Comment | File | Size | Author |
---|---|---|---|
#12 | apply-position-on-parent-element-2902308-12.patch | 3.7 KB | Ramya Balasubramanian |
#9 | 2902308-9.patch | 2.35 KB | komalk |
#3 | after-patch-button.png | 1.51 MB | msankhala |
#3 | after-patch-parent-div.png | 1.47 MB | msankhala |
#3 | before-patch-button.png | 1.5 MB | msankhala |
Comments
Comment #3
msankhala CreditAttribution: msankhala as a volunteer and at Srijan | A Material+ Company commentedThis patch applies cleanly. Steps I followed:
Even though the provided patch is for 8.5.x but this cleanly applies to 8.6.x as well.
left/right/top
is applied to the contextual button wrapper.right
is also applied to actual contextual button.left/right/top/top
is only applied to the contextual button wrapper. No position values are applied to actual button.Before:
After:
Comment #9
komalk CreditAttribution: komalk at Srijan | A Material+ Company for Drupal India Association commentedPatched #2 failed to apply on 9.1.x-dev. Rerolled the patch and fixed the failed tests.
Comment #11
Ramya Balasubramanian CreditAttribution: Ramya Balasubramanian at Srijan | A Material+ Company for Drupal India Association commentedComment #12
Ramya Balasubramanian CreditAttribution: Ramya Balasubramanian at Srijan | A Material+ Company for Drupal India Association commentedFixing the test cases. uploaded a patch again
Comment #14
Ramya Balasubramanian CreditAttribution: Ramya Balasubramanian at Srijan | A Material+ Company for Drupal India Association commentedI have tried this, but can't able to fix this and also I have checked here they have already mentioned about the chrome issue https://git.drupalcode.org/project/drupal/-/blob/9.1.x/core/modules/layo.... Not sure about whether this is a known issue or not. So I am unassigning this ticket.
Comment #18
larowlanI'm not sure why we're making this change
Can we get an issue summary update here detailing what the change is and why it's needed?
Comment #22
smustgrave CreditAttribution: smustgrave at Mobomo commentedIf still a valid task please reopen updating issue summary per #18
Thanks!