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

Touch Events -> Resize widgets #247

Open
martaszmit opened this issue Oct 11, 2021 · 13 comments
Open

Touch Events -> Resize widgets #247

martaszmit opened this issue Oct 11, 2021 · 13 comments
Labels
enhancement New feature or request

Comments

@martaszmit
Copy link

Hello!
First of all I would like to thank you guys for last changes in Lumino and for possibility to rearrange widgets on touch screens (@bign8 👏 ).

Regarding this changes, would be great to add possibility to resize widgets on touch screens as well.

What do you think about this proposal?

Cheers!

@martaszmit martaszmit added the enhancement New feature or request label Oct 11, 2021
@welcome
Copy link

welcome bot commented Oct 11, 2021

Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗

If you haven't done so already, check out Jupyter's Code of Conduct. Also, please try to follow the issue template as it helps other other community members to contribute more effectively.
welcome
You can meet the other Jovyans by joining our Discourse forum. There is also an intro thread there where you can stop by and say Hi! 👋

Welcome to the Jupyter community! 🎉

@blink1073
Copy link
Contributor

Hi @martaszmit, yes, that makes sense. I believe the relevant changes would be in SplitPanel.handleEvent.

@martaszmit
Copy link
Author

Thank you for hint :)
I tried to take a look on it, but when I cloned a repository, installed dependencies I tried to run yarn build:src and it fails on @lumino/messaging build script. Did I something wrong?
image

@blink1073
Copy link
Contributor

What is the output of node --version?

@martaszmit
Copy link
Author

v14.17.1

@blink1073
Copy link
Contributor

Hmm, I just tried in a fresh environment with node v14.17.4 and yarn 1.22.17 on my macOS 11.6 and it built fine. What version of yarn and which OS are you using?

@martaszmit
Copy link
Author

yarn 1.22.15 on windows 10 enterprise

@blink1073
Copy link
Contributor

I also tried with Wndows 10 Pro and it built fine, we're also building against Windows in CI. I've not seen an error like that before, I'm not sure what else to suggest other than to try creating a new clean environment or use a conda environment to get node.

@martaszmit
Copy link
Author

I confirm, works fine now, I started again in a fresh environment 👍

@blink1073
Copy link
Contributor

Ha, node is definitely like that sometimes...

@martaszmit
Copy link
Author

Hi,
I spotted that, without any changes it works sometimes, quite randomly as you can see in the video below.
I tried to add changes similar to these: #123 in SplitPanel , but nothing has changed.

screen-capture.-.Copy.mov

@blink1073
Copy link
Contributor

I'm not able to view the video. Can you please push your work in progress to a pull request? We can look at the code together, and the changes will be testable from a tablet using the readthedocs build for the PR.

@martaszmit
Copy link
Author

Of course, thank you :)
PR is ready: #251

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants