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
Customize search button and placeholder #94
base: main
Are you sure you want to change the base?
Conversation
@nngu6036 not all my requests have been fixed and the tests still not passing. |
I suspect it's going to work better if we use "leave blank for default 'search'" as that will allow the portlet to work in multiple languages at once. What do you think @petschki ? |
@@ -38,3 +38,9 @@ Scenario: Searching through a portlet with ajax disabled | |||
# and should be 4 filter options | |||
|
|||
|
|||
Scenario: Customize search porlet text with ajax disabled |
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.
needs to be rebased against the testing branch. There aren't tests specific to portlets anymore. The same test should work for tiles and portlets
<tal:urlquery repeat="item python:view.urlquery"> | ||
<input type="hidden" name="${python:item[0]}" value="${python:item[1]}"/> | ||
</tal:urlquery> | ||
<input name="SearchableText" type="text" | ||
placeholder="Search" | ||
placeholder="${view/settings/placeholder}" |
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.
need the same changes for tiles
), | ||
description=_("help_header", u"Placeholder of the search input."), | ||
required=False, | ||
default=_(u"Search"), |
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.
Should have no default so the default will be translated based on the language of the site.
Same as the button text
No description provided.