-
Notifications
You must be signed in to change notification settings - Fork 467
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
Enable different item label for selected item #892
Conversation
Codecov Report
@@ Coverage Diff @@
## master #892 +/- ##
==========================================
+ Coverage 95.02% 95.03% +<.01%
==========================================
Files 124 124
Lines 2172 2174 +2
==========================================
+ Hits 2064 2066 +2
Misses 108 108
Continue to review full report at Codecov.
|
Should fix #845 |
Looks good for me ! We just need some tests in this PR. |
@@ -21,6 +21,7 @@ def get_results(self, context): | |||
{ | |||
'id': self.get_result_value(result), | |||
'text': self.get_result_label(result), | |||
'selected_text': self.get_selected_result_label(result), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is worth adding this field only if it is different from text
. Now one text is sent in two fields even when it is identical.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's true but we're going to merge this as is so if you got hard feelings about this then perhaps make another pr or open a ticket 😂
Will be in next release. |
This pull request adds ability to display selected result (in select field) in different way than in dropdown e.g. display detail item in dropdown but only name in field after selection.