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): In search block on edit, the sort on and sort order are not working #205

Merged
merged 2 commits into from
Mar 12, 2024

Conversation

dobri1408
Copy link
Contributor

@dobri1408 dobri1408 changed the title (fix): In search block on edit the sort on, and sort order are not working (fix): In search block on edit, the sort on and sort order are not working Feb 28, 2024
@ichim-david ichim-david requested a review from avoinea February 28, 2024 15:31
@ichim-david ichim-david self-requested a review March 8, 2024 10:19
Copy link
Member

@ichim-david ichim-david left a comment

Choose a reason for hiding this comment

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

@dobri1408 as I wrote in the pull request added in core the sorting works now until you select no value in which case the search preview doesn't update.

plone/volto#5262 (comment)
See if you can do something about it, it's not a major thing as the important thing is to get the sorting to work when you apply it but it would be nice to get the search preview to refresh when you basically remove any option from the sidebar widgets

@ichim-david ichim-david merged commit e6a6c2d into develop Mar 12, 2024
6 checks passed
@ichim-david ichim-david deleted the search-block-edit branch March 12, 2024 15:09
ichim-david added a commit that referenced this pull request Mar 18, 2024
ichim-david added a commit that referenced this pull request Mar 25, 2024
* Revert "(fix): In search block on edit, the sort on and sort order are not working (#205)"

This reverts commit e6a6c2d.

* feat(slate): add h4 in slate toolbar

* Automated release 1.31.1

* change(slate): renamed addFormat to addSlateToolbarButton

and pass an object it make it clearer on what does what in the options passed

* style: Automated code fix

* Automated release 1.31.1

* feat(slate): add new h3 slate element

* Automated release 1.31.1

* change(slate): Use F as icon and renamed title of new heading Slate toolbar button to Figure title

* style: Automated code fix

* Automated release 1.31.1

* Update package.json

* Automated release 1.32.0

---------

Co-authored-by: David Ichim <[email protected]>
Co-authored-by: EEA Jenkins <@users.noreply.github.com>
Co-authored-by: eea-jenkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants