- Issue created by @atowl
- Merge request !3983437516 - Make chosen-single drop downs the same size as other drop downs → (Open) created by atowl
- Assigned to atowl
- Status changed to Needs work
3 months ago 5:24pm 3 April 2024 - 🇮🇳India djsagar
Not able to reproduce this issue, Please provide More detail.
- Status changed to Needs review
3 months ago 11:28pm 3 April 2024 - 🇮🇳India ehsann_95
@atowl Can't we replace the padding value on the same class instead of shifting the padding values to the span?
- Merge request !4013437516 -Replace padding values to match form element. → (Open) created by Unnamed author
- 🇳🇿New Zealand atowl
Hi @ahsannazir,
Yes, looks like that works as well. i think using the variable(s) --input-padding-vertical is better.
So the question now is really, which is more correct? to place the padding on the class itself or on the element within the class.
Happy either way, I'll let the maintainer decide. - 🇮🇳India ehsann_95
Yes @atowl both works. I'd prefer the original way like it was before. Replacing the values on the class itself
- Status changed to RTBC
2 months ago 2:21am 24 April 2024 - 🇳🇿New Zealand atowl
Changing status to reviewed and tested,
Maintainer can choose which fork they want to include.