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

hollow floating wire #1491

Merged
merged 5 commits into from
Mar 27, 2022
Merged

hollow floating wire #1491

merged 5 commits into from
Mar 27, 2022

Conversation

kianzarrin
Copy link
Collaborator

@kianzarrin kianzarrin commented Mar 25, 2022

fixes #1485 : make it easy to distinguish between adding/removing connection

Visual changes :

  • floating wire is white and hollow.

  • when connection is made the underneath connection is visible.

  • +/- cursors help to distinguish adding/removing lane connection

  • cursor code:

    • Added CursorUtil to create/release cursors
    • Added OverrideCursor to subtools.
    • OverrideCursor also handles the nope_cursor (code moved to lane connection tool).

TMPE.zip

@kianzarrin
Copy link
Collaborator Author

Cities_A62gmvBCHl
Cities_BtYOCOGYnE

@kianzarrin kianzarrin requested review from krzychu124, kvakvs, DaEgi01 and chameleon-tbn and removed request for krzychu124 and kvakvs March 25, 2022 13:40
@kianzarrin kianzarrin self-assigned this Mar 25, 2022
Copy link
Collaborator

@kvakvs kvakvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minus cursor looks very similar to ⛔ road sign, maybe change color to darker red or black
But this is nice

@chameleon-tbn
Copy link
Contributor

cursor_remove_lane_connector_-_31_darker
maybe this?

@originalfoo
Copy link
Member

what about TMPE logo blue background for the - icon?

@kianzarrin
Copy link
Collaborator Author

which TMPE logo?

@originalfoo
Copy link
Member

which TMPE logo?

the mod logo from workshop page; the darker blue with white - sign might work:

image

@krzychu124
Copy link
Member

Why without a pointer? Also... now when I think of it, why not without circle, just pointer + sign ➕(green), ➖(dark blue)?

@kianzarrin
Copy link
Collaborator Author

@chameleon-tbn Can you please address the sprite ideas?

@kianzarrin
Copy link
Collaborator Author

meanwhile cham plays with sprites can we do the rest of the code review?

Copy link
Member

@originalfoo originalfoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@originalfoo originalfoo added Usability Make mod easier to use UI User interface updates LANE ROUTING Feature: Lane arrows / connectors labels Mar 27, 2022
@originalfoo originalfoo added this to the 11.6.5.2 milestone Mar 27, 2022
@kianzarrin kianzarrin added the DO NOT MERGE YET Don't merge this PR, even if approved, until further notice label Mar 27, 2022
@kianzarrin
Copy link
Collaborator Author

thanks. should I wait for cham to come up with better icon or should I just merge and come up with sprite later?
merging will unblock future progress and reduce chance of conflict.

@originalfoo
Copy link
Member

I think merge and we can start testing stuff and come up with better icon later if desired. I'm really liking the way this is working already in this PR.

@kianzarrin kianzarrin merged commit abb5655 into master Mar 27, 2022
@kianzarrin kianzarrin deleted the 1485-hollow-floating-wire branch March 27, 2022 23:45
@chameleon-tbn
Copy link
Contributor

Why without a pointer? Also... now when I think of it, why not without circle, just pointer + sign ➕(green), ➖(dark blue)?

What's the need of a pointer there? why not center the + / - into the connector circle? ... would clearly show what it does...

blue_minus_31
green_plus_31

@chameleon-tbn Can you please address the sprite ideas?

cursor_remove_lane_connector_blue_31
cursor_add_lane_connector_31

anything else?

@kvakvs
Copy link
Collaborator

kvakvs commented Mar 28, 2022

"Pointer" + "something to the current edit mode" is a good idea, i like

@kianzarrin
Copy link
Collaborator Author

Ill create a issue for such improvements

@krzychu124
Copy link
Member

Why without a pointer? Also... now when I think of it, why not without circle, just pointer + sign ➕(green), ➖(dark blue)?

What's the need of a pointer there? why not center the + / - into the connector circle? ... would clearly show what it does...

blue_minus_31 green_plus_31

@chameleon-tbn Can you please address the sprite ideas?

cursor_remove_lane_connector_blue_31 cursor_add_lane_connector_31

anything else?

Because we already use "pointer" + "thing" to do initial selection? It's just to make UX more or less consistent, also similar to how game handles user interactions where it shows cursor

@kianzarrin
Copy link
Collaborator Author

#1498

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO NOT MERGE YET Don't merge this PR, even if approved, until further notice LANE ROUTING Feature: Lane arrows / connectors UI User interface updates Usability Make mod easier to use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lane Connection tool does not give UI feedback when lane is connected.
5 participants