Skip to content

fix(select): option not truncated correctly in multiple mode - #7857

Merged
tinayuangao merged 1 commit into
angular:masterfrom
crisbeto:7211/select-truncation
Dec 1, 2017
Merged

fix(select): option not truncated correctly in multiple mode#7857
tinayuangao merged 1 commit into
angular:masterfrom
crisbeto:7211/select-truncation

Conversation

@crisbeto

Copy link
Copy Markdown
Member

Currently, when the text of an option is truncated, it ends up removing everything except the ellipsis on Firefox, IE and Edge. These changes fix the truncation and simplify the option view. This seems to have started happening after the switch to mat-form-field.

Relates to #7211.

@crisbeto
crisbeto requested a review from kara as a code ownerOctober 17, 2017 17:56
@googlebotgooglebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 17, 2017
@crisbeto

Copy link
Copy Markdown
MemberAuthor

Also bonus fun fact: Apparently Pokemon names can't be longer than 10 characters.

@mmalerbammalerba added pr: lgtm action: merge The PR is ready for merge by the caretaker and removed pr: needs review labels Oct 17, 2017
@mmalerba

Copy link
Copy Markdown
Contributor

I assume that fun fact is no accident, makes things easier on Nintendo developers ;)

@crisbeto
crisbetoforce-pushed the 7211/select-truncation branch from 372c518 to 6ccdca6CompareNovember 19, 2017 08:45
Currently, when the text of an option is truncated, it ends up removing everything except the ellipsis on Firefox, IE and Edge. These changes fix the truncation and simplify the option view.
Relates to angular#7211.
@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-botangular-automatic-lock-botBot locked and limited conversation to collaborators Sep 7, 2019
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: mergeThe PR is ready for merge by the caretakercla: yesPR author has agreed to Google's Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@crisbeto@mmalerba@kara@googlebot@tinayuangao