Skip to content

Commit

Permalink
chore(css): increase z-index of site menu class from 1 to 50
Browse files Browse the repository at this point in the history
BREAKING CHANGE:
To allow for usage of the z-index property for elements in the content area
without the More menu dropdown being displayed behind these elements
the z-index value in the elgg-menu site class has been increased to 50
  • Loading branch information
iionly committed Oct 22, 2015
1 parent c2e73ca commit 34af1d7
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
8 changes: 8 additions & 0 deletions docs/guides/upgrading.rst
Expand Up @@ -378,6 +378,14 @@ Elgg's sprites and most of the CSS classes beginning with ``elgg-icon-``
Usage of ``elgg_view_icon()`` is backward compatible, but static HTML using the ``elgg-icon``
classes will have to be updated to the new markup.

Increase of z-index value in elgg-menu-site class
-------------------------------------------------

The value of z-index in the elgg-menu-site class has been increased from 1 to 50 to allow for page elements
in the content area to use the z-index property without the "More" site menu's dropdown being displayed
behind these elements. If your plugin/theme overrides the elgg-menu-site class or views/default/elements/navigation.css
please adjust the z-index value in your modified CSS file accordingly.

Introduced third-party library for sending email
------------------------------------------------

Expand Down
2 changes: 1 addition & 1 deletion mod/aalborg_theme/views/default/elements/navigation.css
Expand Up @@ -177,7 +177,7 @@
left: 0;
top: 0;
position: relative;
z-index: 1;
z-index: 50;
}
.elgg-menu-site > li {
float: left;
Expand Down
2 changes: 1 addition & 1 deletion views/default/elements/navigation.css.php
Expand Up @@ -164,7 +164,7 @@
SITE MENU
*************************************** */
.elgg-menu-site {
z-index: 1;
z-index: 50;
}

.elgg-menu-site > li > a {
Expand Down

0 comments on commit 34af1d7

Please sign in to comment.