Ticket #2405 (assigned defect)

Opened 2 years ago

Last modified 23 months ago

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:1 Changed 2 years ago by rgrp

Assigning to ross for review and assignment.

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:

  1. 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.
  1. 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?
  1. 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.

comment:4 Changed 2 years ago by icmurray

  • Keywords [?d] added

comment:5 Changed 23 months ago by icmurray

  • Milestone changed from current-ckan-sprint-2012-06-25 to ckan-v1.8

Bump: Questions for client team above...

comment:6 Changed 23 months ago by icmurray

  • Milestone changed from ckan-v1.8 to ckan-v1.9
Note: See TracTickets for help on using tickets.