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

Some issues with new map view #6455

Closed
eladb opened this issue May 12, 2024 · 2 comments · Fixed by #6528
Closed

Some issues with new map view #6455

eladb opened this issue May 12, 2024 · 2 comments · Fixed by #6528
Assignees
Labels
🐛 bug Something isn't working 🛫 console Console

Comments

@eladb
Copy link
Contributor

eladb commented May 12, 2024

I tried this:

https://github.com/winglang/containers-example/

This happened:

This was the result:

image

Previously it looked like this:

image

I expected this:

Issues:

  1. We are missing some manually created connections

  2. No "access" information on connections

  3. Some layout issue with the connection from the queue:
    image

  4. It appears like forward() is an inflight method of Queue and it is not and database() is an inflight method of Workload. They are manually created connections, not methods. Ideally I would love the text "forward" and "database" to appear on the connection line itself and not as a method.

  5. The lines are not centered vertically:
    image

Is there a workaround?

No response

Anything else?

No response

Wing Version

No response

Node.js Version

No response

Platform(s)

No response

Community Notes

  • Please vote by adding a 👍 reaction to the issue to help us prioritize.
  • If you are interested to work on this issue, please leave a comment.
@eladb eladb added the 🐛 bug Something isn't working label May 12, 2024
@ainvoner ainvoner added the 🛫 console Console label May 12, 2024
@skyrpex
Copy link
Contributor

skyrpex commented May 16, 2024

Port alignment PR #6494

@monadabot
Copy link
Contributor

Congrats! 🚀 This was released in Wing 0.74.12.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working 🛫 console Console
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants