-
Notifications
You must be signed in to change notification settings - Fork 21
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
Labels scroll down the page as canvas is scrolled horizontally #86
Comments
Hi @gingerbeardman . Thanks for the report. Which browser is this? The label scrolling is done by JavaScript, and it's sadly indeed possible that it breaks in other browsers. |
Strange, Chrome I test of course. Safari is hard to test since it's proprietary and not available on Linux, but Chrome should've worked - and I can't reproduce it yet. Could you send me the pipe where this breaks? If you don't want to have it public you could send it by mail to support@pipes.digital. Also consider testing with Firefox, that's the browser that sees the most testing. |
Correct pipe is |
I will close here. For now, Safari is out of scope. I'm aware that the editor UI is not completely stable and can break with new browser version. One idea to solve this for good would be to re-implement the whole thing with Flutter. While I'm not generally a fan of Flutter for the Web, this canvas here seems like a perfect use case - it can not be done in HTML anyway, and the implementation would be a lot more stable. |
Safari is out of scope? WebKit 27% of browser market share (Chrome is 59%) |
I'm simply not able to test with Safari. But we can also keep this open and ask for help instead? If someone wanted to provide patches for the editor javascript/css, I'd happily merge them! |
See this screen recording for the bug:
https://imgur.com/FgSQorf
Screenshot:
The text was updated successfully, but these errors were encountered: