Ticket #1282 (closed enhancement: fixed)

Opened 3 years ago

Last modified 3 years ago

JS and WUI tidy up

Reported by: rgrp Owned by: rgrp
Priority: major Milestone: ckan-v1.5
Component: ckan Keywords:
Cc: Repository: ckan
Theme: none

Description (last modified by rgrp) (diff)

Consolidate js. Remove cruft.

  • Move stuff to vendor
    • Have all vendor libs local (?)
    • jquery, jquery-ui etc
  • Consolidate autocomplete to use jquery-ui
    • package autocomplete (groups)
    • tag autocomplete (package)
    • user autocomplete (authz group create)
      • turns out that authz group edit had autocomplete disabled so refactoring meant enabling ...
  • Move forms.css into main
  • Remove tag cloud from front page (can be example going forward)
  • Remove bookmarklet (since in wiki at: http://wiki.ckan.net/Managing_Data_Packages#How_Do_I_Install_the_CKAN_Bookmarklet.3F)
  • Move name slug generation into standard js and clean up

Possible:

  1. JS unit tests for the given pages ...
  2. Option to use external site to serve js and css (e.g. CDN)

Change History

comment:1 Changed 3 years ago by rgrp

  • Priority changed from awaiting triage to major
  • Description modified (diff)

comment:2 Changed 3 years ago by rgrp

  • Description modified (diff)

comment:3 Changed 3 years ago by rgrp

  • Description modified (diff)

comment:4 Changed 3 years ago by rgrp

  • Description modified (diff)

comment:5 Changed 3 years ago by rgrp

  • Description modified (diff)

comment:6 Changed 3 years ago by rgrp

  • Description modified (diff)

comment:7 Changed 3 years ago by rgrp

  • Status changed from new to closed
  • Resolution set to fixed

Done in cset:4b6838a820bb.

Have not done either of the optional items. JS testing for what we have is relatively hard and of dubious value.

Re CDN stuff requires work and of doubtful of immediate value since can just configure cache or main site to serve static stuff directly. (Likely more value from compressing js and css ...).

Note: See TracTickets for help on using tickets.