Elkfox/shopify-theme-framework

Quantity elements should not be input field

Closed this issue · 4 comments

With input fields, we have to wrestle with the user agent styling of text and number inputs.

Cam commented

Interesting idea! A fallback would be more difficult.

A fallback would be more difficult

In what way?

Cam commented

If it falls back to an input field, the users can still add quantities without requiring JS. Maybe that doesn't matter though, what were you thinking the replacement would be?

Good point, I mostly don't like it being a number field. User agent styling on some browsers has not been reset and you get the up and down buttons. But it also shouldn't really be a text field/editable.
Could use noscript as a fallback but that is ugly...

I don't want Cartfox to have to rely on an input field value either, on ajax templates it doesn't make much sense to be able to edit the number.