villebro opened a new pull request #14896:
URL: https://github.com/apache/superset/pull/14896


   ### SUMMARY
   Harmonize AntD Select components that are used in Explore/Dashboard view. 
This changes the following:
   - Changes AdHoc Filter Select to show the `StopOutline` Icon if "NOT IN" 
operator is selected. This is to align the behavior with the native select 
filter polish introduced in #14873
   - Show "Create: <value>" instead of "<value>" when given the option to 
create a select option that doesn't exist in the dataset. This is to mimic the 
behavior of the `react-select` based component used in Filter Box: 
![image](https://user-images.githubusercontent.com/33317356/119962068-591a2600-bfaf-11eb-903e-7dd837af7795.png)
   
   ### BEFORE
   When "NOT IN" is selected in the adhoc filter popover, we currently use the 
default `CheckOutlined` icon:
   
![image](https://user-images.githubusercontent.com/33317356/119961520-d4c7a300-bfae-11eb-9178-fe50cf0957e2.png)
   When being presented with the option to create a value that doesn't exist in 
the dataset, the option to use a value that doesn't exist is shown as-is:
   
![image](https://user-images.githubusercontent.com/33317356/119962266-88c92e00-bfaf-11eb-968d-7e7e6fbbfe3b.png)
   
   
   ### AFTER
   When "NOT IN" is selected in the adhoc filter popover, we now use the 
`StopOutlined`
   
![image](https://user-images.githubusercontent.com/33317356/119960095-47d01a00-bfad-11eb-81c0-cce508d98212.png)
   When being presented with the option to create a value that doesn't exist in 
the dataset, The creatable value is prefixed with "Create" and the value placed 
inside quotes:
   
![image](https://user-images.githubusercontent.com/33317356/119961707-06406e80-bfaf-11eb-8b1e-e9aae7d0442f.png)
   
   
   
   
   ### TESTING INSTRUCTIONS
   <!--- Required! What steps can be taken to manually verify the changes? -->
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to