Skip to content

Commit

Permalink
Removing notification count from page title after seen #603
Browse files Browse the repository at this point in the history
  • Loading branch information
andystrobel committed Mar 24, 2015
1 parent c6ee638 commit 7cdb54a
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
8 changes: 2 additions & 6 deletions css/style.css
Expand Up @@ -863,9 +863,11 @@ textarea.placeholder {
}
#space-menu-dropdown {
width: 300px !important;
max-height: 250px !important;
}
#search-menu-dropdown {
width: 300px !important;
max-height: 250px !important;
}
.notifications {
position: inherit !important;
Expand All @@ -886,9 +888,3 @@ textarea.placeholder {
font-size: 20px !important;
}
}
@media screen and (max-width: 768px) {
#space-menu-dropdown .media-list,
#search-menu-dropdown .media-list {
max-height: 250px;
}
}
4 changes: 2 additions & 2 deletions css/style.less
Expand Up @@ -1131,12 +1131,12 @@ input.placeholder, textarea.placeholder {

#space-menu-dropdown {
width: 300px !important;
max-height: 250px; !important;
max-height: 250px !important;
}

#search-menu-dropdown {
width: 300px !important;
max-height: 250px; !important;
max-height: 250px !important;
}

.notifications {
Expand Down
5 changes: 5 additions & 0 deletions protected/modules_core/notification/widgets/views/list.php
Expand Up @@ -46,6 +46,11 @@ function markNotificationsAsSeen() {
// hide notification badge at the top menu
$('#badge-notifications').css('display', 'none');
$('#mark-seen-link').css('display', 'none');

// remove notification count from page title
var pageTitle = $('title').text().replace(/\(.+?\)/g, '');
$('title').text(pageTitle);

}});
}

Expand Down

0 comments on commit 7cdb54a

Please sign in to comment.