Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Default close button in chips jumps to next line if space is not enough #4264

Closed
jualvarez opened this issue Aug 20, 2015 · 1 comment
Closed

Default close button in chips jumps to next line if space is not enough #4264

jualvarez opened this issue Aug 20, 2015 · 1 comment
Assignees
@topherfangio
Labels
pr: merge ready This PR is ready for a caretaker to review type: bug
Milestone
1.0-rc1

Comments

@jualvarez
Copy link

Here is a screenshot from the demo

image
@topherfangio
Copy link
Contributor

Heh, that's kinda funny looking. We'll see what we can do to fix it :-)

@topherfangio topherfangio self-assigned this Aug 20, 2015
@topherfangio topherfangio added this to the 0.12.0 milestone Aug 20, 2015
topherfangio added a commit that referenced this issue Aug 21, 2015
@topherfangio
fix(chips): Fix styling issues with padding & remove.
37b6f2d 
Update SCSS to fix:
 * remove icon jumping to next line on long chips
 * right padding of chips without a remove icon
 * location of remove icon in Firefox/Safari

fixes #4264, fixes #3276, #2410
@topherfangio topherfangio mentioned this issue Aug 21, 2015
Closed
topherfangio added a commit that referenced this issue Aug 21, 2015
@topherfangio
fix(chips): Fix styling issues with padding & remove.
3676c90 
Update SCSS to fix:
 * remove icon jumping to next line on long chips
 * right padding of chips without a remove icon
 * location of remove icon in Firefox/Safari

fixes #4264, fixes #3276, fixes #2410
@topherfangio topherfangio added the pr: merge ready This PR is ready for a caretaker to review label Aug 21, 2015
kennethcachia pushed a commit to kennethcachia/material that referenced this issue Sep 23, 2015
@topherfangio @kennethcachia
fix(chips): Fix styling issues with padding & remove.
aed1338 
Update SCSS to fix:
 * remove icon jumping to next line on long chips
 * right padding of chips without a remove icon
 * location of remove icon in Firefox/Safari

fixes angular#4264. fixes angular#3276. fixes angular#2410. closes angular#4275.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@topherfangio topherfangio

Labels
pr: merge ready This PR is ready for a caretaker to review type: bug
Projects
None yet
Milestone
1.0-rc1
Development

Successfully merging a pull request may close this issue.

fix(chips): Fix styling issues with padding & remove. angular/material
2 participants
@topherfangio @jualvarez