search

nus-oss-test / testrepo4

TEAMMATES system is online at
http://teammatesv4.appspot.com
0 stars 0 forks source link

UI has no common headers for <head></head> part #1715

Closed damithc closed 10 years ago

damithc commented 10 years ago

From Gigi...@gmail.com on March 02, 2014 10:27:50

What can be improved on the system? What can be added to the system? Add the common header(s) for part. Otherwise, if frontend programmer wants to add lines in all UI head's part, there are 40+ files need to edit.

By using this enhancement, we can edit the whole UI headers by only editing one file. E.g. imagine we need to add a js file for the whole UI Please specify: 1) Exact page to implement/improve feature 40+ UI files with 2) The details of the feature Use one template engine to build a header, then other pages just simply include the header.

Original issue: http://code.google.com/p/teammatespes/issues/detail?id=1660

damithc commented 10 years ago

From dam...@gmail.com on March 01, 2014 19:48:02

Kai, do you want to give this one a try next?

Status: Accepted
Labels: Difficulty-High Priority-Medium

damithc commented 10 years ago

From Gigi...@gmail.com on March 01, 2014 19:55:35

Can :) but it will take me some time

damithc commented 10 years ago

From Gigi...@gmail.com on March 01, 2014 20:51:01

Major steps required:

  1. need to figure out a full list of exact pages to modify for this enhancement
  2. try to find out the common elements in the tag. Only common ones will be included
  3. then build a commonHeader.jsp to include common elements in the tag
  4. change html files (eg index.html, contact.html ...) to jsp first, which will include commonHeader.jsp inside tag
  5. then change the rest in the list
  6. testing ..
damithc commented 10 years ago

From dam...@gmail.com on March 01, 2014 23:37:50

This might take a while. JSP includes are tricky. We'll KIV for now.

damithc commented 10 years ago

From dam...@gmail.com on May 01, 2014 00:53:49

Closing temporarily. To be reopened.

Status: Done
Labels: Temp

damithc commented 10 years ago

From dam...@gmail.com on May 01, 2014 21:32:06

Changing the status back

Status: Accepted
Labels: -Temp