Merge Request #33

← To merge requests From btn-new-post into master
Merged
noosfero-themes/angular-theme!33
Created by Michel Felipe

Fixes Bug - Show new post btn

Was remove the condition ng-show="vm.parentId" constraint to show this button.

A more complete solution, in my opinion, is:

  • Use hooks or hostspot (whatever) to add new buttons to My Item Navbar menu, whether buttons were added by plugin or by core components.

This change Fixes #86 issue.

Assignee: None
Milestone: None

Merged by Evandro Junior

Commits (1)
2 participants
Showing 1 changed file   Show diff stats
src/app/article/content-viewer/navbar-actions.html
  Diff comments   Edit   View file @0f90777
@@ -5,7 +5,7 @@ @@ -5,7 +5,7 @@
5 <i class="fa fa-caret-down"></i> 5 <i class="fa fa-caret-down"></i>
6 </a> 6 </a>
7 <ul class="dropdown-menu" uib-dropdown-menu ng-show="vm.profile"> 7 <ul class="dropdown-menu" uib-dropdown-menu ng-show="vm.profile">
8 - <li ng-show="vm.parentId"> 8 + <li>
9 <a href="#" ui-sref="main.cms({profile: vm.profile.identifier, parent_id: vm.parentId})"> 9 <a href="#" ui-sref="main.cms({profile: vm.profile.identifier, parent_id: vm.parentId})">
10 <i class="fa fa-file fa-fw fa-lg"></i> {{"navbar.content_viewer_actions.new_post" | translate}} 10 <i class="fa fa-file fa-fw fa-lg"></i> {{"navbar.content_viewer_actions.new_post" | translate}}
11 </a> 11 </a>
@@ -17,6 +17,5 @@ @@ -17,6 +17,5 @@
17 </li> 17 </li>
18 </ul> 18 </ul>
19 </li> 19 </li>
20 -  
21 -</ul>  
22 20
  21 +</ul>