-
Notifications
You must be signed in to change notification settings - Fork 78
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
Multiselect: Select All checkbox should only select visible (filtered) items #3845
Comments
@tmcconechy is this a different feature you think? Should we make this configurable and leave the current behavior as the other option? |
I feel like this should be the correct behavior but open to debate. So thinking make an option selectAllFilterOnly or whatever and set it to true by default |
Any progress on this issue? |
@thomas-blackwell we have this issue scheduled for our June sprint, starting June 1 |
@jbrcna Looks like @janahintal raised #4055 for this. I'm addressing in #4056, should be fixed after we merge that |
Can this fix be applied on datagrid too? |
I dont think this issue has anything to do with datagrid specifically as it is around the lookup components functionality? |
Yes it does :-(
Please try it yourself in order to be fully convinced
Thanks
Cheers
//Asaf
…On Tue, Mar 30, 2021 at 4:33 PM Tim McConechy ***@***.***> wrote:
I dont think this issue has anything to do with datagrid specifically as
it is around the lookup components functionality?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#3845 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AS5C6WDSJO56YCOWDZZ4XNLTGHHLDANCNFSM4MVPPZIA>
.
|
OK, that would be a separate issue... But i don't get it so maybe need some steps, closest I can think of is: https://master-enterprise.demo.design.infor.com/components/datagrid/example-multiselect.html
This issue relates to lookup and no further work will be done on it here. |
Found it - the bug is related to the fact that the selectCount: true was
not present in the toolbar attribute on the Script, but I had the <span
class="datagrid-result-count">(N Results)</span> explictly included in the
HTML
Do you want me to open a new issue on that?
…On Tue, Mar 30, 2021 at 4:41 PM Tim McConechy ***@***.***> wrote:
OK, that would be a separate issue... But i don't get it so maybe need
some steps, closest I can think of is:
https://master-enterprise.demo.design.infor.com/components/datagrid/example-multiselect.html
- apply a filter
- select all
- revert the filter
- only what was filtered is selected
This issue relates to lookup and no further work will be done on it here.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#3845 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AS5C6WBMF7FCPGWTAC4Y6STTGHIKLANCNFSM4MVPPZIA>
.
|
Yes, if you have an issue - this is closed nothing further is being done here. |
Is your feature request related to a problem or use case? Please describe.
selectAll option in dropdown component should select only what is currently in the list when filter is applied
Describe the solution you'd like
An option for selectAll to select everything or filtered list.
Describe alternatives you've considered
adding new parameter 'selectFiltered'
Additional context
The text was updated successfully, but these errors were encountered: