etn-ccis/blui-angular-workflows

Use Waves SVG for Dark Theme

Closed this issue · 4 comments

Describe the bug

The create-account success page in the custom example is using an image that only looks good in the light theme. React workflow example is using a waves svg that looks good in both light and dark, use that one here.

What is the expected behavior?

The picture should look good on dark and light themes.

What are the steps to reproduce?

  1. Clone the repo (git clone https://github.com/brightlayer-ui/angular-workflows.git).
  2. CD into the project (cd angular-workflows)
  3. Run the example project (cd login-workflow && yarn start:example)
  4. Navigate to http://localhost:4200/auth/register/create-account
  5. Inspect the page & update the <body> class from blui-blue to blui-blue-dark. This will set the dark theme manually.
  6. Fill in the required information to create an account.
  7. Observe the final page; it should match the image below.

Screenshots or links to minimum reproduction example

image

The React Workflows don't have a waves image at all. The React Native ones do, but it looks like they are the same one that's used here (white background). We will need an updated graphic from @huayunh that has a transparent background that we can use for both light and dark theme.

The no wave image is the default state:
image

But we shall allow users to customize that area. For example, if you want to do the Eric Atha thing, you shall be allowed:

Screen.Recording.2023-03-28.at.10.52.43.AM.mov

@huayunh understood — this fix is JUST in the example project, not in the main code...but we don't have the image asset that works well for light and dark theme (I don't see it anywhere in the react workflow repository even though it's mentioned in the description as being there, and the one in the RN repository has the same issue as reported here on angular). Do you have a wave image that uses a transparent background somewhere in Figma that we can use for this?

I produced these two that does not look exactly the same as the one in the screenshot, but should capture the spirit:
SVG, PNG
(Try SVG first)

Preview:
image