[#SAK-15097] Add the Ability to embed a bit of HMTL from a Site Property in the Background Document of a Site

Sakai

Add the Ability to embed a bit of HMTL from a Site Property in the Background Document of a Site

Created: 09-Dec-2008 10:59 Updated: 27-Oct-2009 13:39

Component/s:

Portal

Affects Version/s:

2.6.0

Fix Version/s:

2.7.0

Time Tracking:

Not Specified

File Attachments:

SAK-15097-2-5-x.txt (2 kB)

2.6.x Status:	None
2.5.x Status:	None
2.4.x Status:	None

Description

« Hide

This will add a site property which is only accessible by the admins that allows the creation of a property that will contain an HTML snippet to be included in the page when the site navigation is shown. There are many possible uses of this feature - my first use will be to make it so that I can integrate hab.la chat in all my classes.

Since the feature will totally default to off and can only be triggered by admin action on a site - this is a feature that folks can experiment with.

The site property will be titled sakai:htmlInclude

All

Comments

Work Log

Change History

Subversion Commits

git Commits

Sort Order:

[ Permlink | « Hide ]

Charles Severance added a comment - 14-Dec-2008 22:07

This adds support for a new site property called sakai:htmlInclude - when
this property is present, it is included in the site navigation just below
the icon in a div titled "siteHTMLInclude" - if this property is not set
on a site nothing is changed. The typical use case for this might be to add
a bit of Javascript to support a tool such as hab.la to a site. Since
this is off by default and only the admin user can set this property, this
feature will have no impact unless the admin user sets the property on
the site.

Sending portal-impl/impl/src/java/org/sakaiproject/portal/charon/site/PortalSiteHelperImpl.java
Sending portal-render-engine-impl/pack/src/webapp/vm/defaultskin/includePageNav.vm
Transmitting file data ..
Committed revision 55285.

[ Permlink | « Hide ]

Charles Severance added a comment - 14-Dec-2008 22:07

All set - committed in 55285

[ Permlink | « Hide ]

Charles Severance added a comment - 15-Dec-2008 22:56

This is a 2-5-x patch implementing this feature. It could go safely back to 2-5-x with no impact. But it does allow folks to start experimenting with adding stuff like analytics to sites.

[ Permlink | « Hide ]

Steve Swinsburg added a comment - 06-Jan-2009 10:28 - edited

Just adding my vote here about getting this back into the branches. Even though its a new feature, since it needs to be explicitly turned on for each site by an admin user, and the patch is so safe, as a branch manager I vote for it being ok.

I have applied this patch to a local instance of the 2.5.x branch, built and tested and there are no conflicting issues.