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.
Follow-up to #2611268: Link field on node edit causes horizontal scrollbar
Email fields (when set to a multiple value field) causes a horizontal scrollbar at certain widths.
See screenshots
Comment | File | Size | Author |
---|---|---|---|
#20 | 2710743-19.patch | 1.42 KB | djsagar |
#10 | interdiff-10.txt | 962 bytes | imalabya |
#10 | 2710743-10.patch | 1.02 KB | imalabya |
Comments
Comment #2
kostyashupenkoComment #3
kostyashupenkoPatch here. Changes below
width 768px
width 320px
Comment #4
Manjit.SinghThis is not working for me when sidebar/toolbar is opened. Please check the screenshot.
Anyone else facing the same issue ?
Comment #5
kostyashupenko@Manjit.Singh, screen in the description of task - is a Bartik theme, but your screen comes from Seven theme. But yea, i can confirm your bug for Seven. Do i need to solve this issue here in this task?
Comment #6
Manjit.Singhoopppsss :( completely forget that where i was checking.
And yeah we can solve this issue for seven, but we have to create another issue for it. Because This is Bartik specific issue.
Comment #7
emma.mariaThis is a Bartik issue. We can no longer put fixes for both Bartik and Seven in the Classy theme so we need separate issues.
Comment #8
emma.mariaThis selector is too specific, it affects all input elements. We only want to target the text and email input fields within "field-multiple-table" component.
Comment #9
emma.mariaRegarding #7 I change my mind. As it's a small fix I'm moving this to the CSS component and specifying it's a fix for both Core themes.
Comment #10
imalabyaAdded a patch which targets only "text" and "email" field.
For Seven theme, the width had to be adjusted for iPads(768px)
Comment #11
imalabyaComment #19
djsagar CreditAttribution: djsagar at OpenSense Labs commentedNee work for 8.9.x-dev.
Comment #20
djsagar CreditAttribution: djsagar at OpenSense Labs commentedCreating patch for 8.9.x-dev.
Please review.
Thanks!
Comment #21
Madhu kumar CreditAttribution: Madhu kumar as a volunteer and at Zyxware Technologies commentedApplied patch #20 cleanly and fields within the table markup are working as expected.
RTBC
Comment #22
radheymkumar CreditAttribution: radheymkumar commentedApplied patch #20 success
Thanks
Comment #23
bnjmnmRemoving credit for #22.
Comment #25
vikashsoni CreditAttribution: vikashsoni as a volunteer and at Zyxware Technologies commentedApplied #20 patch in drupal-9.3.x-dev
Successfully applied and looks good for me
For ref sharing screenshot ............
Comment #28
Manibharathi E R CreditAttribution: Manibharathi E R at Srijan | A Material+ Company for Srijan | A Material+ Company commentedPatch #20 Applied success fully on Drupal 9.4.x.
Steps to Reproduce:
1. Install Drupal Instance
2. Create One plain text with Multiple Allowed Values
3. Add new content in the Mobile view
4. Click on the Manage Button. You can able to reproduce the issue.
Comment #29
gaurav-mathur CreditAttribution: gaurav-mathur at Dotsquares Ltd. commentedComment #30
gaurav-mathur CreditAttribution: gaurav-mathur at Dotsquares Ltd. commentedPatch #20 successfully applied and looks good for me.
Comment #32
mherchelMoving this over to the Seven project, as it's no longer in Drupal core.