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

Allow custom hint text at DefaultSearchView #187

Merged
merged 4 commits into from
Feb 28, 2024
Merged

Conversation

akvus
Copy link
Contributor

@akvus akvus commented Feb 7, 2024

This Pull Request addresses the issue of the 'Search' hint in DefaultSearchView being fixed and untranslatable. The goal is to modify this hint to be customizable, thereby enabling its translation into various languages.

@rokk4
Copy link

rokk4 commented Feb 15, 2024

@Fintasys could you look into merging this? Thank you :)

@Fintasys
Copy link
Owner

Fintasys commented Feb 16, 2024

Yes, but sorry I haven't found time yet. There are a few things I want to check before allowing wordings directly in the configuration. You shouldn't be blocked by waiting for my review since you can just copy the default implementation, change the string and assign it to customSearchView property in the configuration.

Copy link
Owner

@Fintasys Fintasys left a comment

Choose a reason for hiding this comment

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

I think it's good to go. Let me merge it and release it with 2.1.0. Thank you for your improvement 🙏

@Fintasys Fintasys merged commit e7e971a into Fintasys:master Feb 28, 2024
1 check passed
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.

None yet

3 participants