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

Missing font awesome icons within logical flow SVG. #6441

Closed
mharward-gr opened this issue Feb 10, 2023 · 2 comments
Closed

Missing font awesome icons within logical flow SVG. #6441

mharward-gr opened this issue Feb 10, 2023 · 2 comments
Labels
bug Defect affecting the currently released version of Waltz duplicate
Milestone

Comments

@mharward-gr
Copy link

Description

When viewing logical flows (both normal and beta views) for an application in the UI, the font-awesome icons within the SVG do not show.

image

Font awesome icons used elsewhere (in normal html) appear as expected.

Experimenting with this seems to show that adding the class fa to the text elements within the SVG (with an existing attribute of font-family="FontAwesome" seems to fix the issue.

image

I've checked and this doesn't seem to be an issue with the font being missing, though interestingly the icons are still appearing in https://demo.waltz.finos.org/, so it may be either a regression or something weird with our setup (docker image deployed on kubernetes, no public internet).

Would someone be able to take a look?

Waltz Version

waltz:1.46

Steps to Reproduce

  1. Visit application page, open data flows section, see logical flows tab (where logical flows exist)
  2. See that beside each upstream and downstream source the question icon is missing (replaced by a square icon)

Expected Result

The icon is rendered correctly

Actual Result

The icon is not properly rendered - rendered as a square 'missing icon' character

@mharward-gr mharward-gr added the bug Defect affecting the currently released version of Waltz label Feb 10, 2023
@davidwatkins73
Copy link
Member

Hi,

Thanks for the detailed report. @JWoodland-Scott has already fixed this in the 1.47 release. The related issue is #6350.

We tried a bunch of different options to get font-awesome to work in all contexts with webpack-5 and never got a solution that worked for both displaying, printing and image-downloading. The fix involved creating small, custom svg symbols for use by d3 (or in raw svg).

@mharward-gr
Copy link
Author

Perfect, thank you @davidwatkins73 and @JWoodland-Scott. Sorry for the duplicate!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Defect affecting the currently released version of Waltz duplicate
Projects
None yet
Development

No branches or pull requests

2 participants