fix: update with unified tokens

[mcoker]

What

Closes #312

Description

Updates some regular borders to control border. Here are some other regular borders I was curious if we should update? I couldn't find an example of these tokens in use except for the first one

• --pf-topology__node__decorator__bg--Stroke - it's the border around the status icon. I updated the one on the left to show (dark theme)
  
• --pf-topology__node__label__icon__background--Stroke
• --pf-topology__node--m-selected--separator--Stroke
• --pf-topology__group--m-alt-group--topology__group__background--Stroke
• --pf-topology__group__collapsed-badge__node__label__badge__rect--Stroke
• --pf-topology__edge--m-hover--background--Stroke
• These 3 all look like they go together:
  • --pf-topology-default-create-connector--m-hover--line--Stroke
  • --pf-topology-default-create-connector--m-hover--arrow--Fill
  • --pf-topology-default-create-connector--m-hover--arrow--Stroke

[patternfly-build]

Preview: https://react-topology-pr-topology-318.surge.sh

[mcoker]

@andrew-ronaldson updated all of the icons I could find with RH equivalents. The only icon I couldn't find an equivalent for was the skull icon used here?

react-topology/packages/demo-app-ts/src/demos/topologyPackageDemo/DemoNode.tsx

Line 37 in f45bdac

import SignOutAltIcon from '@patternfly/react-icons/dist/esm/icons/skull-icon';

Looks like it's on the "topology package" demo if you turn on node icons

[jeff-phillips-18]

@andrew-ronaldson updated all of the icons I could find with RH equivalents. The only icon I couldn't find an equivalent for was the skull icon used here?

react-topology/packages/demo-app-ts/src/demos/topologyPackageDemo/DemoNode.tsx

Line 37 in f45bdac

import SignOutAltIcon from '@patternfly/react-icons/dist/esm/icons/skull-icon';

@mcoker Feel free to replace it with anything. It is only for demo purposes.

[mcoker]

@jeff-phillips-18 thanks updated! The icon is only half visible but looks like that's a pre-existing issue.

what it looks like on main now

[mcoker]

@andrew-ronaldson fixed the demo app issue, I needed to update patternfly versions in there as well. You can run it locally - you'll need to pull the latest from my branch, run yarn install, then start the dev and demo app servers to see the change. I also published the demo app here if that's easier - http://pf-topology-unified.surge.sh/

[andrew-ronaldson]

Can you update the decorators to also have the control tokens.

[mcoker]

@andrew-ronaldson updated good sir - http://pf-topology-unified.surge.sh/

Also noticed the icon circle uses the regular border token. Is that ok or want me to update that to the control token, too?

[andrew-ronaldson]

let's make everything control for more contrast please and thank you

[mcoker]

@andrew-ronaldson updated

[jeff-phillips-18]

@mcoker The demo app is not displaying correctly:

Created the demo app surge: https://unified-token-updates.surge.sh/

[jeff-phillips-18]

@mcoker Never mind, my mistake! For some reason my yarn install failed and I hadn't noticed.

updated the surge at: https://unified-token-updates.surge.sh/

[jeff-phillips-18]

LGTM

[github-actions]

🎉 This PR is included in version 6.6.0-prerelease.1 🎉

The release is available on:

• GitHub release
• npm package (@prerelease dist-tag)

Your semantic-release bot 📦🚀