-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
fix: fixed react flow link #4302
Conversation
@govardhanshah456 could you please keep the original color of the link and just move the position to be centered as you did but with 30px top margin? |
I see, could you please just move it to the center with the old color? And just moving it down by 30px? |
You can refer the latest commit, I have made that changes on it only |
@govardhanshah456 mind sharing a photo of how it looks after the changes? |
|
@scopsy & @LetItRock any changes required? I guess mostly we would like to change text color in dark mode. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a small adjustment here
@govardhanshah456 please comment on the original issue so I can assign you |
Co-authored-by: Dima Grossman <[email protected]>
Co-authored-by: Dima Grossman <[email protected]>
Co-authored-by: Dima Grossman <[email protected]>
yes, did that, and commited the suggestions provided |
@scopsy Hey, can you assign this to me? i have commented on the issue as well for the same. Thanks |
What change does this PR introduce?
Made the React Flow Link More foundable.
Why was this change needed?
closes #4266
Other information (Screenshots)