-
Notifications
You must be signed in to change notification settings - Fork 539
disabled color #177
disabled color #177
Conversation
|
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 |
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.
Please cleanup this indentation to match what it was before.
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.
Fixed!
0a09b2e to
76602eb
Compare
|
Awesome :) Would you be up for updating the README.md with documentation and the example project? |
76602eb to
6b06fe3
Compare
Hugo Tunius (k0nserv)
left a comment
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.
Awesome, this looks really good. Lastly could we just add some tests for this new behaviour?
|
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. |
|
Hey Spyros Zarzonis (@zarzonis) yeah I'll do that right away :) |
|
That's it released. Thanks a bunch Kanjana Sirinavasatian (@kanjanaSi) for the awesome contribution 🎉 |
|
🎉🎉🎉🎉🎉 |


No description provided.