Provide a proper unique id and a review state class for the first lev…

…el tabs Fixes #184
plone · Feb 23, 2019 · a5693d5 · a5693d5
1 parent a1993c9
commit a5693d5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/news/184.bugfix b/news/184.bugfix
@@ -0,0 +1,2 @@
+Provide a proper unique id and a review state class for the first level tabs
+[ale-rt]
diff --git a/plone/app/layout/navigation/navtree.py b/plone/app/layout/navigation/navtree.py
@@ -376,7 +376,6 @@ class NavTreeProvider(ContentProviderBase):
     )
     _item_markup_template = (
         u'<li class="{id}{has_sub_class}">'
-        u'{opener}'
         u'<a href="{url}" class="state-{review_state}"{aria_haspopup}>{title}</a>{opener}'  # noqa: E 501
         u'{sub}'
         u'</li>'
@@ -426,9 +425,10 @@ def navtree(self):
             entry = tab.copy()
             entry.update({
                 'path': '/'.join((navtree_path, tab['id'])),
-                'uid': None,
-                'review_state': None,
+                'uid': tab['id'],
             })
+            if 'review_state' not in entry:
+                entry['review_state'] = None
             if 'title' not in entry:
                 entry['title'] = (
                     tab.get('name')
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		Provide a proper unique id and a review state class for the first level tabs
		[ale-rt]