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

refactor(25464): Improver visual identity of the workspace #556

Conversation

vanch3d
Copy link
Contributor

@vanch3d vanch3d commented Sep 10, 2024

See https://hivemq.kanbanize.com/ctrl_board/57/cards/25464/details/

This PR is part of the bi-directional epic and improves the rendering and interactions of the workspace

  • device nodes are now "glued" to their adapter, any change in one location results in a similar move to the other.
  • host nodes are similarly "glued" to the bridge nodes
  • when adapters are added to a group, their device is also added
  • zoom range has been increased, especially "out"-ward
  • content of the nodes are now responsive to the zoom. The defaut rendering for each type of nodes switch to a "skeleton" rendering when zooming "out" (e.g. showing a status icon and skeleton placeholders for the topic subscriptions)
  • theme of the toolbar is now matching across Data Hub and Workspace

Out-of-scope

Before

screenshot-localhost_3000-2024_09_10-13_42_57

After

screenshot-localhost_3000-2024_09_10-13_42_33

@cla-bot cla-bot bot added the cla-signed label Sep 10, 2024
@vanch3d vanch3d force-pushed the epic/23777/bi-directional-frontend branch from 7ef1fa9 to d7026fe Compare September 10, 2024 11:10
@vanch3d vanch3d self-assigned this Sep 10, 2024
@vanch3d vanch3d force-pushed the refactor/25464/workspace-visual-identity branch from ea874e4 to d59bbf6 Compare September 10, 2024 11:12
@vanch3d vanch3d marked this pull request as ready for review September 10, 2024 12:50
Copy link
Contributor

@oli-hivemq oli-hivemq left a comment

Choose a reason for hiding this comment

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

Only 1 suggestion. Otherwise LGTM 🦣

@vanch3d vanch3d force-pushed the refactor/25464/workspace-visual-identity branch from 6528c60 to 70bc502 Compare September 11, 2024 08:56
@vanch3d vanch3d merged commit 2d7158f into epic/23777/bi-directional-frontend Sep 11, 2024
8 of 9 checks passed
@vanch3d vanch3d deleted the refactor/25464/workspace-visual-identity branch September 11, 2024 09:31
vanch3d added a commit that referenced this pull request Sep 11, 2024
refactor(25464): Improver visual identity of the workspace

* refactor(25464): export visual mapping for adapter status

* refactor(25464): export zoom selectors

* refactor(25464): add glue to adapter and bridge nodes

* refactor(25464): add style to the minimap

* refactor(25464): change icon

* refactor(25464): refactor nodes to introduce skeleton rendering

* refactor(25464): hide toolbars when nodes are being dragged

* refactor(25464): add devices to the group when adapters are added

* test(25464): fix tests

* fix(25464): change theme to match data hub reactflow

* refactor(25464): refactor the toolbar button group for reuse

* test(25464): add tests

* fix(25464): fix edge markers

* test(25464): fix tests

* fix(25464): fix review issues
vanch3d added a commit that referenced this pull request Sep 23, 2024
refactor(25464): Improver visual identity of the workspace

* refactor(25464): export visual mapping for adapter status

* refactor(25464): export zoom selectors

* refactor(25464): add glue to adapter and bridge nodes

* refactor(25464): add style to the minimap

* refactor(25464): change icon

* refactor(25464): refactor nodes to introduce skeleton rendering

* refactor(25464): hide toolbars when nodes are being dragged

* refactor(25464): add devices to the group when adapters are added

* test(25464): fix tests

* fix(25464): change theme to match data hub reactflow

* refactor(25464): refactor the toolbar button group for reuse

* test(25464): add tests

* fix(25464): fix edge markers

* test(25464): fix tests

* fix(25464): fix review issues
vanch3d added a commit that referenced this pull request Sep 27, 2024
refactor(25464): Improver visual identity of the workspace

* refactor(25464): export visual mapping for adapter status

* refactor(25464): export zoom selectors

* refactor(25464): add glue to adapter and bridge nodes

* refactor(25464): add style to the minimap

* refactor(25464): change icon

* refactor(25464): refactor nodes to introduce skeleton rendering

* refactor(25464): hide toolbars when nodes are being dragged

* refactor(25464): add devices to the group when adapters are added

* test(25464): fix tests

* fix(25464): change theme to match data hub reactflow

* refactor(25464): refactor the toolbar button group for reuse

* test(25464): add tests

* fix(25464): fix edge markers

* test(25464): fix tests

* fix(25464): fix review issues
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants