-
Notifications
You must be signed in to change notification settings - Fork 1.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: URL PREVIEW cut off first few lines for very long URL [INS-3640] #7190
Conversation
ffb14d0
to
f0cab96
Compare
@@ -82,7 +82,7 @@ export const RenderedQueryString: FC<Props> = ({ request }) => { | |||
|
|||
return ( | |||
<Wrapper> | |||
<span className={className}>{previewString}</span> | |||
<span className={classNames('my-auto', className)}>{previewString}</span> |
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.
nit:
Is there a reason for using my-auto as the class name?
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.
It should be from tailwind.
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.
thanks!
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.
LGTM. I may suggest if we add screenshot, it would help reviewers who haven't touched the codebase for a while (:p) to understand the changes
Good idea, I will add screenshots in future PRs |
Closes #7178
Change
How to test