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

Need country flag feature enabled and some improvements #21

Open
hassan31 opened this issue May 3, 2019 · 8 comments
Open

Need country flag feature enabled and some improvements #21

hassan31 opened this issue May 3, 2019 · 8 comments

Comments

@hassan31
Copy link

hassan31 commented May 3, 2019

Hi,

This is a very good library and its easy to use and simple one. But what about country flags feature to be enabled in this?

And one more thing it would be nice if you give the customization of search controller, like in my case I've navigation bar color of black and search bar text color is also black, I saw you have declared it private and unable to change. However, I can change using manual integration.

Let me know if you want to enable this feature and I'll create a PR.

Regards,
Hassan

@graycampbell
Copy link
Owner

Thanks for the kind words and suggestions. I'm certainly open to adding country flags - I think it would be a good addition. I'm going to do a little research to make sure I can find the highest quality images for the flags.

Just to clarify your second point about search controller customization so I understand fully - would the issue you're encountering be fixed by simply making the searchController property on GCCountryPickerViewController public instead of private? Are you just trying to access the searchBar property of searchController so that you can change the appearance of the search bar?

@hassan31
Copy link
Author

hassan31 commented May 3, 2019

Hi,

I've my own customized theme for the navigation bar, please see attached Screenshot_1 that is background color and white for its tint color. that looks great so far now.

Screenshot_1

Now see Screenshot_2, in which I've typed the country name 'Canada' but I can't see it as its text color is white. And also see the cancel button is in blue color and the tint color of the searchBar text is also in blue, As my theme is something dark wooden color with white, so that's why I need these changes. IIf you can customize your library more that would be great.

Screenshot_2

Look forward to hearing from you soon. Thanks

@graycampbell
Copy link
Owner

Would you also like to be able to customize the appearance of the table view? Allowing you to customize the appearance of the search bar would really just be as simple as making the searchController property public instead of private, but allowing you to customize the appearance of the table view would require more work.

@hassan31
Copy link
Author

hassan31 commented May 4, 2019

Yes, I think, there should be options to show the country flag before the country name and options to hide toggle country code, etc.

@hassan31
Copy link
Author

hassan31 commented May 4, 2019

And, also I'd like you to introduce the currency feature as well in this library. Because I don't want to use multiple pods for the same kind of purpose. In my project, I need currency, country code, country picker with the flag, etc.

@hassan31
Copy link
Author

hassan31 commented May 4, 2019

you can introduce enum like a country picker, currency picker, country code picker with its flag.

@graycampbell
Copy link
Owner

Just to clarify, when you say country code, do you mean calling code? As in the number that you put at the beginning of a phone number? Or do you mean the ISO 3166-2 country code that is unique to each country?

@hassan31
Copy link
Author

hassan31 commented May 4, 2019

Just to clarify, when you say country code, do you mean calling code? As in the number that you put at the beginning of a phone number? Or do you mean the ISO 3166-2 country code that is unique to each country?

Yes Calling Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants