tiagobento commented on PR #3200:
URL: 
https://github.com/apache/incubator-kie-tools/pull/3200#issuecomment-3105109571

   @jomarko Thanks for the review. Answering your comments:
   
   > 1. I noticed, we have different widths for these context menus (hit policy 
selector, expression type selector, function type selector). Do we want to 
unify this as part of this PR?
   
   Not for this PR, but I'll look into it.
   
   ---
   
   > 2. Is expected if I open the context menu for decision table cell, there I 
navigate to insert menu, new menu items are shown to me, like prompting me to 
say how many insertions I want to do and then I close this menu by click 
outside, and reopen the menu by click on the same cell, then still the 'Insert' 
menu items are shown. Just asking, not sure
   
   Every time you close the Context menu and reopen it, it will be reset to its 
initial state. That's the expected behavior and wasn't changed on this PR.
   
   ---
   
   > 3. Are no scrollbars expected? Especially for Sample DMN Runner table mode 
is difficult to see all columns. as workaround keybord navigation needs to be 
used.
   
   Using the scroll works for me. Scrollbars aren't rendered but they weren't 
before either. I'll look into it as part of another PR as well. Thanks!


-- 
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]

Reply via email to