[MPlayer-dev-eng] GUI (GTK+) cleanup?

[Andras Mohari]

On Wed, Oct 30, 2002 at 09:42:21 +0100, Zoltan Ponekker wrote:

Na, akkor en is magyarul. (Sorry to the others!)

> > * Removed unnecessary gtk_widget_set_name(), gtk_widget_ref() and
> >   gtk_object_set_data_full() calls.  Reason:  Widget names are
> >   (nearly?) never used in the code, so gtk_widget_set_name() can be
> >   dropped.  gtk_object_set_data_full() is often used to gtk_widget_unref()
> >   the widget.  Do not call gtk_widget_ref() in the first place (why do we
> >   need a ref count of 2?), so both calls can be removed.
> 
> Jo, ezt anno glade raktabe, reszemrol elfert.

Nem tudom minek rakja bele, folosleges szemet.

> > * Removed "show" and "hide" signal handlers and changed the "destroy"
> >   handler to simply NULLify the AboutBox variable.  gtkVAboutBox is
> >   also removed as it's not needed to check whether the window is
> >   visible or not -- you have AboutBox for that purpose.
> 
> Es ki is nullazza azt a valtozot gtk_*_destroy ?

Igen, pontosabban a "destroy" kezelojenek a gtk_widget_destroyed()
funkcio van beallitva, annak meg at van adva az AboutBox valtozo cime,
amit majd kinullaz. Annak a funkcionak direkt ez a feladata, csak a GTK+
1.2-ben nincs dokumentalva. :)

> > * Made the OK button use the "clicked" signal instead of "released".
> 
> Miert jobb click, mint release ?

Mert ha nem a gomb felett engeded el az egergombot, akkor nem csinal
semmit.

> > * Removed GtkFrame widgets; the whole stuff is simply packed in a GtkVBox.
> >   (I think frames are used _way to much_.  Is it intentional?  Looks like
> >   they are used for decoration, but frames are for grouping widgets
> >   visually.  Windows look kinda "overcrowded" to me this way.)
> 
> Nem. Az kell. Diszito jellege van, es nem ugy nez ki, mint egy atlagos
> hirtelen osszehanyt ocsmany gtk - s ize, mint pl. glade.

Hm, izlesek es pofonok. En pont forditva gondolom. :)
Nehany helyen jol jonnek, pl. logikailag egyuve tartozo widgetek
bekeretezesere, de egyebkent...meg amugy is csak egy gyerekuk lehet,
igy kulon boxokat kell belejuk pakolni tobb widget hasznalatahoz.

> > * Changed window's wmclass to "about.MPlayer".  It's not a good idea to
> >   translate the wmclass name (MSGTR_About)...
> 
> Neeee. Mert az a class fog megjelenni talca es egyebb marhasagokon. Es ez
> meg ronda is, es miert ne lehetne az is internacionalizalt ?

Az ICCCM szerint a wmclass-t amugy sem kellene piszkalni, az alkalmazas
minden ablakanak ugyanazt a wmclass-t kellene hasznalni. Mondjuk ez csak
egy dolog. :)  Ha pedig valaki be akarja allitani, hogy az
"About.MPlayer" ablakokat az ablakkezeloje pl. ne dekoralja, akkor
a dolog nem fog mukodni, ha eppenseggel magyar, lengyel stb. forditast
hasznal... Reszemrol ez az igazi erv. :)
(Es hol jelenik meg a wmclass? Az a progi se normalis, amelyik ezt
jeleniti meg a talcan. Miert nem a WM_NAME vagy WM_ICON_NAME (vagy mi
a nyavaja) vackokat hasznalja?)
 
> > * Removed unecessary line continuation characters from the about box
> >   string and replaced text length with -1.
> 
> Kozmetika nem nagyon kene ... ez nem xine.

OK, csak arra gondoltam, igy nem kell a szoveg maceralasakor azokkal is
bajlodni.

-- 
Andras Mohari
mayday at mail.nagyatad.hu