search

andreamazz / AMTagListView

UIScrollView subclass that allows to add a list of highly customizable tags.
MIT License
758 stars 110 forks source link

Demo App: Tags aren't appearing in TagListView. #20

Closed bfeher closed 9 years ago

bfeher commented 9 years ago

Hi, slick control!

I used pod try AMTagListView and chose option 2. I built using Xcode 6.1.1 (6A2008a), and ran using an iPhone 5s (8.1) simulator.

When hitting enter, the UITextField does empty, and the tag is created, but it never appears on screen. After peeking under the hood I found this offending line of code: screen shot 2015-01-19 at 13 50 44

Is there a reason for this line? After removing it things work properly, but I feel shady just removing lines of code without fully understanding the entire project.

Awesome control btw, thanks for sharing it! :D

bfeher commented 9 years ago

Woops, so after digging around further, I see that the code above is necessary, and the real bug is here in AMViewController.m: screen shot 2015-01-19 at 14 45 57

I suspect there it should be return ![text isEqualToString:@"bad"];

andreamazz commented 9 years ago

Hi @bfeher Yup, my bad, that condition should be inverted, I'll fix it right away. Thanks!