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.

Sign up for GitHub

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

Jump to bottom

Clear all text should be in the center of the filter liable if the customer is trying to filter the orders. #6778

Closed
Sanjaybhattwebkul opened this issue Sep 30, 2022 · 4 comments
Closed

Clear all text should be in the center of the filter liable if the customer is trying to filter the orders. #6778

Sanjaybhattwebkul opened this issue Sep 30, 2022 · 4 comments
Assignees
@papnoisanjeev
Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. Hacktoberfest Bagisto Hacktoberfest Event. UI UI package related issue.
Milestone
1.5.0

Comments

@Sanjaybhattwebkul
Copy link
Contributor

Bagisto Version: Master

Steps To reproduce--

1. Log in a customer.
2. Change app locale to Arabic.
3. Click on orders inside the profile section.
4. Apply filter on orders.
5. Try to remove all applied filters.

Expected result: Clear all text should be in the center of the filter liable if the customer is trying to filter the orders.

Actual result: Clear all is not in the center of the filter liable if the customer is trying to filter the orders. Please find the screenshot Here
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Something isn't working. UI UI package related issue. Hacktoberfest Bagisto Hacktoberfest Event. labels Sep 30, 2022
@Sanjaybhattwebkul
Copy link
Contributor Author

The mouse property should a be pointer for the clear all text.

@papnoisanjeev papnoisanjeev mentioned this issue Sep 30, 2022
Merged
@papnoisanjeev papnoisanjeev added the Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. label Sep 30, 2022
@saimaheshtaduri
Copy link

I can work on this. Assign it to me. @Sanjaybhattwebkul

@shikhar13012001
Copy link

I would like to work on this. Can you assign this to me? @Sanjaybhattwebkul

@Sanjaybhattwebkul Sanjaybhattwebkul added this to the 2.0.0 milestone Oct 10, 2022
jitendra-webkul added a commit that referenced this issue Oct 10, 2022
@jitendra-webkul
Merge pull request #6783 from papnoisanjeev/clear-all-button-issue
065cd3a 
Clear ALL button issues #6778
@Sanjaybhattwebkul Sanjaybhattwebkul added Bug Fixed When bug issue is fixed. and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. labels Oct 19, 2022
@Sanjaybhattwebkul
Copy link
Contributor Author

@jitendra-webkul
Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@papnoisanjeev papnoisanjeev

Labels
Bug Fixed When bug issue is fixed. Bug Something isn't working. Hacktoberfest Bagisto Hacktoberfest Event. UI UI package related issue.
Projects
None yet
Milestone
1.5.0
Development

No branches or pull requests
4 participants
@papnoisanjeev @saimaheshtaduri @shikhar13012001 @Sanjaybhattwebkul