-
Notifications
You must be signed in to change notification settings - Fork 478
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
[Bug]: Poor choice of right-arrow icon #1858
Comments
can you tell where i can see this |
It's in the main trace view, e.g. when you run a HotROD demo app and open one of its traces |
More concretely, you can run this example https://github.com/jaegertracing/jaeger/tree/main/examples/hotrod |
The arrow in v1.48 was using IMArrowRight from react-icons/im and we don't have the similar icon in react-icons/io5 to replace https://react-icons.github.io/react-icons/icons?name=io5. We are using: We can revert it but it will conflict with idea #1723. Do you have any suggestions? |
I am fine to stick with thinner arrow, it just needs to be styled to align with the center of the circle |
## Which problem is this PR solving? - Resolves: #1858 ## Description of the changes - Add `vertical-align: middle` to align arrow with circle ## How was this change tested? - Manually ![image](https://github.com/jaegertracing/jaeger-ui/assets/12762270/f6512bff-9a79-43f2-9512-d0a7b6a949f3) ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [ ] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Dat Diep <[email protected]>
In v1.48:
In v1.50:
The arrow is thinner and does not align with the text vertically.
The text was updated successfully, but these errors were encountered: