Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Fieldcontain: Reset width for notext buttons #7525

Closed
wants to merge 1 commit into from

Conversation

Shepard
Copy link
Contributor

@Shepard Shepard commented Jun 27, 2014

Buttons with .ui-btn-icon-notext normally have a fixed width of 1.75em.
However when putting them as inline buttons inside a .ui-field-contain, from a certain page width on they get a width:auto, making them use the available width on the page instead of staying at a fixed width.
This overrides the width:auto with the 1.75em again in that situation.

Buttons with .ui-btn-icon-notext normally have a fixed width of 1.75em.
However when putting them as inline buttons inside a .ui-field-contain, from a certain page width on they get a width:auto, making them use the available width on the page instead of staying at a fixed width.
This overrides the width:auto with the 1.75em again in that situation.
@Shepard
Copy link
Contributor Author

Shepard commented Jun 27, 2014

Sorry, I was a bit too fast in submitting this and didn't check the guidelines for pull requests. I will sign the CLA and submit a bug report as well.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.16%) when pulling 4969ec0 on Shepard:patch-1 into 14c712f on jquery:master.

@Shepard
Copy link
Contributor Author

Shepard commented Jun 27, 2014

Submitted a ticket: #7526
Description in my pull request is a bit wrong, the inline button actually gets a width of 0 instead of using the available space.

@Shepard
Copy link
Contributor Author

Shepard commented Jun 27, 2014

And I've signed the CLA now.

@jaspermdegroot
Copy link
Contributor

@Shepard - Thanks for the PR! I noticed you've signed the CLA with a different email address than you use in your git config. Can you please sign again using your Git config email address? Thanks!

@arschmitz - I reviewed and tested this and we should land this change. It's an addition to 7c4eb00 where I added the width override for inline buttons. Just haven't thought of icon-only buttons at that time.

@jaspermdegroot jaspermdegroot self-assigned this Jun 30, 2014
@jaspermdegroot jaspermdegroot added this to the 1.4.4 milestone Jun 30, 2014
@Shepard
Copy link
Contributor Author

Shepard commented Jul 4, 2014

Sorry for the delay, I signed the CLA with the other email address as well now.

@arschmitz
Copy link
Contributor

Yeah this looks good to me too we just need to make sure to rebase to fix the commit message when we land this 👍

@arschmitz arschmitz closed this in 2dd9cf4 Aug 22, 2014
arschmitz pushed a commit that referenced this pull request Aug 26, 2014
Buttons with .ui-btn-icon-notext normally have a fixed width of 1.75em.
However when putting them as inline buttons inside a .ui-field-contain,
from a certain page width on they get a width:auto, making them use the
available width on the page instead of staying at a fixed width.
This overrides the width:auto with the 1.75em again in that situation.

Fixes gh-7526
Closes gh-7525

(cherry picked from commit 2dd9cf4)
agcolom pushed a commit to agcolom/jquery-mobile that referenced this pull request Nov 26, 2014
Buttons with .ui-btn-icon-notext normally have a fixed width of 1.75em.
However when putting them as inline buttons inside a .ui-field-contain,
from a certain page width on they get a width:auto, making them use the
available width on the page instead of staying at a fixed width.
This overrides the width:auto with the 1.75em again in that situation.

Fixes jquery-archivegh-7526
Closes jquery-archivegh-7525
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants