Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

API method to close Notification #2114

Closed
vaadin-bot opened this issue Feb 6, 2012 · 5 comments · Fixed by #10483
Closed

API method to close Notification #2114

vaadin-bot opened this issue Feb 6, 2012 · 5 comments · Fixed by #10483

Comments

@vaadin-bot
Copy link
Collaborator

Originally by @johannesh2

There should be an API method for closing a Notification shown with Window.showNotification(Notification). The method could be e.g. Window.hideNotification(Notification) or Notification.close().

This would be used in addition to the timeout. Example use case: If a timeout is 10 seconds for a notification, but the state of the application changes so that the notification is not relevant any more, it should be possible to hide the notification.

Imported from https://dev.vaadin.com/ issue #8351
@vaadin-bot
Copy link
Collaborator Author

Originally by @magi42

Marked #8382 as duplicate of this one.

@Legioth Legioth mentioned this issue Feb 27, 2017
Closed
@Thibstars
Copy link

@Legioth Is there a reason why this issue got closed?

@elmot
Copy link
Contributor

elmot commented Apr 14, 2017

This one is not closed. A red button indicates #8689 is closed.

@EduFrazao
Copy link

This issue is still relavant. Today I need to write a custom extension to display and possibly hide notifications programatically.

@arabedini
Copy link

opened this Issue on Feb 6, 2012 !

Artur- added a commit to Artur-/vaadin that referenced this issue Dec 29, 2017
@Artur-
Add Notification.hide() to hide a notification from the server
157b144 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114
@Artur- Artur- mentioned this issue Dec 29, 2017
Merged
Artur- added a commit to Artur-/vaadin that referenced this issue Jan 2, 2018
@Artur-
Add Notification.hide() to hide a notification from the server
1a76764 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114, vaadin#10481
Artur- added a commit to Artur-/vaadin that referenced this issue Jan 4, 2018
@Artur-
Add Notification.close() to hide a notification from the server
a5f29be 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114, vaadin#10481
Artur- added a commit to Artur-/vaadin that referenced this issue Jan 4, 2018
@Artur-
Add Notification.close() to hide a notification from the server
5b9156e 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114, vaadin#10481
Artur- added a commit to Artur-/vaadin that referenced this issue Jan 5, 2018
@Artur-
Add Notification.close() to hide a notification from the server
580a755 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114, vaadin#10481
Artur- added a commit to Artur-/vaadin that referenced this issue Jan 5, 2018
@Artur-
Add Notification.close() to hide a notification from the server
aa48dce 
Also fixes the problem that notifications were never removed on the server side

Fixes vaadin#2114, vaadin#10481
tsuoanttila pushed a commit that referenced this issue Jan 9, 2018
@Artur- @tsuoanttila
Add Notification.close() to hide a notification from the server (#10483)
59a2daf 
Also fixes the problem that notifications were never removed on the server side

Fixes #2114, fixes #10481
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add Notification.close() to hide a notification from the server Artur-/vaadin
5 participants
@EduFrazao @elmot @Thibstars @arabedini @vaadin-bot