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

Sketch: Text nodes disappear temporarily on move #1225

Closed
jacobwod opened this issue Oct 25, 2022 · 4 comments
Closed

Sketch: Text nodes disappear temporarily on move #1225

jacobwod opened this issue Oct 25, 2022 · 4 comments
Assignees
Labels
focus:accessibility plugin:draw Functionality and features of the Draw plugin
Projects
Milestone

Comments

@jacobwod
Copy link
Member

Describe the bug
The move interaction for text nodes is not super-intuitive. There is probably a good reason why it works as it does, but can we try to do something about it?

Skarminspelning.2022-10-25.kl.13.57.44.mov
@jacobwod jacobwod added plugin:draw Functionality and features of the Draw plugin focus:accessibility labels Oct 25, 2022
@jacobwod jacobwod added this to the 3.x milestone Oct 25, 2022
@jacobwod jacobwod added this to Approved ideas in Hajk via automation Oct 25, 2022
@Hallbergs
Copy link
Member

It is OpenLayers standard style used within the move interaction. Should be easy to override!

@jacobwod
Copy link
Member Author

I've had a look and the main issue is that it's hard to extend an existing OL style. We either use the default edit style (current situation) or we provide a new one. I can't find the actual default edit style though - OL docs state here a default edit style, but it is not true (e.g. polygons in the docs lack a stroke, but they do have a blue stroke in the real default style as we can see in Hajk).

I'll see if I can dig into this a bit more, but no quick solution as it looks now.

@jacobwod
Copy link
Member Author

jacobwod commented Nov 3, 2022

This isn't easy, but I have a quick fix which may (or may not) be better. Please review these two options:

Current situation

Features get another style during move interaction. This is mostly nice, except for text nodes, which just disappear.

Skarminspelning.2022-11-03.kl.15.07.34.mov

Alternative solution

Features' style remains unchanged during move interaction. This is not as nice for the polygon, but we still have a small indication about this interaction: mouse cursor is changed to hand. The big win here is the edit interaction and what happens with the text node: we can edit and review the changes while we type!

Skarminspelning.2022-11-03.kl.15.11.10.mov

If everyone's fine with this, I prefer the alternative solution. Please let me know.

@jacobwod jacobwod self-assigned this Nov 3, 2022
@jacobwod jacobwod moved this from Approved ideas to In progress in Hajk Nov 3, 2022
@Hallbergs
Copy link
Member

I prefer the alternative with no "move-styling" as well! As you said, the pointer cursor should be enough.

@jacobwod jacobwod moved this from In progress to Done in Hajk Nov 4, 2022
@jacobwod jacobwod modified the milestones: 3.x, 3.11 Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
focus:accessibility plugin:draw Functionality and features of the Draw plugin
Projects
Hajk
  
Done
Development

No branches or pull requests

2 participants