appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

Published 20 hours ago • verified publisher icon appsmithorg

Select widget deselects on clicking again

Open somangshu opened this issue 2 years ago • 6 comments

The select widget works weird on prod as well, selecting the widget and then clicking on it sometimes deselects the widget in previous and this version of react, this is because of the combination of our keyboard navigation on the property pane that we kick off when a widget is selected vs the select widgets own keyboard navigation support. This is a hunch based on how the widget behaves but needs more investigation coz its a third party widget.

Originally posted by @marks0351 in https://github.com/appsmithorg/appsmith/issues/16257#issuecomment-1236882805

somangshu avatar Sep 21 '22 10:09 somangshu

@Tooluloope can you check what is happening here, I believe this is happening in the edit mode.

@marks0351 can provide more context here

somangshu avatar Sep 21 '22 10:09 somangshu

https://www.loom.com/share/d478188c87124f02b769b806c0c546af @Tooluloope

marks0351 avatar Sep 21 '22 11:09 marks0351

https://www.loom.com/share/b7938d17a978448685a2c27f6b9cf991

marks0351 avatar Sep 29 '22 09:09 marks0351

Here are my observations:

  • SelectWidget makes use of Blueprint Select
    • When we open the dropdown, we can scroll the canvas
    • When we open the dropdown, the focus sometimes shifts from input to property pane's first property. This happens due to a change I've done while working on making the property pane keyboard accessible. This is something that is found to be fixed in another PR that I'm working on currently.
  • TreeSelect, MultiSelect, MultiSelectTree makes use of rc-select & rc-tree-select components
    • When we open the dropdown, we are unable to scroll the canvas unless we close the dropdown
    • Sometimes, when we open the dropdown the property pane is not getting opened

The problem with SelectWidget is something that is related to the property pane. I need to investigate further on what's happening with TreeSelect, MultiSelect and MultiSelectTree. I think it's something to do with rc-select

cc: @somangshu @marks0351

aswathkk avatar Sep 29 '22 11:09 aswathkk

When we open the dropdown, we are unable to scroll the canvas unless we close the dropdown

This seems like an appropriate behaviour according to me, compared to the dropdown scrolling with the page.

Sometimes, when we open the dropdown the property pane is not getting opened

We will have to check this internally, Let us know if you have more info on this @aswathkk

somangshu avatar Oct 06 '22 05:10 somangshu

Tested and verified the fix for the entire select widget family on this PR.

laveena-en avatar Oct 18 '22 04:10 laveena-en