Closed smblott-github closed 8 years ago
@Narno Thoughts?
And obligatory XKCD:
Hello @smblott-github, thanks for your feedback.
OK, what is happening : GHN fetchs notifications asynchronously from the GH API while respecting the interval allowed between each call. If you receive an e-mail about a new notification and clicks on the icon (or use a keybord shortcut) between 2 calls, the GHN counter is not yest up-to-date.
About your possible fixes proposals:
Wholly revert #32.
I love this feature, I can't kill it! :-) (I think @sindresorhus has the last word)
Go to https://github.com/notifications?all=1 if there seems to be no new notifications.
Not sure it would be usable for most of user, maybe as an option?
Add a dynamic check as to whether there are new notifications.I like this idea, but I don't know what will be the impacts on performances and risk of API spam.
Maybe the best solution is working on the desktop notification feature? #25 :-)
I would revert #32. I do not like this behaviour.
Getting the notification count is fast. Why not just fetch the count when the user clicks the button before taking them to GitHub. That way we'll be sure to do the correct thing.
Getting the notification count is fast. Why not just fetch the count when the user clicks the button before taking them to GitHub.
That seems better than the status quo.
However, I'd still argue that GHN is not a bookmark for the github home page; it's for notifications. So this would be a better alternative target.
(Somebody argued that that might be confusing for users. No more confusing, I'd say, than dumping them in the home page -- which for at least some users is completely useless.)
Another proposal: Do something like https://github.com/Narno/Facebook-Notifier/blob/master/README.md (see the screenshot of the options' page).
Ok, let's revert to only going to the notifications page (don't see the point of going to ?all=1
). In hinsight, that makes the most sense. If you want to go to the front page, just use a bookmark.
Sorry @Narno. Not interested in adding an option for this (You could always do so in a fork, though).
No problem, I understand.
Now I'm waiting that some guys come here after the revert, to create issues : "wtf, why did you delete this funk*ng smart feature?!" :bowtie:
+1, it was confusing to all of a sudden be taken to GH home page.
It has already been decided. Just waiting for someone to do the actual work.
@smblott-github Help me out; How do you bind GitHub notifier to a key?
@rojepp Go to the extensions page (chrome://extensions/
), at the bottom on the right, click keyboard shortcuts
, then follow your nose.
Please consider reverting (or fixing up) #32 (click when there are no notifications goes to GitHub home page).
Bad use case:
Ctrl-Shift-G
(which I bind to GitHub Notifier).This is a really bad UX.
Possible fixes: