Skip to content

Favicons in Menu and Growl notification#7

Open
netj wants to merge 18 commits intogodenzim:masterfrom
netj:master
Open

Favicons in Menu and Growl notification#7
netj wants to merge 18 commits intogodenzim:masterfrom
netj:master

Conversation

@netj
Copy link

@netj netj commented Apr 19, 2011

Hi,

I've modified Reader Notifier Reloaded to show Favicon of each feed item's site in the menu and Growl notifications. Please pull my work if you're interested! I'm providing prebuilt binary of my version from my downloads page, but I would be happier if my enhancements get distributed via the auto-update feature. :)

Thanks,

~Jaeho

netj and others added 13 commits August 15, 2010 13:38
 - I think it's better to show the alert only when the action was
   initiated by the user.  Internet connection may come and go, but
   Reader Notifier disturbs the user with an alert even though he/she is
   not interacting with it.
 - Did `genstrings -o English.lproj Controllers/*.m` before adding
   Korean translations.
 - Localization Suite from http://www.loc-suite.org/ helped a lot.
 - Displaying the favicon of each feed item's linked site in Growl
   notifications.
 - Favicons are fetched all from Google's cache service,
   s2.googleusercontent.com instead of bothering each host.  The only
   problem with this approach is that all favicons, even though some
   small number of sites provide larger ones, become 16x16 which might
   not look so smooth with most of the Growl styles.
A textfield label was blocking its popup button :(
 - It seems showing favicons in the menu improves the overall usability a lot.
 - FeedBurner and alikes obfuscate URLs identifiable with the host part
   being one among feedproxy.google.com, feeds.gawker.com, etc.
 - Now displays favicons in notifications even when minimal function.
@tyilo
Copy link

tyilo commented Sep 25, 2011

The favicon for iDownloadBlog (http://feeds.feedburner.com/iphonedlb) is the standard feedburner icon, not the icon of the blog.
In Reader Notifier Reloaded:
Alt Text
In Google Reader:
Alt Text

netj added 5 commits February 2, 2013 11:59
Conflicts:
	Reader Notifier.xcodeproj/project.pbxproj
- Rewrote README (in Markdown).
- New place to distributed built versions. (file downloads are no longer
  supported at Github)
- Forked Sparkle appcast: new key, and URL pointing to my Dropbox.
- Wrote a small shell script for simplifying the process of creating zip
  archives, signing and deploying updates.
- Merged alanshutko's Google Reader URL fix.
- Growl framework 2.0.1.
  Notification Center support comes for free on OS X 10.8!
- Sparkle Korean language support.
to 2.3.1 to see if Sparkle updates actually work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants