UMB-CS682-Team-02 / tracker

0 stars 1 forks source link

Fix page title displayed by browser when chart template is used. #52

Closed rouilj closed 5 months ago

rouilj commented 6 months ago

The current browser title when the chart template is used is:

List of Issues - Roundup Issue Tracker

There are a few problems:

  1. it's a chart not a list
  2. it may not be charting issues
  3. it would be nice to have the title reflect the group and other properties.

1 and 2 are fairly straight forward:

1 figure out where in the chart template the title for the page in the header is set. 2 determine what type of class is being displayed using the request object

then write the TAL to output the correct info.

3 is more tricky. It can require some advanced TAL or better, a new extension function in chart.py to generate the title including the group parameters.

Ideally the chart title and the page title should be close to each other. For example a chart title of "Issues grouped by priority and status" should have the page title the same as the chart title, or maybe: "chart: ".

If you create a new helper to generate the page title, consider using that method to also generate the chart title and refactor the handler() methods to use it to keep things consistent.