Review of "Persian Calendar" version 33

Details Page Preview

Displays Persian (Iranian/Jalali) calendar in the top panel It offers: 1. Displays the Persian/Iranian/Jalali calendar 2. Holiday indicator 3. Day change notifications 4. Converts dates between the Persian, Gregorian, and Hijri (lunar) calendars 5. Event listings: 5.1. Official solar events 5.2. Official lunar events 5.3. Official international events 5.4. Traditional Persian events 5.5. Notable Persian figures Please “rate” the project here and give it a “star” on GitHub. If you encounter any issues or have suggestions, feel free to open an issue there on GitHub!

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.