feat(tree): multi-selection no longer requires holding shift key #3733
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
bug
anveshmekala
added
the
2 - in development
|
This PR has been automatically marked as stale because it has not had recent activity. Please close your PR if it is no longer relevant. Thank you for your contributions. |
github-actions
bot
added
the
Stale
anveshmekala
removed
the
Stale
jcfranco
requested changes
Dec 31, 2021
There was a problem hiding this comment.
This is looking good, @anveshmekala!
calcite-tree has a few selection modes, so it'd be good to get some additional manual testing to drive new E2E tests since the current E2E test suite doesn't cover all of them. @benelan Can you help with this?
|
@anveshmekala Should this use the |
anveshmekala
changed the title
anveshmekala
commented
Jan 7, 2022
|
/merge master |
|
/merge master |
|
Tested the branch locally, LGTM! One thing I noticed is that cc @jcfranco Edit - repro sample: https://codepen.io/benesri/pen/QWOLJPa |
I don't think so. That event should be exclusively tied to selection which is different from the visual state of the tree. The doc seems to indicate this as well. |
|
/merge master |
jcfranco
approved these changes
Jan 24, 2022
|
@benelan @anveshmekala Thanks for doing manual testing on this. 💪 |
|
/merge master |
anveshmekala
deleted the
anveshmekala/3117-fix-tree-multi-select-with-click
branch
y0n4
pushed a commit
that referenced
this pull request
Jan 27, 2022
* fix(tree): enable multi select with pointer * save selected items at main level * fix single selection mode * deselect siblings when mode is single * fix deselecting tree item when mode is multi * fix deselecting with shift keydown * refactor e2e test * remove unused imports in e2e * feedback changes * emit calciteTreeSelect event without shift key * refactor code * add e2e test and feedback changes Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
y0n4
pushed a commit
that referenced
this pull request
Jan 27, 2022
* fix(label, input, input-message)!: deprecate and no longer support status prop for calcite-label * alter tests for status being passed in calcite label * update the readme for labels using status and add status prop to input/input-message * add deprecated comment for status prop * refactor(panel, shell, tree-item): Add keys to conditionally rendered slots. (#3962) * refactor(panel, shell, tree-item): Add keys to conditionally rendered slots. * add const for key. * fix code * review cleanup * feat(panel): Add method to scroll content. #3924 (#3960) * ci(screener): updating to Chrome 97 (#3940) * 1.0.0-next.373 * docs: update component READMEs (#3972) Co-authored-by: jcfranco <[email protected]> * ci(screener): enlarging screen resolution (#3977) * refactor(input): fix all types that are any (#3981) * refactor: Update relevant components to handle slotting after init (#3889) * refactor: Update relevant components to use ConditionalSlotComponent #3686 * cleanup * cleanup * cleanup * fix links that aren't direct slots * remove assigned slot check, only get direct children * fix test * cleanup * cleanup * partial review fixes * revert changes to combobox and card * refactor(slider): remove drag event listeners on component disconnected (#3969) * refactor(slider): remove eventListeners on DOM disconnected * feedback changes * add private method for duplicate code * rename method * build: update browserslist db (#3985) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * test(input): when both 'ArrowUp' and 'ArrowDown' are pressed at the same time most recently pressed key takes over (#3989) Co-authored-by: Eliza Khachatryan <[email protected]> * alter verbiage of deprecated prop * ci(screener): restoring pre-Sauce integration settings (#3994) * docs(template): provide context for the repro samples (#3970) * docs(templates): provide content for the repro samples * use cc latest version Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * chore: pin node 16 and npm 8 for volta (#3964) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * docs(readme): remove sketch info until it is up to date (#3996) * fix(date-picker): update utils locale to get the lang code if regional code is not found (#3968) * fix(date-picker): update utils locale to get the lang code if regional code is not found * alter util changes based from pr comments * 1.0.0-next.374 * ci: allow primary and secondary contacts to merge on all prs (#3999) * ci: allow primary and secondary contacts to merge on all prs * cleanup * use admin token * fix(date-picker, input-date-picker): update pt-BR and pt-PT localization files (#3980) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * feat(tree): multi-selection no longer requires holding shift key (#3733) * fix(tree): enable multi select with pointer * save selected items at main level * fix single selection mode * deselect siblings when mode is single * fix deselecting tree item when mode is multi * fix deselecting with shift keydown * refactor e2e test * remove unused imports in e2e * feedback changes * emit calciteTreeSelect event without shift key * refactor code * add e2e test and feedback changes Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * 1.0.0-next.375 * test(dropdown): correct logical errors in the test (#3998) * test(dropdown): correct logical errors in the test * assert both events after each toggle action and use page.waitForEvent before doing assertions Co-authored-by: Eliza Khachatryan <[email protected]> * deprecate and restore the use of status prop * restore reading status prop from label Co-authored-by: Matt Driscoll <[email protected]> Co-authored-by: Erik Harper <[email protected]> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Ben Elan <[email protected]> Co-authored-by: jcfranco <[email protected]> Co-authored-by: Anveshreddy mekala <[email protected]> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Eliza Khachatryan <[email protected]> Co-authored-by: Eliza Khachatryan <[email protected]> Co-authored-by: JC Franco <[email protected]>
anveshmekala
removed
the
2 - in development
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue: #3117
Summary
This will enable to select multiple
calcite-tree-itemswhen mode is set tomulti | multi-childrenwithout holdingshiftkey. Additionally,calciteTreeSelectwill emit the array of selected items instead of one.