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

Patches that link map regions together should be more obvious #3556

Open
hhyyrylainen opened this issue Jul 19, 2022 · 3 comments · May be fixed by #5786
Open

Patches that link map regions together should be more obvious #3556

hhyyrylainen opened this issue Jul 19, 2022 · 3 comments · May be fixed by #5786

Comments

@hhyyrylainen
Copy link
Member

as it is right now hard to tell which patch in a region you need to move to to be able to move outside the region. Maybe the region connection lines should continue all the way to the patches they actually connect?

@84634E1A607A
Copy link
Contributor

84634E1A607A commented Jul 19, 2022

Previously I discussed this with @Kemikal1, he said that when ocean regions ajacent, all depths are connected. This makes multiple parallel lines.

@hhyyrylainen
Copy link
Member Author

I was thinking that there still would be one line between the regions, but each patch would draw a line to where the region line connects, a very quick mockup:
2022-07-19_13 10 08 8021

That can also get really busy with a lot of connections, but that's the best idea I've got.

@hhyyrylainen hhyyrylainen changed the title Patches that link regions together should be more obvious Patches that link map regions together should be more obvious Sep 28, 2022
@hhyyrylainen hhyyrylainen moved this to Backburner in Thrive Planning Aug 7, 2024
@Patryk26g
Copy link
Contributor

Patryk26g commented Dec 29, 2024

I was messing around a little bit with that issue and got results that look like that but I don't know whether I like it or not. In some places it looks quite fine but in others it can get a little bit too crowded
image

Maybe if the color was little bit darker for these connections it would look better because otherwise it gets mixed up with the internal connections between patches. We could also make the distances between the patches' icons bigger

@Patryk26g Patryk26g linked a pull request Jan 1, 2025 that will close this issue
4 tasks
@hhyyrylainen hhyyrylainen added this to the Release 0.8.1 milestone Jan 8, 2025
@hhyyrylainen hhyyrylainen moved this from Backburner to In progress in Thrive Planning Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

3 participants