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

fix: aligned powered by text in search popup #270

Merged
merged 2 commits into from
Oct 27, 2022

Conversation

bhanuloshali1995
Copy link
Contributor

@bhanuloshali1995 bhanuloshali1995 commented Oct 18, 2022

Pull Request

Related issue

Fixes #269

What does this PR do?

  • Aligns the search popup's bottom Powered By text to the right.

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Comment on lines 209 to 212
display flex
justify-content space-between
align-items center
justify-content flex-end
align-items center
column-gap 10px
Copy link
Contributor

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

Copy link
Contributor Author

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.

Copy link
Contributor

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot 2022-10-27 at 11 16 21

Screenshot 2022-10-27 at 11 16 42

Copy link
Contributor Author

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
@bidoubiwa bidoubiwa added the bug Something isn't working label Oct 27, 2022
@bidoubiwa
Copy link
Contributor

bors merge

@bidoubiwa
Copy link
Contributor

bors r-

@bidoubiwa
Copy link
Contributor

bors merge

@bidoubiwa bidoubiwa merged commit b712733 into meilisearch:main Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Footer Powered by is at the bottom right instead of bottom left
2 participants