-
Notifications
You must be signed in to change notification settings - Fork 14
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: aligned powered by text in search popup #270
fix: aligned powered by text in search popup #270
Conversation
MeiliSearchBox.vue
Outdated
display flex | ||
justify-content space-between | ||
align-items center | ||
justify-content flex-end | ||
align-items center | ||
column-gap 10px |
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.
There is still a gap between the text and the logo. I think if we just remove all these lines, it works
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 won't work if we remove all these lines. The issue was because of justify-content space-between.
The little space is because of column-gap 10px. I don't know if it would look good if all the space is removed but if you want 0 space let me know. I'll remove the 10px gap b/w the text and logo.
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.
Did you remove display flex
as well? I tried it locally and it seems to work
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.
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.
Oh, flex was there. I have updated the PR. Thanks.
Removing all the flex related properties
bors merge |
bors r- |
bors merge |
Pull Request
Related issue
Fixes #269
What does this PR do?
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!