Skip to content
This repository has been archived by the owner on Apr 6, 2023. It is now read-only.

BREAKING CHANGE: show notification message #93

Conversation

GusBaamonde
Copy link
Contributor

  • Adapt the user-menu container to be able to show it by class and not by ID, reusing the style of the container

panel-notificaciones

@@ -11,8 +11,8 @@ $("body").on({
}
}, ".menu-main .submenu-item");

$("body").on('click', '#user-menu--open', function() {
$('.user-menu').toggleClass('open');
$("body").on('click', '.user-menu--open', function() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is not a breaking change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you're right

@GusBaamonde GusBaamonde changed the title feat: show notification message BREAKING CHANGE: show notification message Apr 26, 2019
@andresmoschini
Copy link
Member

Relacionado con: FromDoppler/doppler-webapp#148

@GusBaamonde GusBaamonde merged commit 538931b into FromDoppler:master Apr 29, 2019
@andresmoschini
Copy link
Member

🎉 This PR is included in version 1.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants