Ticket #2405 (assigned defect)
Improvements to data viewer embed
Reported by: | rgrp | Owned by: | icmurray |
---|---|---|---|
Priority: | major | Milestone: | ckan-v1.9 |
Component: | ckan | Keywords: | [?d] |
Cc: | ross, rgrp | Repository: | ckan |
Theme: | none |
Description
- Decent powered by icon for the Embed for the DataHub?? (make this text customizable and allow html ...)
- Current icon and text looks really bad
- Embed link uses uuid rather than dataset name (this goes to perma-url question ...)
- Data viewer embed url includes all the resource info (not needed -- just url and backend info ...)
- Embed extension urls do not respect original domain but instead use site_url (I'm guessing). E.g. datahub.io domain ends up with links to embed on thedatahub.org
Change History
comment:2 Changed 2 years ago by ross
- Owner changed from ross to icmurray
- Status changed from new to assigned
Suggest Sam might be able to provide you the icon
comment:3 Changed 2 years ago by icmurray
- Cc ross, rgrp added
- Milestone changed from ckan-v1.8 to current-ckan-sprint-2012-06-25
Questions for client team:
- Did you have anything in mind for improving the "Powered by CKAN" text and icon? I'll sort out the vertical alignment of the text and icon; and see if Sam has a better icon.
- The branded section itself is already customizable by a snippet (https://github.com/okfn/ckan/blob/master/ckan/templates/snippets/data-viewer-embed-branded-link.html). Is that enough? Or are you asking for something else?
- Personally I think the URI should use the resource uuid, and not name. And from recent discussions on ckan-dev, I think the consensus is the same. How important is this for you? Perhaps it's something that could be overriden in the datahub extension.
Note: See
TracTickets for help on using
tickets.
Assigning to ross for review and assignment.