Review of "Persian Calendar" version 33

Details Page Preview

Shows Persian date in the top panel. It shows: 1. Persian calendar 2. It can show, today is a holiday or not! 3. Show notification onDayChanged! 4. Date converter between Persian, Gregorian and Lunar Hijri 5. Events: 5.1. Official solar events. 5.2. Official lunar events. 5.3. Official international events. 5.4. Traditional Persian events. 5.5. Persian personages. Please “rate” here and “star” the project on GitHub. Please open an issue on GitHub if you found something or have an idea!

Extension Homepage
https://github.com/omid/Persian-Calendar-for-Gnome-Shell

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Previous Reviews on this Version

Meng Zhuo waiting for author
PLEASE disconnect all the signals
omid posted a review
I didn't change those parts in this version... how and whee should I do that?
Meng Zhuo waiting for author
Your can check on http://blog.mecheye.net/2012/02/requirements-and-tips-for-getting-your-gnome-shell-extension-approved/ Like line 43: Schema.connect('changed::color', Lang.bind( you should assign to an variable and disconnect it in the disable( destory ) function. example(code is not standard ): let schema_change_signal = Schema.connect('changed::color', Lang.bind(..)) in disable: Schema.disconnect(schema_change_signal); ------------------------------------------- I am a new reviewer in extension site, any problem is welcome, you can contact me at #gnome-shell IRC channel.
omid posted a review
Thanks a lot mengzho... please check next commit :-)
Meng Zhuo rejected
Reject for newer version.