Skip to content

Conversation

mgsnova
Copy link

@mgsnova mgsnova commented May 20, 2013

A click on the feed name in the feed listing will show the unread stories of the feed (or the zen page if zero). The "mark all as read" button works for the feeds stories. An additional (and duplicate!? "home" button links back to the feed listing). The refresh button is adjusted as well.

  • the url for the feeds stories is "/news/:id" this may be change to "/feeds/:id/stories" or "/feeds/:id/news"
  • I'm not 100% happy about the conditionals in the view and controller regarding whether a feed_id is given or not.

@swanson
Copy link
Collaborator

swanson commented May 20, 2013

I'm curious why we need to include the feed_id? The story id's are unique so mark_all_as_feed should be fine. Maybe we should just make it an AJAX post so we don't need to redirect.

@swanson
Copy link
Collaborator

swanson commented May 29, 2013

Going to put this on hold till we figure out #46 - I think the UI challenges are the same.

@swanson swanson closed this May 29, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants