Ticket #2704 (closed enhancement: fixed)
add list of resources to edit dataset page sidebar
Reported by: | shevski | Owned by: | johnmartin |
---|---|---|---|
Priority: | minor | Milestone: | demo phase 5 |
Component: | ckan | Keywords: | demo-theme |
Cc: | Repository: | ckan | |
Theme: | none |
Description
when editing a dataset (e.g. here: http://s031.okserver.org:2375/en/dataset/edit/afghanistan-election-data) add sidebar list of resources for user to click & browse through taking you to the relevant resource edit pages
Change History
comment:3 Changed 22 months ago by toby
@ira,
this isn't really needed then! there is no point doing work if we need to do it again editing resources is easy enough now anyway
I suggest we just close this ticket as a won't fix and #2705 as well while we're at it.
I'm also not happy about these being added to the phase 2 milestone when we are at the end of finishing it and these are not bugfixes or critical functionality
comment:4 Changed 22 months ago by toby
- Milestone changed from demo phase 2 to demo phase 3
need some thought
comment:5 Changed 21 months ago by aron.carroll
This relates to #2705. We need to decide what happens when the user clicks on one of the sidebar links.
At the moment they'll get a warning if they've edited the form telling them they'll lose thier changes. Perhaps we should submit the form using ajax, showing a spinner to the user, then send them on to the next resource.
This is the same situation as if we add an "add new resource" button to the listing.
comment:6 Changed 21 months ago by toby
@aron,
I don't like auto submit as people may think (reasonably) that their changes are discarded.
We could catch all clicks and then ask the user if they've changed something but this feels a little crazy - better than the browser dialog. hmmm
maybe we should discuss this ticket with the other backlog of undecided/unsure
personally I think we need to review the whole edit/view/delete logic as it feels real poor from my perspective - I need to edit something to delete it - that is a wtf imho
let's work out what we are doing before wasting energy on this
@ira,
again not sure this makes sense from a ux perspective
@aron,
reassign to me if you want it doing should be trivial