jinghua-qa commented on pull request #18784: URL: https://github.com/apache/superset/pull/18784#issuecomment-1045156965
I have 2 questions for the new look: 1, the clear all button is always grey out and make me feel like it is disable, also i feel like the buttons are not consistent, for example the clear button did not have a background but apply all have a grey back ground, and it makes me feel like apply is more standout <img width="1792" alt="Screen Shot 2022-02-18 at 12 38 20 PM" src="https://user-images.githubusercontent.com/81597121/154758060-2cf8eb2a-3abc-4846-ac9d-f56e7f232abc.png";> 2, if i have multiple filters, the scroll bar is hidden, and user need to move the mouse into the filters area to be able to scroll, and there in no clear window to guide the user to move the mouse, which could be a little confusing? https://user-images.githubusercontent.com/81597121/154758393-60d9b221-f5dc-4728-8f0d-b048c3b5b739.mov -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
![https://user-images.githubusercontent.com/81597121/154758060-2cf8eb2a-3abc-4846-ac9d-f56e7f232abc.png"](https://user-images.githubusercontent.com/81597121/154758060-2cf8eb2a-3abc-4846-ac9d-f56e7f232abc.png"){kind=link}