Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

select: increasing font size of select increases font size of options, reseting it back to base value messes up positioning #7637

Open
fxck opened this issue Oct 8, 2017 · 2 comments
Labels
area: material/select P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent

Comments

@fxck
Copy link
Contributor

fxck commented Oct 8, 2017

https://stackblitz.com/edit/angular-material2-issue-d1e88y?file=styles.scss

select third option, close, open select again

a) it didn't increase font size of option pre form-field
b) positioning was working correctly, despite different font size, pre form-field

cc @crisbeto

@fxck
Copy link
Contributor Author

fxck commented Oct 8, 2017

In case you are wondering why would I want different item size for options and select

image

there's no reason why would I want the options to be as big as the active option, quite the opposite, it would look pretty bad, and be pretty bad UX, since barely three options would fit into one "screen".

This is what "broke" it d914cc4#diff-102eaa94de98ed2cabd87565ba514125R89 @mmalerba

#7627 and #7638 are both related to the same component, it's nearly impossible to do something like this with material, especially since the mat-form-field stuff, I completely understand components like these are not part of the material spec, but it would be nice if slight style modifications would be even possible, especially when it's just about styling most of the time, not the internal apis.

@mmalerba
Copy link
Contributor

mmalerba commented Oct 9, 2017

I'll take a look and see if we can do the positioning based on the option size instead of the trigger size, just in case people override it like you're doing.

@mmalerba mmalerba self-assigned this Oct 9, 2017
@mmalerba mmalerba added the P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent label Oct 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: material/select P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
3 participants