search

tlovett1 / feed-pull

Pull feeds into WordPress
29 stars 13 forks source link

Test Pull display #9

Open tlovett1 opened 9 years ago

tlovett1 commented 9 years ago

Hey @sc0ttkclark, I think the test pull thing is a good idea. However, I think we can do it a bit more elegantly. Any ideas?

sc0ttkclark commented 9 years ago

Was just chatting with Dave Ross, we think we could pull up a new modal with the content listed there. Also, need to array_map( 'esc_html', ... ) all the feed post_field / meta_fields / taxonomy_fields data and convert that SimpleXMLElement on display so it's an escaped array of data too.

Currently using var_export but I think we could display it better just looping through it and outputting it in a nicer <ul>.

tlovett1 commented 9 years ago

Definitely agree. Can you take care of this? No rush.

sc0ttkclark commented 9 years ago

Sure, I've been meaning to dive into the modal init for these types of cases.