Skip to content

Improve role select when editing membership#2265

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:improve-role-dropdown
Oct 13, 2017
Merged

Improve role select when editing membership#2265
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
spadgett:improve-role-dropdown

Conversation

@spadgett
Copy link
Member

@spadgett spadgett commented Oct 13, 2017

  • Only match role names to avoid false positives
  • Remove unnecessary sort since it's sorted in the controller
  • Add a track by to the repeat
  • Highlight search matches in the dropdown

openshift web console 2017-10-13 12-06-07

Fixes #2260

* Only match role names to avoid false positives
* Remove unnecessary sort since it's sorted in the controller
* Add a `track by` to the repeat
* Highlight search matches in the dropdown
@spadgett
Copy link
Member Author

/assign @benjaminapetersen

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Oct 13, 2017
@benjaminapetersen
Copy link
Contributor

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 13, 2017
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue.

@openshift-merge-robot openshift-merge-robot merged commit 4c16ab5 into openshift:master Oct 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants