Needs work
Project:
Drupal core
Version:
main
Component:
Claro theme
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Aug 2021 at 12:24 UTC
Updated:
15 Dec 2023 at 12:55 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
katrienca possible solution to this is to add a max-width
Comment #3
katriencok, this only will work off course when the parent is an inline-block element
Comment #4
vikashsoni commentedPatch working fine after patch we able to reduce size for ref sharing screenshot....
Comment #5
chetanbharambe commentedVerified and tested patch #3.
Patch applied successfully and looks good to me.
Testing Steps:
# Goto: Admin -> Structure -> Content types -> Click on manage field and add text field
# Add respective content type
# See the fields
# Goto: admin/structure/types/manage/article/form-display
# Change the size
# Note that the text field is always in full width
Expected Results:
# The text field should be as per size which the user sets in the form display setting.
Actual Results:
# The text field is always in full width.
Please refer attached screenshots.
Looks good to me.
Can be a move to RTBC.
Comment #6
larowlanWould it be better to revert the change from #3151119: Text field with multiple values overflows on narrow screens and instead just set a
max-widthComment #7
gauravvvv commentedPatch updated, Attached interdiff for patch #3, #7.
Please review.
Comment #8
alexverb commentedWith this patch the text field does not seem to resize to the width of it's container when resizing to a width smaller than the textfield size. Unless you go below 600px then form.css:306 puts text fields back to 100% while bigger screen sizes are already set to max-width by form.css.
I also think the initial fix to put it on 100% was incorrect. But I don't know how to ensure that it resizes to the width of the container when going smaller. My guess is that this is an effect of the drag and drop tables as other text fields do not seem to behave that way.
Comment #9
alexverb commentedComment #12
Harish1688 commentedTested the bug not found the issue in (9.4.x-dev to 9.5.x-dev) textfield takes the value and working fine, without any patch.
Every patch use this property (max-width: 100%;), but this property added in the form.css to resolve this.
Comment #13
longwaveThe Seven theme has been removed from Drupal 10 core. However, I confirmed that this issue also affects Claro - text fields with multiple values ignore the "Textfield size" option set in manage form display, so moving this to Claro for further discussion in core.
Comment #16
akhil babuI tried to recreate this issue in claro theme
Steps followed
\
Output
Output
Same as previous case.
Output
Output
Not sure how the width is supposed to change based on the textfield size. However, I believe the current behavior is not correct.