Skip to content

feat(select): Allow select options in popover to be individually styl…#17305

Closed
audaxion wants to merge 90 commits into
ionic-team:masterfrom
audaxion:ion-select-improvement
Closed

feat(select): Allow select options in popover to be individually styl…#17305
audaxion wants to merge 90 commits into
ionic-team:masterfrom
audaxion:ion-select-improvement

Conversation

@audaxion

Copy link
Copy Markdown
Contributor

…eable.

Short description of what this resolves:

Allows individual select options in a select popover to be styled separately.

Changes proposed in this pull request:

  • Add cssClass to SelectPopoverOption and apply it to ion-label in the popover.

Ionic Version: 4.0.0

@ionitron-bot ionitron-bot Bot added package: angular @ionic/angular package package: core @ionic/core package labels Jan 29, 2019
@brandyscarney

Copy link
Copy Markdown
Member

Thank you for the PR! This looks great, but would it possible to instead grab the class and pass it? Like the following:

<ion-select-option class="my-class">red</ion-select-option>

I'll have to look into this more but wanted to comment first. 🙂

@brandyscarney brandyscarney self-requested a review February 22, 2019 22:54
@audaxion

Copy link
Copy Markdown
Contributor Author

I like that suggestion. I've updated the PR to use class instead of the new property cssClass.

brandyscarney added a commit that referenced this pull request May 14, 2020
@brandyscarney

Copy link
Copy Markdown
Member

I'm sorry it's taken so long for me to get back to this. I went ahead and implemented it on a new PR instead of trying to fix the conflicts with master: #21304

I added you as a co-author though so you still get the contribution: 5285824

This will go into our 5.2 release. Thank you for your work on this! 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: angular @ionic/angular package package: core @ionic/core package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants