Navigation Menus - Add site-wide primary navigation menu #3715
Assignees
Comments
defstat
added a commit
to defstat/pkp-lib
that referenced
this issue
May 20, 2018
defstat
added a commit
to defstat/omp
that referenced
this issue
May 20, 2018
defstat
added a commit
to defstat/ojs
that referenced
this issue
May 20, 2018
|
PRs: pkp-lib: #3716 |
defstat
added a commit
to defstat/pkp-lib
that referenced
this issue
May 22, 2018
|
@NateWr I fixed the review issue and I made the PR on bootstrap 3 plugin. Just pointing out that the logic behind showing or not the search control is different between the default theme and the bootstrap theme. Specifically the default theme shows the search control if the user is within a context. The bootstrap theme shows the search control if there are any journals configured. Should we keep that difference? bootstrap3: NateWr/bootstrap3#85 @Vitaliy-1 This issue/changes may concern your pkp/healthSciences theme development as well? |
NateWr
added a commit
that referenced
this issue
May 22, 2018
[pkp-lib] #3715 Change header file to add primary nm at side-wide level
|
Thanks @defstat. Can I close this? |
|
(And thanks I've filed an issue on the Bootstrap repo to look at it.) |
|
@NateWr Yes please. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the primary navigation menu doesn't show at the site level (for example at the first page of an OJS site with more than one journals), even though the site-wide navigation menu tool allows the addition of a primary navigation menu.
The text was updated successfully, but these errors were encountered: