Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: tests/test_site/pages/sitemap.tmpl

Issue 29511661: Noissue - Adds sort filter for sitemap to fix inconsistent rendering (Closed)
Patch Set: Created Aug. 10, 2017, 5:25 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tests/expected_output/sitemap ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 template = empty 1 template = empty
2 product = foo 2 product = foo
3 title = test_metadata 3 title = test_metadata
4 tags = [popular, bar] 4 tags = [popular, bar]
5 <h1> Unfiltered </h1> 5 <h1> Unfiltered </h1>
6 <ul> 6 <ul>
7 {%- for unfiltered_page in get_pages_metadata() -%} 7 {%- for unfiltered_page in get_pages_metadata()|sort -%}
8 <li>title : {{ unfiltered_page['page']}} </li> 8 <li>title : {{ unfiltered_page['page']}} </li>
9 {%- endfor %} 9 {%- endfor %}
10 </ul> 10 </ul>
11 11
12 <h1> Filtered </h1> 12 <h1> Filtered </h1>
13 <ul> 13 <ul>
14 {%- for filtered_page in get_pages_metadata({'tags': ['popular', 'bar']}) -% } 14 {%- for filtered_page in get_pages_metadata({'tags': ['popular', 'bar']})|so rt -%}
15 <li>title : {{ filtered_page['page'] }} </li> 15 <li>title : {{ filtered_page['page'] }} </li>
16 {%- endfor %} 16 {%- endfor %}
17 </ul> 17 </ul>
18 18
19 <h1> String Filter For Listy Option </h1> 19 <h1> String Filter For Listy Option </h1>
20 <ul> 20 <ul>
21 {%- for filtered_page in get_pages_metadata({'tags': 'popular'}) -%} 21 {%- for filtered_page in get_pages_metadata({'tags': 'popular'})|sort -%}
22 <li>title : {{ filtered_page['page'] }} </li> 22 <li>title : {{ filtered_page['page'] }} </li>
23 {%- endfor %} 23 {%- endfor %}
24 </ul> 24 </ul>
OLDNEW
« no previous file with comments | « tests/expected_output/sitemap ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld