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: Fixed issue where content on the search results page wasn't centered #10570

Merged
merged 16 commits into from
Jan 17, 2023

Conversation

itayweb
Copy link
Contributor

@itayweb itayweb commented Nov 26, 2022

Resolved / Related Issues
Fixed #10549

Validation
How did you test these changes?

  • Built and ran the app

@yaira2
Copy link
Member

yaira2 commented Nov 26, 2022

It's not supposed to be hidden, the issue is with the alignment 🤔

@itayweb
Copy link
Contributor Author

itayweb commented Nov 27, 2022

It's not supposed to be hidden, the issue is with the alignment 🤔

The person who report this issue suggested 2 ways, either hiding it or align it, do you prefer I'll align it?

@yaira2
Copy link
Member

yaira2 commented Nov 27, 2022

It should be properly aligned

@itayweb
Copy link
Contributor Author

itayweb commented Dec 3, 2022

Let me know if the code style need a fix

@itayweb itayweb changed the title Fix: Fixed "Bug: Search non-indexed objects" not centered by hiding it Fix: Fixed "Bug: Search non-indexed objects" not centered Dec 3, 2022
Copy link
Contributor

@QuaintMako QuaintMako left a comment

Choose a reason for hiding this comment

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

LGTM aside the formatting, but someone with more experience on xaml files should review as well.

Copy link
Contributor

@QuaintMako QuaintMako left a comment

Choose a reason for hiding this comment

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

Some issues on the formatting still. Maybe your git does some automatic formatting when committing?

@itayweb
Copy link
Contributor Author

itayweb commented Dec 12, 2022

I tried again to fix the indent even though in my ide it doesn't seem to be a problem, let me know

@QuaintMako
Copy link
Contributor

I tried again to fix the indent even though in my ide it doesn't seem to be a problem, let me know

Yup, seems something went bad formatting-wise with the last commits (+980 lines modified). I'll take a look to see if I can fix the indent myself.

@itayweb
Copy link
Contributor Author

itayweb commented Dec 14, 2022

I tried again to fix the indent even though in my ide it doesn't seem to be a problem, let me know

Yup, seems something went bad formatting-wise with the last commits (+980 lines modified). I'll take a look to see if I can fix the indent myself.

Ok, thanks

@QuaintMako
Copy link
Contributor

@itayweb that should do it formatting wise ;)

In case you did not know, you can chose if your tabs are made of spaces or actual tabs under visual studio. More info on Stack Overflow.

@QuaintMako
Copy link
Contributor

The footer is now well centered, but maybe we could have the two messages to be centered relative to each other? Meaning having Search unindexed items. to be slightly more on the right. (Asking @yaira2 to be sure)

10570

@yaira2
Copy link
Member

yaira2 commented Dec 22, 2022

The best way to solve this issue would be to move it out of the footer and put it under the "No items found" message.

@yaira2 yaira2 changed the title Fix: Fixed "Bug: Search non-indexed objects" not centered Fix: Fixed issue where content on the search results page wasn't centered Jan 5, 2023
@yaira2 yaira2 added changes requested Changes are needed for this pull request and removed needs - code review labels Jan 5, 2023
@QuaintMako
Copy link
Contributor

QuaintMako commented Jan 16, 2023

That should be it for this PR. The border controller has been removed and the button is now centered.

Centered

@yaira2
Copy link
Member

yaira2 commented Jan 16, 2023

@QuaintMako while you're at it, do you want to remove the . from "Search unindexed items"?

yaira2
yaira2 previously approved these changes Jan 17, 2023
Copy link
Member

@yaira2 yaira2 left a comment

Choose a reason for hiding this comment

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

LGTM

@yaira2 yaira2 added ready to merge Pull requests that are approved and ready to merge and removed changes requested Changes are needed for this pull request labels Jan 17, 2023
@yaira2 yaira2 merged commit 3d3e3fb into files-community:main Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Pull requests that are approved and ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Search non-indexed objects" not centered
3 participants