Design/UX #4224
closedAdd labeling to RSS feed 'Refresh time' options
Added by Samantha Raddatz about 9 years ago. Updated almost 9 years ago.
0%
Description
The sixth step, to add RSS feeds, only has numericals under the ‘Refresh time’ drop down menu. It is not clear to what those refer (seconds, minutes, days, etc).
Clarify what unit the ‘Refresh time’ is in.
Change “seperate” to “separate”
Files
| 21creategroup2.png (51.7 KB) 21creategroup2.png | circled areas | Samantha Raddatz, 2015-06-25 11:00 AM |
Updated by Boone Gorges about 9 years ago
- Category name changed from User Experience to Groups (misc)
- Assignee changed from Boone Gorges to Samantha Raddatz
Gah. Any objection to removing this step from group creation? It's really terrible.
At least, let's remove the "Refresh time" option. We can make a decision about this for the user - there's no reason why this should be configurable.
Updated by Matt Gold about 9 years ago
RSS feeds might be another option to add to what I described here as non-essential optional functions - https://redmine.gc.cuny.edu/issues/4225#note-2
Updated by Boone Gorges about 9 years ago
- Assignee changed from Samantha Raddatz to Daniel Jones
- Target version set to Future release
Matt Gold wrote:
RSS feeds might be another option to add to what I described here as non-essential optional functions - https://redmine.gc.cuny.edu/issues/4225#note-2
Agreed, but this is pretty egregious. Let's get rid of the Refresh Time option in the short term. Dan, can you have a look? At the very least, hide it with JS. But ideally, let's take it out of the markup (and replace with a hidden input if necessary). If this will require modding the plugin, let me know.
Updated by Daniel Jones about 9 years ago
Yeah it looks like the easiest way to do this will be to modify the plugin to make it a hidden input with the default value we want, both on create and edit screens. Should I go ahead and do that? Besides that, I do think the only other way would be hiding with javascript. Would I do that by adding a new script in our theme's functions.php file? I guess it's possible that there's a filter that we would hook onto that might let us modify the markup before it's output - do you know if that's an option off the top of your head, Boone?
Updated by Boone Gorges about 9 years ago
- Status changed from New to Assigned
Thanks for looking, Dan. Let's go ahead and modify the plugin. It's been abandoned, so it shouldn't pose any issues, but I'll make a note that we should disable automatic updates for it, in case someone adopts it.
Updated by Boone Gorges about 9 years ago
Wait - scratch that last comment. I realize now that we're already running a fork, which we maintain and ship with Commons In A Box. (So many plugins, I get them confused.) https://github.com/cuny-academic-commons/external-group-blogs/
I've added Ray as a watcher. Ray, do you have any objection to making the suggested changes right in the plugin itself? It should only affect Commons In A Box users, and I can't imagine that any of them will complain about these simplifications.
Updated by Daniel Jones almost 9 years ago
Okay, did that here: https://github.com/cuny-academic-commons/cac/commit/94b667fa7d0bcd30b2062d819b8c6c6e9b421cf5
Let me know if I should change something!
Updated by Boone Gorges almost 9 years ago
- Status changed from Assigned to Resolved
- Target version changed from Future release to 1.8.7
Yeah, this looks good to me.
Ray, do you think we ought to roll this into the mainline plugin, as distributed with Commons In A Box? I lean yes.
Pulling into the 1.8.7 release to address Sam's UX review. I'm also going to mark as Resolved to clear the milestone, but Dan, feel free to use the ticket for further discussion about changes in the Github version of the plugin.
Updated by Raymond Hoh almost 9 years ago
Ray, do you think we ought to roll this into the mainline plugin, as distributed with Commons In A Box? I lean yes.
Yeah, we should.
We should give Dan push access to the External Group Blogs repo:
https://github.com/cuny-academic-commons/external-group-blogs
Updated by Boone Gorges almost 9 years ago
Thanks, Ray. Dan, you should have push access to the Github repo that Ray linked to. Please feel free to make the change there, too - and leave a note in readme.txt to that effect. Thanks!