-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Clean Press section #508
Comments
More issues have come in. This one seems pretty unobjectionable:
These ones require some discussion:
|
So, my $.02 on the separate feed issue is that I don't necessarily have a problem with breaking out the press releases from the weekly digest on the press landing page. I wanted to see how that looked, so here it is: I also made a few other small tweaks:
On the separate-but-related issue of also including a link to the "All updates" view, with no filters applied: this to me feels a little odd. Maybe just because press items are the only ones there. But maybe it's not weird. It could also be something that's more of a call-to-action card in the side. |
And I should add that the number of posts visible on each one could definitely change. I just had a couple samples in my local db, which is why only a few are showing up in each feed. |
We can go ahead and make the titles sentence case. Thank you! cc: @PaulClark2 |
Just to keep things organized, these are the things we need to sort through:
I know we need more info about the press photo, but I generally vote that we make these changes so we can go live and do real testing with users. Let me know if I'm being too trigger happy, though |
On the photo—yep, a straightforward change to make, I just need more information (as you said) from Christian and Judy about what they're looking for in that spot, and why the current one isn't a good fit. That will give me better direction to go hunting in, and come back with options. What's the best way for me to get in touch with them? @emileighoutlaw do you already have conversations planned with them this week that I could piggyback off of? Or should I email/call them separately? |
Recapping from our conversation earlier, we're going to go with the suggested edits and then test them. |
Ok, here's what I've got, minus the new photo (ignore the floating menu and wagtail button). I extended the pattern we're using elsewhere to show "related content" to include the "Latest updates" card there (we would need an icon for it). And then I took the opportunity to also include links to the calendar and campaign finance data, both of which seem highly relevant, though they're new ideas. @jenniferthibault feel free to comment here or we can find time to pair too. |
I can look at this tomorrow. Today I need to find photo options for Judy. |
Looks like we need to split the intro for press releases and Weekly Digests into two intros. We could do something more comprehensive, like
But that feels rather redundant to me, and honestly like more than I want to read. What if we cut it down? So we just get at this:
And the more I think about it, I'd like to start with something minimal for testing anyway, to see if journalists actually need any additional information to be successful in their tasks. |
Sounds good. Let's try it. |
I showed this to our critique group (which includes @nickykrause ) and one thing that came up was that the blurb for the press releases in this context might not be super helpful, but that it could be helpful to have specially written blurbs for the Weekly Digests. I don't know that we want to take action on this now, but something to look for in usability testing next week. |
Resolved by https://github.com/18F/fec-cms/pull/535 |
Per our demo with Press, before launch we need to:
Will keep adding to this as feedback comes in.
@AmyKort, as we get feedback from the lovely Press folks, if you want to post i here, we'll triage.
The text was updated successfully, but these errors were encountered: