Use Granite About dialog

Bug #1166026 reported by Cassidy James Blaede
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
pomidor
New
Medium
Unassigned

Bug Description

You should use Granite's About dialog instead of the GNOME one for consistency sake. This also gives you a help item inside the dialog instead of needing another item in the AppMenu.

Revision history for this message
Jaap Broekhuizen (jaapz-b) wrote :

You need to use Granite.Application too if you want to use the Granite.Widgets.AboutDialog to it's full extent, which I don't yet really know how to do in python yet.

Revision history for this message
dkotrada (dkotrada) wrote :
Changed in pomidor:
importance: Undecided → Low
dkotrada (dkotrada)
Changed in pomidor:
importance: Low → Medium
Revision history for this message
Jaap Broekhuizen (jaapz-b) wrote :

I think this will be easier to implement if you ditch Glade based ui and build the ui in python.

Revision history for this message
dkotrada (dkotrada) wrote :

You mean PyGTK ? There are no GTK3 support as I know.

Revision history for this message
dkotrada (dkotrada) wrote :

Here is the code for granite version where I am able to start an empty granite dialog. So far https://code.launchpad.net/~dkotrada/+junk/granite

Revision history for this message
Jaap Broekhuizen (jaapz-b) wrote :

Not PyGTK, but the Gtk from gi.repository :)
And don't put your subclass of Application in AboutDialog.py.
The Application should be the center of your app, so if i were you i'd put your class Application(Granite.Application) in pomidor/__init__.py, and instantiate it in main().

Look at how it is done in Vala apps, it is very similar to that.

Revision history for this message
Jaap Broekhuizen (jaapz-b) wrote :

Maybe you should implement this in a new branch, then you can do a merge request to the main branch, and i can review your code :)

Revision history for this message
dkotrada (dkotrada) wrote : Re: [Bug 1166026] Re: Use Granite About dialog

I'll use this branch: https://code.launchpad.net/~dkotrada/+junk/granite

Thank you I'll try to put it in __init__.py

2013/4/25 Jaap Broekhuizen <email address hidden>

> Maybe you should implement this in a new branch, then you can do a merge
> request to the main branch, and i can review your code :)
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1166026
>
> Title:
> Use Granite About dialog
>
> Status in Pomidor Time Manager:
> New
>
> Bug description:
> You should use Granite's About dialog instead of the GNOME one for
> consistency sake. This also gives you a help item inside the dialog
> instead of needing another item in the AppMenu.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/pomidor/+bug/1166026/+subscriptions
>

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.