Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#2306 closed change (fixed)

Add TOC macro to web.adblockplus.org

Reported by: kzar Assignee: kzar
Priority: P3 Milestone:
Module: Websites Keywords:
Cc: saroyanm Blocked By:
Blocking: #2118 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/5158802752012288/

Description

Background

We are working to replace the the anwtoc Anwiki table of contents tags with something that will work after our website has been migrated.

What to change

Create a new toc macro that takes a page name and optional title_tag and outputs the page's TOC HTML based on the result of combination of the get_page_content global function and the toclist filter. Have it be available for include from any template that needs it.

Change History (5)

comment:1 Changed 5 years ago by saroyanm

  • Priority changed from Unknown to P3

comment:2 Changed 5 years ago by kzar

  • Status changed from new to reviewing

comment:3 Changed 5 years ago by kzar

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

comment:5 Changed 4 years ago by Ross

  • Verified working set

Implemented and working.

Tested by:
Running the website converter script as per the read me and checking the TOC output for the mentioned pages + serving pages from CMS.

Tested with:
www / r18024
sitescripts / r501
web.adblockplus.org / r68
website-converter / master / 105208883c4336a3788820fe9ab379cf639f8387

Note: See TracTickets for help on using tickets.