Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

Conversation

@kanjanaSi
Copy link
Contributor

No description provided.

@k0nserv
Copy link
Contributor

Hey Kanjana Sirinavasatian (@kanjanaSi) this looks really neat :) Can you post some screenshots of this as well?

attributedPlaceholder = NSAttributedString(
string: placeholder,
attributes: [
NSAttributedStringKey.foregroundColor: color, NSAttributedStringKey.font: font
Copy link
Contributor

Choose a reason for hiding this comment

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

Please cleanup this indentation to match what it was before.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed!

@kanjanaSi
Copy link
Contributor Author

Hugo Tunius (@k0nserv)
screen shot 2017-10-26 at 7 52 23 pm

@k0nserv
Copy link
Contributor

Awesome :) Would you be up for updating the README.md with documentation and the example project?

@k0nserv
Copy link
Contributor

This control is confusing

image

It seems to be doing the opposite for me i.e when it's "disabled" the text fields is actually enabled and vice versa

Copy link
Contributor

@k0nserv Hugo Tunius (k0nserv) left a comment

Choose a reason for hiding this comment

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

Awesome, this looks really good. Lastly could we just add some tests for this new behaviour?

@zarzonis

Hugo Tunius (@k0nserv) Hi Hugo. Do you have any plan to release this? I was ready to implement this feature and I just found out this PR.

@k0nserv
Copy link
Contributor

Hey Spyros Zarzonis (@zarzonis) yeah I'll do that right away :)

@k0nserv Hugo Tunius (k0nserv) merged commit 109a275 into Skyscanner:master Nov 1, 2017
@k0nserv
Copy link
Contributor

That's it released. Thanks a bunch Kanjana Sirinavasatian (@kanjanaSi) for the awesome contribution 🎉

@kanjanaSi
Copy link
Contributor Author

🎉🎉🎉🎉🎉

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants