Skip to content
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

Multi selection #145

Merged
merged 8 commits into from
May 26, 2023
Merged

Multi selection #145

merged 8 commits into from
May 26, 2023

Conversation

martinRenou
Copy link
Member

Screencast.from.2023-05-16.09-49-16.webm

Fix #131 and fix #137

@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch martinRenou/jupytercad/multi_selection

@martinRenou
Copy link
Member Author

The UI-tests are polluted by the JupyterLab 4 update notification

@trungleduc
Copy link
Member

The UI-tests are polluted by the JupyterLab 4 update notification

IIRC there is a config to disable the notification popup

@martinRenou
Copy link
Member Author

Visual regression is legit now

@martinRenou martinRenou marked this pull request as draft May 17, 2023 09:10
@martinRenou
Copy link
Member Author

Converting to draft, I want to add support for multi-selection in the tree view and add a visual regression test for it.

@martinRenou
Copy link
Member Author

martinRenou commented May 25, 2023

update snapshots again again

@martinRenou martinRenou reopened this May 25, 2023
@martinRenou martinRenou marked this pull request as ready for review May 25, 2023 16:13
@martinRenou
Copy link
Member Author

@trungleduc This is ready!

@martinRenou
Copy link
Member Author

The PR also fixes an issue with selection bouncing between the tree and the 3d view

@martinRenou
Copy link
Member Author

please update snapshots again

@martinRenou martinRenou reopened this May 26, 2023
@martinRenou
Copy link
Member Author

@trungleduc green again

@trungleduc
Copy link
Member

Thanks @martinRenou

@martinRenou martinRenou merged commit e0fbd09 into jupytercad:main May 26, 2023
@martinRenou martinRenou deleted the multi_selection branch May 26, 2023 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pre-fill operator dialogs Support multi-selection
2 participants