Opened 12 years ago
Closed 12 years ago
#23629 closed enhancement (fixed)
Twenty Thirteen: Form elements aren't equal when on mobile
Reported by: | karmatosed | Owned by: | lancewillett |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 3.6 |
Component: | Bundled Theme | Keywords: | needs-testing needs-refresh has-patch |
Focuses: | Cc: |
Description
I've attached an image which is due to a px fixed width:
#commentform input[type="text"] { width: 270px; }
This could become a % on mobile to get it more even.
I'm marking as enhancement as would need a check with the designer before applying anything as to if they want to.
Attachments (4)
Change History (16)
#1
@
12 years ago
- Summary changed from Form elements aren't equal when on mobile to Twenty Thirteen: Form elements aren't equal when on mobile
#5
@
12 years ago
I've just updated with a patch for percentages. We may want to adjust this a bit but it puts the foundation in and gives us the option of varying as it scales down.
#6
@
12 years ago
- Keywords needs-testing added
Thanks for the patch! I'll be testing this early next week.
#7
@
12 years ago
I like the direction of the patch! I think we can cut down on a lot of selectors if we set it to a width of 60% and give it a max-width of 270px per default. What do you think?
#8
@
12 years ago
I'm not sure that gives the equal length we want in mobile. Ideally that would be 100%. It's tricky I understand wanting to have less selectors also.
#9
@
12 years ago
You're right. We could add a selector withwidth: calc(100% - 100px);
in the 643px media query. This would leave us with the 60% as a fall back for IE 6-8, which is good enough IMO.
#10
@
12 years ago
- Keywords needs-refresh has-patch added; needs-patch removed
@karmatosed Can you update the patch for us? Would be great to get this in.
@karmatosed I think it'd be a great improvement to make this percentage based widths instead of pixels. Do you want to cook up a patch for it?