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

mdButton: default min-width excessive for small buttons #2091

Closed
ngraef opened this issue Mar 30, 2015 · 2 comments
Closed

mdButton: default min-width excessive for small buttons #2091

ngraef opened this issue Mar 30, 2015 · 2 comments
Assignees
@marcysutton
Milestone
0.9.0

Comments

@ngraef
Copy link
Contributor

ngraef commented Mar 30, 2015

A button's min-width added in a936e1e is too wide in certain use cases, including:

  1. A non-fab icon button (demo):
    image
  2. A short text button in an action toast (demo):
    image
@marcysutton marcysutton self-assigned this Mar 30, 2015
@marcysutton
Copy link
Contributor

I think small or icon buttons not in a Toast (or other specific component) will need a special class, since there is nothing differentiating them from flat and raised buttons (which require the min-width).

@marcysutton
Copy link
Contributor

This can be outlined in Toolbar + icon demos #1796

marcysutton pushed a commit that referenced this issue Mar 31, 2015
feat(mdButton): add icon button for toolbars, etc.
2880fdc 
Closes #2091, #1796
@marcysutton marcysutton mentioned this issue Mar 31, 2015
Closed
@ajoslin ajoslin added the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Mar 31, 2015
marcysutton pushed a commit that referenced this issue Mar 31, 2015
feat(mdButton): add icon button for toolbars, etc.
3244144 
Closes #2091, #1796
@marcysutton marcysutton removed the in progress Mainly for in progress PRs, but may be used for issues that require multiple PRs label Mar 31, 2015
@marcysutton marcysutton added this to the 0.9.0 milestone Mar 31, 2015
JulianWielga pushed a commit to JulianWielga/material that referenced this issue Apr 1, 2015
@JulianWielga
feat(mdButton): add icon button for toolbars, etc.
4450191 
Closes angular#2091, Closes angular#1796, Closes angular#2106
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@marcysutton marcysutton

Labels
None yet
Projects
None yet
Milestone
0.9.0
Development

Successfully merging a pull request may close this issue.

Icon button support plus Toolbar and Button demos angular/material
3 participants
@ngraef @marcysutton @ajoslin