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.

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

Redesign popup window #61

Closed
cyberbit opened this Issue Jul 24, 2015 · 4 comments

Comments

Projects
None yet
1 participant
@cyberbit
Owner

cyberbit commented Jul 24, 2015

Add nice little tabs, an options link, and other shizzle.

@cyberbit cyberbit added the feature label Jul 25, 2015

@cyberbit cyberbit added this to the Clean Slate (Community) milestone Jul 25, 2015

@cyberbit cyberbit self-assigned this Jul 25, 2015

@cyberbit cyberbit modified the milestones: Clean Slate (Community), Clean Slate (TBD), Clean Slate (Options) Aug 3, 2015

@cyberbit cyberbit added the inwork label Dec 21, 2015

@cyberbit

This comment has been minimized.

Show comment
Hide comment
@cyberbit

cyberbit Dec 28, 2015

Owner

Popup window redesigned, but I don't think it's really necessary to load the notifications there, as the desktop notifications now have actionable content. A better option might be to have several links that do some common actions:

  • Show all notifications (see #83)
  • Clear notifications
  • Open options page
Owner

cyberbit commented Dec 28, 2015

Popup window redesigned, but I don't think it's really necessary to load the notifications there, as the desktop notifications now have actionable content. A better option might be to have several links that do some common actions:

  • Show all notifications (see #83)
  • Clear notifications
  • Open options page
@cyberbit

This comment has been minimized.

Show comment
Hide comment
@cyberbit

cyberbit Dec 28, 2015

Owner

Added basic "Show Notifications" link that does what it's supposed to do. However, storage of the notification data needs to be moved into localStorage, as the event page is unloaded periodically for performance reasons, causing the current storage mechanism to be useless.

Owner

cyberbit commented Dec 28, 2015

Added basic "Show Notifications" link that does what it's supposed to do. However, storage of the notification data needs to be moved into localStorage, as the event page is unloaded periodically for performance reasons, causing the current storage mechanism to be useless.

@cyberbit

This comment has been minimized.

Show comment
Hide comment
@cyberbit

cyberbit Dec 28, 2015

Owner

Moved notification storage into localStorage, working as expected now.

Owner

cyberbit commented Dec 28, 2015

Moved notification storage into localStorage, working as expected now.

@cyberbit

This comment has been minimized.

Show comment
Hide comment
@cyberbit

cyberbit Dec 29, 2015

Owner

Show/clear all notifications buttons now working as expected. Also added options link.

Owner

cyberbit commented Dec 29, 2015

Show/clear all notifications buttons now working as expected. Also added options link.

@cyberbit cyberbit added ready and removed inwork labels Dec 29, 2015

@cyberbit cyberbit removed the ready label Dec 30, 2015

@cyberbit cyberbit closed this Dec 30, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment